Re: [Spice-devel] [xorg-x11-drv-qxl] add fix for invisible cursor after resize (#1116870)

2014-07-07 Thread Hans de Goede
Hi, On 07/07/2014 06:34 PM, Marc-André Lureau wrote: > commit 4a7b1661823eb94a7458941885e5edd4cb07ec32 > Author: Marc-André Lureau > Date: Mon Jul 7 18:28:27 2014 +0200 > > add fix for invisible cursor after resize (#1116870) > > ...rm-restore-cursor-after-resolution-change.patch | 45

Re: [Spice-devel] [PATCH spice-gtk] session: prefix proxy lookup errors

2014-07-07 Thread Alon Levy
On 07/03/2014 07:49 PM, Marc-André Lureau wrote: > Proxy errors are already reported with G_IO_ERROR_PROXY messages, > however proxy name lookup error can be more detailed. ACK > > https://bugzilla.redhat.com/show_bug.cgi?id=1115986 > --- > gtk/spice-session.c | 1 + > 1 file changed, 1 inserti

Re: [Spice-devel] [vdagentd] Add an option such that the daemon will exit after processing a single session.

2014-07-07 Thread Alon Levy
On 06/24/2014 12:05 AM, Jeremy White wrote: > ACK > Signed-off-by: Jeremy White > --- > src/vdagentd.c | 14 +- > 1 file changed, 13 insertions(+), 1 deletion(-) > > diff --git a/src/vdagentd.c b/src/vdagentd.c > index fa47997..b5c7d14 100644 > --- a/src/vdagentd.c > +++ b/src/v

Re: [Spice-devel] [PATCH win-vdagent] Don't refresh displays config when updating

2014-07-07 Thread Alon Levy
On 06/19/2014 08:20 PM, Marc-André Lureau wrote: > wnd_proc() is called during ChangeDisplaySettings(), when handling > monitors config. However, calling get_displays() will overwrite the > desired config. > > So, while updating from 1 to 4 enabled monitors, when the 2nd monitor > config is enable

Re: [Spice-devel] [vdagent] Make sure the child is able to connect to the X server before exiting the mainline.

2014-07-07 Thread Alon Levy
On 06/24/2014 12:05 AM, Jeremy White wrote: > This allows the vdagent to be used in the Xsetup phase of an xdm session; > otherwise, its X11 connection fails because of the greeter display grab. But could you make the comment contain the discussion where you explained this in more detail? > > Si

Re: [Spice-devel] [vdagent] Make sure the child is able to connect to the X server before exiting the mainline.

2014-07-07 Thread Alon Levy
On 06/24/2014 12:05 AM, Jeremy White wrote: > This allows the vdagent to be used in the Xsetup phase of an xdm session; > otherwise, its X11 connection fails because of the greeter display grab. > relies on the fact the socketpair fd's are never 0 (i.e. fd[1] below is never 0). Which I think is w

Re: [Spice-devel] [vdagentd] Add an explicit switch to signal fake uinput devices.

2014-07-07 Thread Alon Levy
On 06/23/2014 11:41 PM, Jeremy White wrote: > ACK > Signed-off-by: Jeremy White > --- > src/vdagentd.c | 11 +-- > 1 file changed, 5 insertions(+), 6 deletions(-) > > diff --git a/src/vdagentd.c b/src/vdagentd.c > index 1098fb6..fa47997 100644 > --- a/src/vdagentd.c > +++ b/src/vdag

Re: [Spice-devel] [PATCH xf86-qxl] drm: restore cursor after resolution change

2014-07-07 Thread Alon Levy
On 07/02/2014 01:22 PM, Marc-André Lureau wrote: > From: Marc-André Lureau > > The Spice server & qemu reset the cursor state when > changing resolution. Although X does restore the > cursor on framebuffer changes, it doesn't for crtc > config. Restoring the cursor here is the simplest > way to s