April 2009 - Xquartz-changes - lists.macosforge.org

[338] AppleSGLX/trunk
by source_changes＠macosforge.org 29 Apr '09

29 Apr '09

Revision: 338 http://trac.macosforge.org/projects/xquartz/changeset/338 Author: gstaplin(a)apple.com Date: 2009-04-28 20:40:04 -0700 (Tue, 28 Apr 2009) Log Message: ----------- tests/simple/simple.mk: Add the query_drawable test and make it build. tests/simple/query_drawable.c: Add a test for glXQueryDrawable, other than the existing pbuffer test. tests/tests.mk: build testbuilds/query_drawable glx_pbuffer.c: Fix a deadlock reported by Ken Thomases. The deadlock occurred in the glXQueryDrawable path when the drawable was a window. The paths for the valid pixmaps and buffers were unaffected. Thank you Ken, for your detailed analysis. Modified Paths: -------------- AppleSGLX/trunk/glx_pbuffer.c AppleSGLX/trunk/tests/simple/simple.mk AppleSGLX/trunk/tests/tests.mk Added Paths: ----------- AppleSGLX/trunk/tests/simple/query_drawable.c Modified: AppleSGLX/trunk/glx_pbuffer.c =================================================================== --- AppleSGLX/trunk/glx_pbuffer.c 2009-04-12 02:24:16 UTC (rev 337) +++ AppleSGLX/trunk/glx_pbuffer.c 2009-04-29 03:40:04 UTC (rev 338) @@ -44,21 +44,6 @@ #include "apple_glx_drawable.h" #include "glx_error.h" - -static pthread_mutex_t queryLock = PTHREAD_MUTEX_INITIALIZER; - -/* - * This is protected by the queryLock. - * Unfortunately XSetErrorHandler requires a lock. - */ -static int errorCount = 0; - -static int -errorHandler(Display *dpy, XErrorEvent *error) { - ++errorCount; - return 0; /* don't exit */ -} - /** * Create a new pbuffer. */ @@ -133,7 +118,6 @@ Window root; int x, y; unsigned int width, height, bd, depth; - int (*old_handler)(Display *, XErrorEvent *); if(apple_glx_pixmap_query(drawable, attribute, value)) return; /*done*/ @@ -142,33 +126,27 @@ return; /*done*/ /* - * This is used because XSetErrorHandler changes global state. - * Ideally we should prevent other threads from calling XSetErrorHandler. - * Another way of handling this might be a private display connection. + * The OpenGL spec states that we should report GLXBadDrawable if + * the drawable is invalid, however doing so would require that we + * use XSetErrorHandler(), which is known to not be thread safe. + * If we use a round-trip call to validate the drawable, there could + * be a race, so instead we just opt in favor of letting the + * XGetGeometry request fail with a GetGeometry request X error + * rather than GLXBadDrawable, in what is hoped to be a rare + * case of an invalid drawable. In practice most and possibly all + * X11 apps using GLX shouldn't notice a difference. */ - pthread_mutex_lock(&queryLock); - - old_handler = XSetErrorHandler(errorHandler); - if(XGetGeometry(dpy, drawable, &root, &x, &y, &width, &height, &bd, &depth)) { switch(attribute) { case GLX_WIDTH: *value = width; - return; + break; case GLX_HEIGHT: *value = height; - return; + break; } - /*FALL THROUGH*/ } - - XSetErrorHandler(old_handler); - - pthread_mutex_unlock(&queryLock); - - __glXSendError(dpy, GLXBadDrawable, drawable, X_GLXGetDrawableAttributes, - false); } Added: AppleSGLX/trunk/tests/simple/query_drawable.c =================================================================== --- AppleSGLX/trunk/tests/simple/query_drawable.c (rev 0) +++ AppleSGLX/trunk/tests/simple/query_drawable.c 2009-04-29 03:40:04 UTC (rev 338) @@ -0,0 +1,45 @@ +#include <stdio.h> +#include <stdlib.h> +#include <X11/Xlib.h> +#include <GL/glx.h> + +int main(int argc, char *argv[]) { + Display *dpy; + int eventbase, errorbase; + int major, minor; + Window root; + unsigned int width, height; + + dpy = XOpenDisplay(NULL); + + if(NULL == dpy) { + fprintf(stderr, "error: unable to open display!\n"); + return EXIT_FAILURE; + } + + if(!glXQueryExtension(dpy, &eventbase, &errorbase)) { + fprintf(stderr, "GLX is not available!\n"); + return EXIT_FAILURE; + } + + printf("GLX eventbase %d errorbase %d\n", eventbase, errorbase); + + if(!glXQueryVersion(dpy, &major, &minor)) { + fprintf(stderr, "GLX version query error!\n"); + return EXIT_FAILURE; + } + + printf("GLX version: %d.%d\n", major, minor); + + root = DefaultRootWindow(dpy); + + glXQueryDrawable(dpy, root, GLX_WIDTH, &width); + + printf("query 1: width %d\n", width); + + glXQueryDrawable(dpy, root, GLX_HEIGHT, &height); + + printf("query 2: height %d\n", height); + + return EXIT_SUCCESS; +} Modified: AppleSGLX/trunk/tests/simple/simple.mk =================================================================== --- AppleSGLX/trunk/tests/simple/simple.mk 2009-04-12 02:24:16 UTC (rev 337) +++ AppleSGLX/trunk/tests/simple/simple.mk 2009-04-29 03:40:04 UTC (rev 338) @@ -13,3 +13,5 @@ $(TEST_BUILD_DIR)/glthreads: tests/simple/glthreads.c $(LIBGL) $(CC) -DPTHREADS -pthread tests/simple/glthreads.c $(INCLUDE) -o $(TEST_BUILD_DIR)/glthreads $(LINK_TEST) +$(TEST_BUILD_DIR)/query_drawable: tests/simple/query_drawable.c $(LIBGL) + $(CC) tests/simple/query_drawable.c $(INCLUDE) -o $@ $(LINK_TEST) Modified: AppleSGLX/trunk/tests/tests.mk =================================================================== --- AppleSGLX/trunk/tests/tests.mk 2009-04-12 02:24:16 UTC (rev 337) +++ AppleSGLX/trunk/tests/tests.mk 2009-04-29 03:40:04 UTC (rev 338) @@ -35,5 +35,6 @@ $(TEST_BUILD_DIR)/triangle_glx_surface \ $(TEST_BUILD_DIR)/triangle_glx_surface-2 \ $(TEST_BUILD_DIR)/triangle_glx_withdraw_remap \ - $(TEST_BUILD_DIR)/triangle_glx_destroy_relation + $(TEST_BUILD_DIR)/triangle_glx_destroy_relation \ + $(TEST_BUILD_DIR)/query_drawable

