Re: [libvirt] [PATCH 01/11] tests: fix some QXL capability combinations that doesn't make sense

2016-10-10 Thread Pavel Hrdina
On Sat, Oct 08, 2016 at 09:52:24AM -0400, John Ferlan wrote: > s/doesn't/don't > > On 09/30/2016 12:02 PM, Pavel Hrdina wrote: > > If one of QEMU_CAPS_DEVICE_QXL_VGA or QEMU_CAPS_DEVICE_QXL is set the > > other one will always be set as well because both devices are tied > > together in QEMU. > >

Re: [libvirt] [PATCH 01/11] tests: fix some QXL capability combinations that doesn't make sense

2016-10-08 Thread John Ferlan
s/doesn't/don't On 09/30/2016 12:02 PM, Pavel Hrdina wrote: > If one of QEMU_CAPS_DEVICE_QXL_VGA or QEMU_CAPS_DEVICE_QXL is set the > other one will always be set as well because both devices are tied > together in QEMU. And subsequent patches are about to make this a whole lot clearer, but in or

[libvirt] [PATCH 01/11] tests: fix some QXL capability combinations that doesn't make sense

2016-09-30 Thread Pavel Hrdina
If one of QEMU_CAPS_DEVICE_QXL_VGA or QEMU_CAPS_DEVICE_QXL is set the other one will always be set as well because both devices are tied together in QEMU. The change of args files is caused by the presence of capability QEMU_CAPS_DEVICE_VIDEO_PRIMARY which means it's safe to use "-device qxl-vga"