https://bugzilla.redhat.com/show_bug.cgi?id=1572491 Commit id '02129b7c0' added a single pagesElem for slightly different purposes. One usage was an output for host page size listing and the other for NUMA supported page sizes. For the former, only the pages unit and size are formatted, while for the latter the pages unit, size, and availability data is formatted. The virt-xml-validate would fail because it expected something extra in the host page size output. So split up pagesElem a bit and create pagesHost and pagesNuma for the differences. Modify some capabilityschemadata output to have the output - even though the results may not be realistic with respect to the original incarnation of the data. Signed-off-by: John Ferlan <jferlan@xxxxxxxxxx> --- docs/schemas/capability.rng | 30 +++++++++++++++++++----------- tests/capabilityschemadata/caps-test2.xml | 3 +++ tests/capabilityschemadata/caps-test3.xml | 6 ++++++ 3 files changed, 28 insertions(+), 11 deletions(-) diff --git a/docs/schemas/capability.rng b/docs/schemas/capability.rng index e1b7858540..66c5de62e5 100644 --- a/docs/schemas/capability.rng +++ b/docs/schemas/capability.rng @@ -129,7 +129,7 @@ </element> </zeroOrMore> <zeroOrMore> - <ref name='pagesElem'/> + <ref name='pagesHost'/> </zeroOrMore> </define> @@ -203,7 +203,7 @@ </optional> <zeroOrMore> - <ref name='pagesElem'/> + <ref name='pagesNuma'/> </zeroOrMore> <optional> @@ -484,17 +484,25 @@ </data> </define> - <define name='pagesElem'> + <define name='pagesHost'> <element name='pages'> - <optional> - <attribute name='unit'> - <ref name='unit'/> - </attribute> - </optional> - <attribute name='size'> - <ref name='unsignedInt'/> - </attribute> + <ref name='pagesElem'/> + </element> + </define> + <define name='pagesNuma'> + <element name='pages'> + <ref name='pagesElem'/> <ref name='unsignedInt'/> </element> </define> + <define name='pagesElem'> + <optional> + <attribute name='unit'> + <ref name='unit'/> + </attribute> + </optional> + <attribute name='size'> + <ref name='unsignedInt'/> + </attribute> + </define> </grammar> diff --git a/tests/capabilityschemadata/caps-test2.xml b/tests/capabilityschemadata/caps-test2.xml index 652fc71e0b..125a322998 100644 --- a/tests/capabilityschemadata/caps-test2.xml +++ b/tests/capabilityschemadata/caps-test2.xml @@ -24,6 +24,9 @@ <feature name='acpi'/> <feature name='ds'/> <feature name='vme'/> + <pages unit='KiB' size='4'/> + <pages unit='KiB' size='2048'/> + <pages unit='KiB' size='1048576'/> </cpu> <power_management> <suspend_mem/> diff --git a/tests/capabilityschemadata/caps-test3.xml b/tests/capabilityschemadata/caps-test3.xml index 479db30612..8294f44425 100644 --- a/tests/capabilityschemadata/caps-test3.xml +++ b/tests/capabilityschemadata/caps-test3.xml @@ -28,6 +28,9 @@ <feature name='acpi'/> <feature name='ds'/> <feature name='vme'/> + <pages unit='KiB' size='4'/> + <pages unit='KiB' size='2048'/> + <pages unit='KiB' size='1048576'/> </cpu> <power_management> <suspend_disk/> @@ -42,6 +45,9 @@ <cells num='2'> <cell id='0'> <memory unit='KiB'>12572412</memory> + <pages unit='KiB' size='4'>4011249</pages> + <pages unit='KiB' size='2048'>0</pages> + <pages unit='KiB' size='1048576'>0</pages> <cpus num='12'> <cpu id='0'/> <cpu id='2'/> -- 2.14.3 -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list
