Re: [Spice-devel] [PATCH] RFC: client/x11: reuse existing platform_win

2011-08-10 Thread Christophe Fergeau
I reproduced this bug, the patch indeed fixes it (though I'm under the impression that X uses 100% CPU for several seconds on resolution changes). What it does also makes sense to me even if I'm no X expert, so ACK from me. Christophe On Thu, Jul 28, 2011 at 06:01:53PM +0200, Marc-André Lureau wr

Re: [Spice-devel] [PATCH] RFC: client/x11: reuse existing platform_win

2011-07-28 Thread Marc-André Lureau
Hi On Thu, Jul 28, 2011 at 6:43 PM, Alon Levy wrote: > On Thu, Jul 28, 2011 at 06:01:53PM +0200, Marc-André Lureau wrote: >> Each time a {Dynamic,MultyMon}Scren is created, a new platform_win is >> created to receive the notifications from X11. This looks very >> suspicious already. Since I am no

Re: [Spice-devel] [PATCH] RFC: client/x11: reuse existing platform_win

2011-07-28 Thread Alon Levy
On Thu, Jul 28, 2011 at 06:01:53PM +0200, Marc-André Lureau wrote: > Each time a {Dynamic,MultyMon}Scren is created, a new platform_win is > created to receive the notifications from X11. This looks very > suspicious already. Since I am not a xlib expert, I suppose that each > window will receive X

[Spice-devel] [PATCH] RFC: client/x11: reuse existing platform_win

2011-07-28 Thread Marc-André Lureau
Each time a {Dynamic,MultyMon}Scren is created, a new platform_win is created to receive the notifications from X11. This looks very suspicious already. Since I am not a xlib expert, I suppose that each window will receive X11 events. Whenever XRR ScreenChangeNotify event is received (caused by ex