[libvirt] [PATCH 0/2] qemu: Support for QXL heads

Martin Kletzander mkletzan at redhat.com
Thu Mar 10 14:20:59 UTC 2016


So this is an old, tiny series that was posted some time ago; actually
a lot of time ago.  I complained because of two things.  First one
being that there are no tests, so this series has them in.  The second
one was that this could break migration from old code since the
parameter was not used before.  So I coded up a migration flag,
handled all the special cases and then started testing it.  And while
testing it, I've found out that QEMU doesn't care at all about this
parameter being there and there is no point in keeping it
super-stable, especially when it can be changed on the fly.  So I
removed all that code and ended up with tiny series very similar to
previous attempts by various people.

Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1283207

Martin Kletzander (2):
  qemu: Check for qxl's max_outputs parameter
  qemu: Add support to QXL's max_outputs parameter

 src/qemu/qemu_capabilities.c                       |  4 ++
 src/qemu/qemu_capabilities.h                       |  2 +
 src/qemu/qemu_command.c                            |  8 ++++
 .../qemuxml2argv-video-qxl-heads.args              | 28 +++++++++++++
 .../qemuxml2argv-video-qxl-heads.xml               | 47 ++++++++++++++++++++++
 tests/qemuxml2argvtest.c                           |  8 ++++
 .../qemuxml2xmlout-video-qxl-heads.xml             | 47 ++++++++++++++++++++++
 tests/qemuxml2xmltest.c                            |  2 +
 8 files changed, 146 insertions(+)
 create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-video-qxl-heads.args
 create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-video-qxl-heads.xml
 create mode 100644 tests/qemuxml2xmloutdata/qemuxml2xmlout-video-qxl-heads.xml

--
2.7.2




More information about the libvir-list mailing list