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/sexpr2xmldata/sexpr2xml-fv-net-ioemu.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/sexpr2xmldata/sexpr2xml-fv-net-ioemu.xml')
-rw-r--r--tests/sexpr2xmldata/sexpr2xml-fv-net-ioemu.xml4
1 files changed, 2 insertions, 2 deletions
diff --git a/tests/sexpr2xmldata/sexpr2xml-fv-net-ioemu.xml b/tests/sexpr2xmldata/sexpr2xml-fv-net-ioemu.xml
index 4dc218f7f..a1fb4a96c 100644
--- a/tests/sexpr2xmldata/sexpr2xml-fv-net-ioemu.xml
+++ b/tests/sexpr2xmldata/sexpr2xml-fv-net-ioemu.xml
@@ -1,8 +1,8 @@
<domain type='xen' id='3'>
<name>fvtest</name>
<uuid>b5d70dd2-75cd-aca5-1776-9660b059d8bc</uuid>
- <memory>409600</memory>
- <currentMemory>409600</currentMemory>
+ <memory unit='KiB'>409600</memory>
+ <currentMemory unit='KiB'>409600</currentMemory>
<vcpu>1</vcpu>
<os>
<type>hvm</type>