summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorEric Blake <eblake@redhat.com>2012-02-22 17:48:38 -0700
committerEric Blake <eblake@redhat.com>2012-03-07 18:24:43 -0700
commit265457845fc51f197d9cc0854fa5e9164565f3bb (patch)
tree17c2ef2e48d336cf288b0612e5d14a7ce575c3fe /tests/xml2sexprdata/xml2sexpr-fv-net-netfront.xml
parentxml: share 'unit' in RNG (diff)
downloadlibvirt-265457845fc51f197d9cc0854fa5e9164565f3bb.tar.gz
libvirt-265457845fc51f197d9cc0854fa5e9164565f3bb.tar.bz2
libvirt-265457845fc51f197d9cc0854fa5e9164565f3bb.zip
xml: output memory unit for clarity
Make it obvious to 'dumpxml' readers what unit we are using, since our default of KiB for memory (1024) differs from qemu's default of MiB; and differs from our use of bytes for storage. Tests were updated via: $ find tests/*data tests/*out -name '*.xml' | \ xargs sed -i 's/<\(memory\|currentMemory\|hard_limit\|soft_limit\|min_guarantee\|swap_hard_limit\)>/<\1 unit='"'KiB'>/" $ find tests/*data tests/*out -name '*.xml' | \ xargs sed -i 's/<\(capacity\|allocation\|available\)>/<\1 unit='"'bytes'>/" followed by a few fixes for the stragglers. Note that with this patch, the RNG for <memory> still forbids validation of anything except unit='KiB', since the code silently ignores the attribute; a later patch will expand <memory> to allow scaled input in the code and update the RNG to match. * docs/schemas/basictypes.rng (unit): Add 'bytes'. (scaledInteger): New define. * docs/schemas/storagevol.rng (sizing): Use it. * docs/schemas/storagepool.rng (sizing): Likewise. * docs/schemas/domaincommon.rng (memoryKBElement): New define; use for memory elements. * src/conf/storage_conf.c (virStoragePoolDefFormat) (virStorageVolDefFormat): Likewise. * src/conf/domain_conf.h (_virDomainDef): Document unit used internally. * src/conf/storage_conf.h (_virStoragePoolDef, _virStorageVolDef): Likewise. * tests/*data/*.xml: Update all tests. * tests/*out/*.xml: Likewise. * tests/define-dev-segfault: Likewise. * tests/openvzutilstest.c (testReadNetworkConf): Likewise. * tests/qemuargv2xmltest.c (blankProblemElements): Likewise.
Diffstat (limited to 'tests/xml2sexprdata/xml2sexpr-fv-net-netfront.xml')
-rw-r--r--tests/xml2sexprdata/xml2sexpr-fv-net-netfront.xml2
1 files changed, 1 insertions, 1 deletions
diff --git a/tests/xml2sexprdata/xml2sexpr-fv-net-netfront.xml b/tests/xml2sexprdata/xml2sexpr-fv-net-netfront.xml
index f278f1fb6..efabc3633 100644
--- a/tests/xml2sexprdata/xml2sexpr-fv-net-netfront.xml
+++ b/tests/xml2sexprdata/xml2sexpr-fv-net-netfront.xml
@@ -6,7 +6,7 @@
<loader>/usr/lib/xen/boot/hvmloader</loader>
<boot dev='hd'/>
</os>
- <memory>409600</memory>
+ <memory unit='KiB'>409600</memory>
<vcpu>1</vcpu>
<on_poweroff>destroy</on_poweroff>
<on_reboot>restart</on_reboot>