1 0

[337] trunk/pkg/README.2.3.3.txt
by source_changes＠macosforge.org 12 Apr '09

12 Apr '09

Revision: 337 http://trac.macosforge.org/projects/xquartz/changeset/337 Author: jeremyhu(a)freedesktop.org Date: 2009-04-11 19:24:16 -0700 (Sat, 11 Apr 2009) Log Message: ----------- 2.3.3 updates. Modified Paths: -------------- trunk/pkg/README.2.3.3.txt Modified: trunk/pkg/README.2.3.3.txt =================================================================== --- trunk/pkg/README.2.3.3.txt 2009-04-09 11:08:21 UTC (rev 336) +++ trunk/pkg/README.2.3.3.txt 2009-04-12 02:24:16 UTC (rev 337) @@ -22,8 +22,9 @@ * quartz-wm * Properly re-enable the close widget when modal windows are destroyed * Fixed a bug when the window was told to unmaximize when it was already not maximized. - * xinput 1.4.0 - * xterm 242 + * xinput 1.4.1 + * xrandr 1.3.0 + * xterm 243b * Xephyr, Xnest, Xfake, Xvfb 1.6.0 * proto: * applewmproto 1.2.0 @@ -60,12 +61,15 @@ * pixman 0.14.0 * xtrans 1.2.3 * server: - * Xquartz fixes from xorg-server-1.4.2-apple41 + * Xquartz fixes from xorg-server-1.4.2-apple42 * DRI Updates for new libGL * Fix CapsLock * Fixed mouse tracking for games like Quake2, Halflife, etc in wine. * Fixed the annoying spaces bug resulting from accessing menus in a window that you moved to another space * When configured for fullscreen mode, initial apps don't start rootless. + * Update window levels when changing rootless status + * Fixed window levels to work with applications that create a desktop (nautilus, etc) + * Properly send tablet proximity events == Changes in 2.3.2.1 == * All changes in 2.3.2 plus:

1 0

[336] trunk/pkg/README.2.3.3.txt
by source_changes＠macosforge.org 09 Apr '09

09 Apr '09

Revision: 336 http://trac.macosforge.org/projects/xquartz/changeset/336 Author: jeremyhu(a)freedesktop.org Date: 2009-04-09 04:08:21 -0700 (Thu, 09 Apr 2009) Log Message: ----------- More 2.3.3 notes Modified Paths: -------------- trunk/pkg/README.2.3.3.txt Modified: trunk/pkg/README.2.3.3.txt =================================================================== --- trunk/pkg/README.2.3.3.txt 2009-04-05 10:27:18 UTC (rev 335) +++ trunk/pkg/README.2.3.3.txt 2009-04-09 11:08:21 UTC (rev 336) @@ -60,11 +60,12 @@ * pixman 0.14.0 * xtrans 1.2.3 * server: - * Xquartz fixes from xorg-server-1.4.2-apple40 + * Xquartz fixes from xorg-server-1.4.2-apple41 * DRI Updates for new libGL * Fix CapsLock * Fixed mouse tracking for games like Quake2, Halflife, etc in wine. * Fixed the annoying spaces bug resulting from accessing menus in a window that you moved to another space + * When configured for fullscreen mode, initial apps don't start rootless. == Changes in 2.3.2.1 == * All changes in 2.3.2 plus:

