On Thu, Feb 23, 2012 at 08:56:50AM +0100, Matthias Bolte wrote: > 2012/2/23 Eric Blake <eblake@xxxxxxxxxx>: > > Make it obvious to 'dumpxml' readers what unit we are using, > > since our default of KiB (1024) differs from qemu's default of MiB. > > > diff --git a/tests/xml2vmxdata/xml2vmx-svga.xml b/tests/xml2vmxdata/xml2vmx-svga.xml > > index b7db4c8..9f93ef9 100644 > > --- a/tests/xml2vmxdata/xml2vmx-svga.xml > > +++ b/tests/xml2vmxdata/xml2vmx-svga.xml > > @@ -1,7 +1,7 @@ > > <domain type='vmware'> > > <name>minimal</name> > > <uuid>564d9bef-acd9-b4e0-c8f0-aea8b9103515</uuid> > > - <memory>4096</memory> > > + <memory units='KiB'>4096</memory> > > <os> > > <type>hvm</type> > > </os> > > Good idea, but why did you decide to name the attribute "units" > (plural) instead of "unit"? As a value can only have one unit the > attribute name "units" is misleading. Agreed, especially since <volume><allocation> already uses "unit" Christophe
Attachment:
pgp70yA22tnfn.pgp
Description: PGP signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list