Re: [Spice-devel] [libvirt] [PATCH v4] qemu: Use heads parameter for QXL driver

2015-07-22 Thread Martin Kletzander
On Wed, Jul 22, 2015 at 10:14:27AM +0100, Daniel P. Berrange wrote: I'm not sure what you are suggesting by "features" here, but I'd prefer if we didn't introduce a chunk of XML which would contain an ever growing set of hacks. It seems sufficient for us to just record the libvirt version number

Re: [Spice-devel] [libvirt] [PATCH v4] qemu: Use heads parameter for QXL driver

2015-07-22 Thread Daniel P. Berrange
On Wed, Jul 22, 2015 at 09:59:00AM +0200, Martin Kletzander wrote: > On Tue, Jul 21, 2015 at 04:36:45PM +0100, Daniel P. Berrange wrote: > >On Tue, Jul 21, 2015 at 11:34:08AM -0400, Laine Stump wrote: > >>On 07/21/2015 09:41 AM, Daniel P. Berrange wrote: > >>> On Tue, Jul 21, 2015 at 03:35:50PM +02

Re: [Spice-devel] [libvirt] [PATCH v4] qemu: Use heads parameter for QXL driver

2015-07-22 Thread Martin Kletzander
On Tue, Jul 21, 2015 at 04:36:45PM +0100, Daniel P. Berrange wrote: On Tue, Jul 21, 2015 at 11:34:08AM -0400, Laine Stump wrote: On 07/21/2015 09:41 AM, Daniel P. Berrange wrote: > On Tue, Jul 21, 2015 at 03:35:50PM +0200, Martin Kletzander wrote: >> On Tue, Jul 21, 2015 at 01:50:22PM +0100, Dan

Re: [Spice-devel] [libvirt] [PATCH v4] qemu: Use heads parameter for QXL driver

2015-07-22 Thread Laine Stump
On 07/21/2015 09:41 AM, Daniel P. Berrange wrote: > On Tue, Jul 21, 2015 at 03:35:50PM +0200, Martin Kletzander wrote: >> On Tue, Jul 21, 2015 at 01:50:22PM +0100, Daniel P. Berrange wrote: >>> On Tue, Jul 21, 2015 at 11:44:27AM +0200, Martin Kletzander wrote: On Tue, Jul 21, 2015 at 09:36:55A

Re: [Spice-devel] [libvirt] [PATCH v4] qemu: Use heads parameter for QXL driver

2015-07-21 Thread Daniel P. Berrange
On Tue, Jul 21, 2015 at 11:34:08AM -0400, Laine Stump wrote: > On 07/21/2015 09:41 AM, Daniel P. Berrange wrote: > > On Tue, Jul 21, 2015 at 03:35:50PM +0200, Martin Kletzander wrote: > >> On Tue, Jul 21, 2015 at 01:50:22PM +0100, Daniel P. Berrange wrote: > >>> On Tue, Jul 21, 2015 at 11:44:27AM +

Re: [Spice-devel] [libvirt] [PATCH v4] qemu: Use heads parameter for QXL driver

2015-07-21 Thread Daniel P. Berrange
On Tue, Jul 21, 2015 at 03:35:50PM +0200, Martin Kletzander wrote: > On Tue, Jul 21, 2015 at 01:50:22PM +0100, Daniel P. Berrange wrote: > >On Tue, Jul 21, 2015 at 11:44:27AM +0200, Martin Kletzander wrote: > >>On Tue, Jul 21, 2015 at 09:36:55AM +0200, Christophe Fergeau wrote: > >>>On Mon, Jul 20,

Re: [Spice-devel] [libvirt] [PATCH v4] qemu: Use heads parameter for QXL driver

2015-07-21 Thread Daniel P. Berrange
On Tue, Jul 21, 2015 at 11:44:27AM +0200, Martin Kletzander wrote: > On Tue, Jul 21, 2015 at 09:36:55AM +0200, Christophe Fergeau wrote: > >On Mon, Jul 20, 2015 at 11:25:52AM +0200, Martin Kletzander wrote: > >>I spend all morning fixing this to be installed properly in the > >>system. Anyway, I f

Re: [Spice-devel] [libvirt] [PATCH v4] qemu: Use heads parameter for QXL driver

2015-07-21 Thread Martin Kletzander
On Tue, Jul 21, 2015 at 01:50:22PM +0100, Daniel P. Berrange wrote: On Tue, Jul 21, 2015 at 11:44:27AM +0200, Martin Kletzander wrote: On Tue, Jul 21, 2015 at 09:36:55AM +0200, Christophe Fergeau wrote: >On Mon, Jul 20, 2015 at 11:25:52AM +0200, Martin Kletzander wrote: >>I spend all morning fix

Re: [Spice-devel] [libvirt] [PATCH v4] qemu: Use heads parameter for QXL driver

2015-07-21 Thread Martin Kletzander
On Tue, Jul 21, 2015 at 08:41:33AM -0400, Frediano Ziglio wrote: On Tue, Jul 21, 2015 at 09:36:55AM +0200, Christophe Fergeau wrote: >On Mon, Jul 20, 2015 at 11:25:52AM +0200, Martin Kletzander wrote: >> I spend all morning fixing this to be installed properly in the >> system. Anyway, I finall

Re: [Spice-devel] [libvirt] [PATCH v4] qemu: Use heads parameter for QXL driver

2015-07-21 Thread Frediano Ziglio
> > On Tue, Jul 21, 2015 at 09:36:55AM +0200, Christophe Fergeau wrote: > >On Mon, Jul 20, 2015 at 11:25:52AM +0200, Martin Kletzander wrote: > >> I spend all morning fixing this to be installed properly in the > >> system. Anyway, I finally managed to make this work and found out the > >> guest

Re: [Spice-devel] [libvirt] [PATCH v4] qemu: Use heads parameter for QXL driver

2015-07-21 Thread Martin Kletzander
On Tue, Jul 21, 2015 at 09:36:55AM +0200, Christophe Fergeau wrote: On Mon, Jul 20, 2015 at 11:25:52AM +0200, Martin Kletzander wrote: I spend all morning fixing this to be installed properly in the system. Anyway, I finally managed to make this work and found out the guest I used for it is not

Re: [Spice-devel] [libvirt] [PATCH v4] qemu: Use heads parameter for QXL driver

2015-07-21 Thread Christophe Fergeau
On Mon, Jul 20, 2015 at 11:25:52AM +0200, Martin Kletzander wrote: > I spend all morning fixing this to be installed properly in the > system. Anyway, I finally managed to make this work and found out the > guest I used for it is not ready to have multiple monitors. Anyway, > looking at everythin

Re: [Spice-devel] [libvirt] [PATCH v4] qemu: Use heads parameter for QXL driver

2015-07-21 Thread Christophe Fergeau
Hi, On Mon, Jul 20, 2015 at 11:25:52AM +0200, Martin Kletzander wrote: > In the meantime, is the only thing this does limiting the maximum? Is > it there just to save some memory or why? Because otherwise I can't > see the use-case in that. I'm not saying there isn't one, just that I > can't fi

Re: [Spice-devel] [libvirt] [PATCH v4] qemu: Use heads parameter for QXL driver

2015-07-20 Thread Martin Kletzander
On Fri, Jul 17, 2015 at 06:02:25PM +0200, Christophe Fergeau wrote: On Fri, Jul 17, 2015 at 03:42:36PM +0200, Martin Kletzander wrote: On Fri, Jul 17, 2015 at 12:11:55PM +0200, Peter Krempa wrote: >On Fri, Jul 17, 2015 at 09:29:44 +0100, Frediano Ziglio wrote: >>Allows to specify maximum number

Re: [Spice-devel] [libvirt] [PATCH v4] qemu: Use heads parameter for QXL driver

2015-07-17 Thread Christophe Fergeau
On Fri, Jul 17, 2015 at 03:42:36PM +0200, Martin Kletzander wrote: > On Fri, Jul 17, 2015 at 12:11:55PM +0200, Peter Krempa wrote: > >On Fri, Jul 17, 2015 at 09:29:44 +0100, Frediano Ziglio wrote: > >>Allows to specify maximum number of head to QXL driver. > >> > >>Actually can be a compatiblity pr

Re: [Spice-devel] [libvirt] [PATCH v4] qemu: Use heads parameter for QXL driver

2015-07-17 Thread Frediano Ziglio
> > On Fri, Jul 17, 2015 at 12:11:55PM +0200, Peter Krempa wrote: > >On Fri, Jul 17, 2015 at 09:29:44 +0100, Frediano Ziglio wrote: > >> Allows to specify maximum number of head to QXL driver. > >> > >> Actually can be a compatiblity problem as heads in the XML configuration > >> was set by defau

Re: [Spice-devel] [libvirt] [PATCH v4] qemu: Use heads parameter for QXL driver

2015-07-17 Thread Martin Kletzander
On Fri, Jul 17, 2015 at 12:11:55PM +0200, Peter Krempa wrote: On Fri, Jul 17, 2015 at 09:29:44 +0100, Frediano Ziglio wrote: Allows to specify maximum number of head to QXL driver. Actually can be a compatiblity problem as heads in the XML configuration was set by default to '1'. Signed-off-by

Re: [Spice-devel] [libvirt] [PATCH v4] qemu: Use heads parameter for QXL driver

2015-07-17 Thread Peter Krempa
On Fri, Jul 17, 2015 at 09:29:44 +0100, Frediano Ziglio wrote: > Allows to specify maximum number of head to QXL driver. > > Actually can be a compatiblity problem as heads in the XML configuration > was set by default to '1'. > > Signed-off-by: Frediano Ziglio > --- > src/qemu/qemu_capabilitie