[libvirt] [PATCH v2 3/4] qemu: use newer -device video device in qemu commandline

Gerd Hoffmann kraxel at redhat.com
Wed Dec 12 15:41:17 UTC 2012


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




More information about the libvir-list mailing list