On 12/11/12 19:25, Eric Blake wrote: >> > + bool qemu1dot2plus = qemuCapsGetVersion(caps) >= 1002000; > Eww. Version string comparison is wrong, as it is feasible that someone > could backport the improved command line to older qemu. We should > instead be going off of whether specific '-device XXX' is supported; > probably by checking for QEMU_CAPS_DEVICE_QXL. Isn't going to fly. -device VGA is present for a loooooooong time already, but only very recently it started to actually work. So checking for the devices being present isn't an option. The change which made it work is avi's memory api, which fixed a nasty initialization order issue. That change is invasive enougth that you don't have to worry about backports at all. cheers, Gerd -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list