]> xenbits.xensource.com Git - libvirt.git/commit
conf: domain: gfx: Iterate over graphics devices when doing validation
authorErik Skultety <eskultet@redhat.com>
Fri, 7 Dec 2018 13:53:46 +0000 (14:53 +0100)
committerErik Skultety <eskultet@redhat.com>
Wed, 12 Dec 2018 14:12:35 +0000 (15:12 +0100)
commitdd45c2710f6fd2d4f8a47f97960532d0e0091e7d
treeb90f863343ac763c5e8448ca1514378b21122db4
parent0d5244cc867155f7548e568f298799d639017698
conf: domain: gfx: Iterate over graphics devices when doing validation

The QEMU validation code for graphics has been in place for a while, but
because it is only executed from virDomainDeviceInfoIterateInternal, it
was never run, since the iterator expects the device to have boot info
which graphics don't have. The unfortunate side effect of this whole mess
was that a few capabilities were missing from the test suite (as commit
d8266ebe1 demonstrated with graphics-spice-invalid-egl-headless test),
which in turn meant that a few graphics tests which expected a failure
happily accepted any failure the test runtime returned which made them
succeed. The impact of this was that we then allowed to start a domain
with multiple OpenGL-enabled graphics devices.

This patch enables iteration over graphics devices. Unsurprisingly,
a few tests started to fail as a result, so fix those too.

Signed-off-by: Erik Skultety <eskultet@redhat.com>
Reviewed-by: John Ferlan <jferlan@redhat.com>
src/conf/domain_conf.c
tests/qemuxml2argvtest.c
tests/qemuxml2xmltest.c