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

Christophe Fergeau cfergeau at redhat.com
Thu Mar 10 14:46:42 UTC 2016


Hey,

On Thu, Mar 10, 2016 at 03:20:59PM +0100, Martin Kletzander wrote:
> 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.

I think the main problem was that if you upgrade from a QEMU which did
not care to a QEMU with the option and a libvirt which supports it,
suddenly your VMs are going to go from supporting multiple heads (4) to
only supporting one as older libvirt always add a 'heads=1' to the XML.

Christophe
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20160310/125062f3/attachment-0001.sig>


More information about the libvir-list mailing list