[prev in list] [next in list] [prev in thread] [next in thread]
List: libvir-list
Subject: Re: [Libvir] os/bootloader
From: John Levon <levon () movementarian ! org>
Date: 2007-12-09 3:22:57
Message-ID: 20071209032257.GA13228 () totally ! trollied ! org ! uk
[Download RAW message or body]
On Sun, Dec 09, 2007 at 12:32:06AM +0000, Daniel P. Berrange wrote:
> > <group>
> > <ref name='os'/>
> > <optional>
> > <ref name='bootloader'/>
> > </optional>
> > </group>
> > <group>
> > <ref name='bootloader'/>
> > <optional>
> > <ref name='os'/>
> > </optional>
> > </group>
> >
> > means I can specify either os, or bootloader, or both. The latter case
> > makes a lot of sense when using pygrub. However, src/xml.c doesn't
> > implement those semantics:
> >
> > 1219 if (!bootloader) {
> > 1220 if ((node = virXPathNode("/domain/os[1]", ctxt)) != NULL) {
> > 1221 /* Analyze of the os description, based on HVM or PV. */
> > 1222 str = virXPathString("string(/domain/os/type[1])", ctxt);
> >
> >
> > I think that we should just remove the "if (!bootloader) {" bit here -
> > agree?
> >
> > I'm looking at the schema and I don't see (at least)
> > domain/devices/input, which appears to be used in src/xml.c too - is the
> > schema out of date in 0.3.3 ? The tests don't pass xmllint!
>
> Sorry, yes its out of date wrt to this. If the schema doesn't match the
> code, then the schema is wrong. The code is definitive (at least for
> the Xen / KVM drivers).
I presume this applies to the second complaint, not the first?
regards
john
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic