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 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:2

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 al

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 >> sys

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

Re: [Spice-devel] [libvirt] [PATCH] qxl: Fix new function name for spice-server library

2015-07-20 Thread Martin Kletzander
On Mon, Jul 20, 2015 at 09:43:23AM +0100, Frediano Ziglio wrote: The new spice-server function to limit the number of monitors (0.12.6) changed while development from spice_qxl_set_monitors_config_limit to spice_qxl_max_monitors (accepted upstream). By mistake I post patch with former name. This

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 maxim

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] [PATCH v3] qemu: Use heads parameter for QXL driver

2015-07-09 Thread Martin Kletzander
On Tue, Jul 07, 2015 at 11:44:11AM +0200, Martin Kletzander wrote: On Mon, Jul 06, 2015 at 09:18:59AM +0100, Frediano Ziglio wrote: Allows to specify maximum number of head to QXL driver. The patch to support the "max_outputs" in Qemu is still not merged but I got agreement on the n

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

2015-07-07 Thread Martin Kletzander
ies.h | 1 + src/qemu/qemu_command.c | 5 + 3 files changed, 8 insertions(+) Changes from v2: - removed capability tests (Martin Kletzander) diff --git a/src/qemu/qemu_capabilities.c b/src/qemu/qemu_capabilities.c index 27686c3..68060cd 100644 --- a/src/qemu/qemu_capabilities.c +++ b/src

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

2015-06-24 Thread Martin Kletzander
On Wed, Jun 24, 2015 at 06:34:50AM -0400, Frediano Ziglio wrote: On Fri, Jun 12, 2015 at 11:57:31AM +0100, Frediano Ziglio wrote: >Allows to specify maximum number of head to QXL driver. > >Signed-off-by: Frediano Ziglio > >The patch to support the "max_outputs" in Qemu is still not merged bu

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

2015-06-24 Thread Martin Kletzander
On Fri, Jun 12, 2015 at 11:57:31AM +0100, Frediano Ziglio wrote: Allows to specify maximum number of head to QXL driver. Signed-off-by: Frediano Ziglio The patch to support the "max_outputs" in Qemu is still not merged but I got agreement on the name of the argument. Actually can be a compati

Re: [Spice-devel] [PATCH] spice-widget: Add spice_display_mouse_grab()

2013-08-28 Thread Martin Kletzander
On 08/27/2013 07:50 PM, Marc-André Lureau wrote: > Hi > > - Mensaje original - >> Similarly to spice_display_mouse_ungrab(), this is intedned to be used >> from outside code to force the display to grab the mouse (and >> keyboard). > > That sounds reasonable, although we probably want the

[Spice-devel] [PATCH] spice-widget: Add spice_display_mouse_grab()

2013-08-27 Thread Martin Kletzander
Similarly to spice_display_mouse_ungrab(), this is intedned to be used from outside code to force the display to grab the mouse (and keyboard). Signed-off-by: Martin Kletzander --- doc/reference/spice-gtk-sections.txt | 2 +- gtk/map-file | 1 + gtk/spice-gtk-sym-file