diff options
author | Eric Blake <eblake@redhat.com> | 2012-02-22 17:48:38 -0700 |
---|---|---|
committer | Eric Blake <eblake@redhat.com> | 2012-03-07 18:24:43 -0700 |
commit | 265457845fc51f197d9cc0854fa5e9164565f3bb (patch) | |
tree | 17c2ef2e48d336cf288b0612e5d14a7ce575c3fe /tests/xml2sexprdata/xml2sexpr-fv-net-netfront.xml | |
parent | xml: share 'unit' in RNG (diff) | |
download | libvirt-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.xml | 2 |
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> |