[PATCH xserver] glamor: Limit outstanding drawing queue with glClientWaitSync

2016-06-04 Thread Keith Packard
The X server doesn't (generally) use SwapBuffers, so we need some other mechanism to avoid flooding the device with queued rendering requests when the server is busy. Limit the amount of outstanding rendering requests queued to the driver by glFenceSync objects and waiting on the next-to-last one

[PATCH xserver 2/4] glamor: Cannot use copies when accessing outside of composite source

2016-06-04 Thread Keith Packard
From: Michel Dänzer Commit b64108fa ("glamor: Check for composite operations which are equivalent to copies") failed to copy conditions from exaComposite which ensure that the composite operation doesn't access outside of the source picture. This fixes rendercheck regressions from the commit abo

[PATCH xserver 1/4] xfree86/modes: Assign xf86_config->cursor in xf86_load_cursor_image v2

2016-06-04 Thread Keith Packard
From: Michel Dänzer Fixes a crash on startup in the radeon driver's drmmode_show_cursor() due to xf86_config->cursor == NULL, because no CRTC was enabled yet, so xf86_crtc_load_cursor_image was never called. (Also use scrn->pScreen instead of xf86ScrnToScreen(scrn)) v2: Set xf86_config->cursor

[PATCH xserver 4/4] xfree86/modes: Remove xf86_reload_cursors v2

2016-06-04 Thread Keith Packard
From: Michel Dänzer No longer needed now that xf86CursorResetCursor is getting called for each CRTC configuration change. v2: Keep xf86_reload_cursors as a deprecated empty inline function until all drivers stop calling it. (Adam Jackson) Reviewed-by: Adam Jackson --- hw/xfree86/drivers/m

[PATCH xserver 3/4] os: Use strtok instead of xstrtokenize in ComputeLocalClient

2016-06-04 Thread Keith Packard
From: Michel Dänzer Fixes leaking the memory pointed to by the members of the array returned by xstrtokenize. Reviewed-by: Adam Jackson --- os/access.c | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/os/access.c b/os/access.c index 58f95a9..8828e08 100644 --- a/os/

[PATCH xserver] ephyr: Process queued X events before blocking

2016-06-04 Thread Keith Packard
If we end up reading all pending X events in the course of other server execution, then our notify FD callback won't get invoked and we won't process them. Fix this by clearing the pending event queue before blocking in the server. Signed-off-by: Keith Packard --- hw/kdrive/ephyr/ephyr.c | 118 +

[PATCH libX11 1/4] Bunch of cleanup Patches

2016-06-04 Thread walter harms
this a a further batch of cleanup patches for libX11. the git repository can be found at https://github.com/xtforever/libX11 This repository contains also the other cleanup patches that were send to the list in the last months. Signed-off-by: harms wha...@bfs.de --- ___

[PATCH libX11 2/4] mark _XDefaultIOError as no_return

2016-06-04 Thread walter harms
mark _XDefaultIOError as no_return. No one comes back from exit() ... Signed-off-by: harms wha...@bfs.de --- src/XlibInt.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/XlibInt.c b/src/XlibInt.c index 7296948..86ae1d9 100644 --- a/src/XlibInt.c +++ b/src/XlibInt.c @@ -

[PATCH libX11 4/4] no need to check XFree arguments

2016-06-04 Thread walter harms
You can save a bit of code. The is no need to check XFree arguments bring free_fontdataOM in line with other free function and check for NULL arg Signed-off-by: harms wha...@bfs.de --- modules/om/generic/omGeneric.c | 32 ++-- 1 file changed, 10 insertions(+), 22 d

[PATCH libX11 3/4] Fixes: warning: variable 'req' set but not,used

2016-06-04 Thread walter harms
Fixes: warning: variable 'req' set but not used [-Wunused-but-set-variable] by marking req _X_UNUSED Solution was discussed on xorg-devel ML Peter Hutter, Alan Coopersmith Re: [PATCH libX11 3/5] fix: warning: pointer targets in passing argument 2 of '_XSend' differ in