1 0

[335] trunk/pkg/README.2.3.3.txt
by source_changes＠macosforge.org 05 Apr '09

05 Apr '09

Revision: 335 http://trac.macosforge.org/projects/xquartz/changeset/335 Author: jeremyhu(a)freedesktop.org Date: 2009-04-05 03:27:18 -0700 (Sun, 05 Apr 2009) Log Message: ----------- Updated 2.3.3 release notes Modified Paths: -------------- trunk/pkg/README.2.3.3.txt Modified: trunk/pkg/README.2.3.3.txt =================================================================== --- trunk/pkg/README.2.3.3.txt 2009-04-02 03:06:42 UTC (rev 334) +++ trunk/pkg/README.2.3.3.txt 2009-04-05 10:27:18 UTC (rev 335) @@ -34,7 +34,7 @@ * xcb-proto 1.4 * xf86rushproto 1.1.2 * lib: - * AppleSGLX 56 + * AppleSGLX 57 * Support the same version of OpenGL as OpenGL.framework (2.1) * GLSL and other features now supported * Many extensions from OpenGL.framework are now available @@ -60,10 +60,11 @@ * pixman 0.14.0 * xtrans 1.2.3 * server: - * Xquartz fixes from xorg-server-1.4.2-apple38 + * Xquartz fixes from xorg-server-1.4.2-apple40 * DRI Updates for new libGL * Fix CapsLock * Fixed mouse tracking for games like Quake2, Halflife, etc in wine. + * Fixed the annoying spaces bug resulting from accessing menus in a window that you moved to another space == Changes in 2.3.2.1 == * All changes in 2.3.2 plus:

1 0

[334] AppleSGLX/tags/AppleSGLX-57/
by source_changes＠macosforge.org 02 Apr '09

02 Apr '09

Revision: 334 http://trac.macosforge.org/projects/xquartz/changeset/334 Author: jeremyhu(a)freedesktop.org Date: 2009-04-01 20:06:42 -0700 (Wed, 01 Apr 2009) Log Message: ----------- Tagging AppleSGLX-57 Added Paths: ----------- AppleSGLX/tags/AppleSGLX-57/

1 0

[333] AppleSGLX/trunk/appledristr.h
by source_changes＠macosforge.org 01 Apr '09

01 Apr '09

Revision: 333 http://trac.macosforge.org/projects/xquartz/changeset/333 Author: jeremyhu(a)freedesktop.org Date: 2009-04-01 14:31:18 -0700 (Wed, 01 Apr 2009) Log Message: ----------- Pad xAppleDRINotifyEvent to 32bytes to match sizeof(xEvent) Modified Paths: -------------- AppleSGLX/trunk/appledristr.h Modified: AppleSGLX/trunk/appledristr.h =================================================================== --- AppleSGLX/trunk/appledristr.h 2009-04-01 21:29:25 UTC (rev 332) +++ AppleSGLX/trunk/appledristr.h 2009-04-01 21:31:18 UTC (rev 333) @@ -157,8 +157,11 @@ CARD32 pad1 B32; CARD32 arg B32; CARD32 pad3 B32; + CARD32 pad4 B32; + CARD32 pad5 B32; + CARD32 pad6 B32; } xAppleDRINotifyEvent; -#define sz_xAppleDRINotifyEvent 20 +#define sz_xAppleDRINotifyEvent 32 typedef struct {

1 0

[332] AppleSGLX/trunk/GL_aliases
by source_changes＠macosforge.org 01 Apr '09

01 Apr '09

Revision: 332 http://trac.macosforge.org/projects/xquartz/changeset/332 Author: jeremyhu(a)freedesktop.org Date: 2009-04-01 14:29:25 -0700 (Wed, 01 Apr 2009) Log Message: ----------- Fix typo for GenTexturesEXT Modified Paths: -------------- AppleSGLX/trunk/GL_aliases Modified: AppleSGLX/trunk/GL_aliases =================================================================== --- AppleSGLX/trunk/GL_aliases 2009-03-31 20:54:55 UTC (rev 331) +++ AppleSGLX/trunk/GL_aliases 2009-04-01 21:29:25 UTC (rev 332) @@ -2,7 +2,7 @@ alias AreTexturesResidentEXT AreTexturesResident alias BindTextureEXT BindTexture alias DeleteTexturesEXT DeleteTextures -alias GenTexturesExt GenTextures +alias GenTexturesEXT GenTextures alias IsTextureEXT IsTexture alias PrioritizeTexturesEXT PrioritizeTextures alias TexImage3DEXT TexImage3D

1 0