On Tue, Apr 24, 2018 at 01:03:46PM +0200, Michal Privoznik wrote:
On 04/23/2018 04:50 PM, Martin Kletzander wrote:
> Signed-off-by: Martin Kletzander <mkletzan(a)redhat.com>
> ---
> docs/schemas/domaincaps.rng | 9 +++++++++
> src/conf/domain_capabilities.c | 3 +++
> src/conf/domain_capabilities.h | 1 +
> src/qemu/qemu_capabilities.c | 11 +++++++++++
> .../domaincapsschemadata/qemu_2.12.0-virt.aarch64.xml | 1 +
> tests/domaincapsschemadata/qemu_2.12.0.ppc64.xml | 1 +
> tests/domaincapsschemadata/qemu_2.12.0.s390x.xml | 1 +
> tests/domaincapsschemadata/qemu_2.12.0.x86_64.xml | 1 +
> .../domaincapsschemadata/qemu_2.6.0-virt.aarch64.xml | 1 +
> tests/domaincapsschemadata/qemu_2.6.0.aarch64.xml | 1 +
> tests/domaincapsschemadata/qemu_2.6.0.ppc64.xml | 1 +
> tests/domaincapsschemadata/qemu_2.6.0.x86_64.xml | 1 +
> tests/domaincapsschemadata/qemu_2.7.0.s390x.xml | 1 +
> tests/domaincapsschemadata/qemu_2.8.0-tcg.x86_64.xml | 1 +
> tests/domaincapsschemadata/qemu_2.8.0.s390x.xml | 1 +
> tests/domaincapsschemadata/qemu_2.8.0.x86_64.xml | 1 +
> tests/domaincapsschemadata/qemu_2.9.0-q35.x86_64.xml | 1 +
> tests/domaincapsschemadata/qemu_2.9.0-tcg.x86_64.xml | 1 +
> tests/domaincapsschemadata/qemu_2.9.0.x86_64.xml | 1 +
> 19 files changed, 39 insertions(+)
>
> diff --git a/docs/schemas/domaincaps.rng b/docs/schemas/domaincaps.rng
> index 39053181eb9a..dca5fa1c8fa0 100644
> --- a/docs/schemas/domaincaps.rng
> +++ b/docs/schemas/domaincaps.rng
> @@ -28,6 +28,9 @@
> <optional>
> <ref name='vcpu'/>
> </optional>
> + <optional>
> + <ref name='iothreads'/>
> + </optional>
> <optional>
> <ref name='os'/>
> </optional>
> @@ -53,6 +56,12 @@
> </element>
> </define>
>
> + <define name='iothreads'>
> + <element name='iothreads'>
> + <empty/>
> + </element>
> + </define>
Shouldn't this be <iothreads supported='yes'/> to be consistent with
the
rest of 'devices'?
I guess it makes sense to differentiate between QEMU not supporting I/O
threads and libvirt not being able to expose the support info in
capabilities. v2 it is
Michal