summaryrefslogtreecommitdiffstats
path: root/src/glx
Commit message (Collapse)AuthorAgeFilesLines
* glx: Don't destroy DRI2 drawables for legacy glx drawablesKristian Høgsberg2010-09-131-1/+11
| | | | | | | | | | | For GLX 1.3 drawables, we can destroy the DRI2 drawable when the GLX drawable is destroyed. However, for legacy drawables, there os no good way of knowing when the application is done with it, so we just let the DRI2 drawable linger on the server. The server will destroy the DRI2 drawable when it destroys the X drawable or the client exits anyway. https://bugs.freedesktop.org/show_bug.cgi?id=30109
* glx: Optimize out no-op make current callsKristian Høgsberg2010-09-091-0/+4
| | | | | This make a lot more sense now that we might have to recreate the glx drawables for legacy code paths.
* glx: Fix another use-after-free problemKristian Høgsberg2010-09-091-1/+3
|
* glx: Destroy pixmap after destroying glx and dri drawablesKristian Høgsberg2010-09-081-2/+2
| | | | | Now that we suppress BadDrawable from DRI2DestroyDrawable, this doesn't matter, but we would get that error before when destroying pbuffers.
* glx: Ignore DRI2 event for drawables we've destroyedKristian Høgsberg2010-09-081-1/+1
| | | | | | Since we now actually destroy GLX drawables, we get into situations where we get events for drawables that no longer exist. Just ignore the event in that case.
* glx: Fix use after free problemKristian Høgsberg2010-09-081-6/+7
|
* glx: Drop broken drawable garbage collectionKristian Høgsberg2010-09-089-59/+48
| | | | | | Doesn't work for pixmaps, was looking up the GLX XID and was never thread safe. Instead, just destroy the client side structures when the drawable is no long current for a context.
* glx: Set an all NULL vtable for dummyContextKristian Høgsberg2010-09-073-8/+9
| | | | | | This reverts 6a6e6d7b0a84e20f9754af02a575ae34081d310c and initializes dummyContext with an all NULL vtable. The context vtable pointer is supposed to always be non-NULL, but the vtable entries can be NULL.
* glx: Fix compilation with out xf86vidmodeKristian Høgsberg2010-09-071-7/+1
|
* glx: Use GLX_BufferSwapComplete unconditionally, we require glproto 1.4.11Kristian Høgsberg2010-09-071-11/+0
|
* Fix crashes when some GLX API entrypoints are called with no current context.Michel Dänzer2010-09-071-5/+5
| | | | | | | | | I was hitting this with gliv. The GLX spec explicitly mentions that glXWaitX, glXWaitGL and glXUseXFont calls are ignored when there's no current context. Not sure what if anything the GLX_EXT_texture_from_pixmap spec says about this, but I think ignoring the calls makes more sense than crashing there as well. :)
* Make XF86VIDMODE extension optionalJon TURNEY2010-09-072-1/+7
| | | | | | | | | | | | | | Code in glx/glxcmds.c which uses the XF86VIDMODE extension is already guarded. Also use that guard to control inclusion of the xf86vmode.h header, and only enable that guard if the XF86VIDMODE extension is found by pkgconfig. This changes the behaviour on platforms which XF86VIDMODE exists, in that XF86VIDMODE used to be mandatory, but is now optional. Presumably other build systems are already arranging for -DXF86VIDMODE to be supplied to the complier when glxcmds.c is compiled, so are not affected by this change Signed-off-by: Jon TURNEY <[email protected]>
* glx: Drop unused dri2proto.h includeKristian Høgsberg2010-09-071-1/+0
|
* glx: Move dpy and scr fields out of direct rendering conditionalKristian Høgsberg2010-09-071-3/+3
| | | | | | | Nothing direct rendering specific about these fields. Moving them out makes no-direct-rendering compilation work again. Signed-off-by: Kristian Høgsberg <[email protected]>
* Some fixes for GLX_INDIRECT_RENDERING only buildJon TURNEY2010-09-073-5/+9
| | | | | | | This fixes some of the build issues with GLX_INDIRECT_RENDERING but !GLX_DIRECT_RENDERING due to recent changes. Signed-off-by: Jon TURNEY <[email protected]> Signed-off-by: Kristian Høgsberg <[email protected]>
* glx: Only clear the stored context tag when the context has been unboundJon TURNEY2010-09-071-5/+8
| | | | | | | | | | | | | | The calling order of ->bind and ->unbind changed and then ->unbind would clear the currentContextTag of the old context before ->bind could reuse it in the make current request, in the indirect case. Instead, clear the old currentContextTag if and only if we send a request to the server to actually unbind it or reassign it to another context. https://bugs.freedesktop.org/show_bug.cgi?id=29977 Signed-off-by: Jon TURNEY <[email protected]> Signed-off-by: Kristian Høgsberg <[email protected]>
* make: Use C++ compiler to link stdc++ library.Brian Paul2010-08-241-2/+3
| | | | | glxinfo and glxgears run on swrast and softpipe without undefined symbol errors.
* glx: Move deref after NULL checkKristian Høgsberg2010-08-041-1/+2
|
* glx: Fix use after free in drisw error pathKristian Høgsberg2010-08-041-2/+1
|
* glx: We no longer need screen extensions for driswKristian Høgsberg2010-08-041-2/+0
| | | | https://bugs.freedesktop.org/show_bug.cgi?id=29177
* glx: dri2InvalidateBuffers() needs the X drawable XID not the GLX oneKristian Høgsberg2010-08-041-2/+2
| | | | This never ceases to entertain.
* glx: Drop _Xglobal_lock while we create and initialize glx displayKristian Høgsberg2010-08-021-21/+39
|
* glx: Fix copy/paste bug in glXWaitX and glXWaitGLKristian Høgsberg2010-07-291-2/+2
| | | | https://bugs.freedesktop.org/show_bug.cgi?id=29304
* glx: Compare old ctx against dummyCtx, not NULLKristian Høgsberg2010-07-293-2/+4
| | | | https://bugs.freedesktop.org/show_bug.cgi?id=29302
* glx: Fix linked list deletion in __glXCloseDisplay()Kristian Høgsberg2010-07-291-1/+1
| | | | I hate single linked lists.
* glx: Split indirect and applegl implementations into different filesKristian Høgsberg2010-07-287-547/+624
|
* glx: Move bind and unbind to context vtableKristian Høgsberg2010-07-2811-275/+205
|
* glx: Rename __GLXcontext and __GLXdisplayPrivate to struct types.Kristian Høgsberg2010-07-2828-757/+770
|
* glx: Rename __GLXscreenConfigs to struct glx_screenKristian Høgsberg2010-07-2810-101/+99
| | | | Because double underscores in private type names is painful.
* glx: Use _X_EXPORT instead of our own PUBLIC macroKristian Høgsberg2010-07-287-101/+74
| | | | We're an X client library, so we can use Xfuncproto.h.
* glx: Rename glcontextmodes.[ch] to glxconfig.[ch]Kristian Høgsberg2010-07-287-7/+4
|
* glx: Rename __GLcontextModes to struct glx_configKristian Høgsberg2010-07-2817-289/+258
| | | | | With this rename, we use 'config' consitently to refer to GLX configurations instead of the modes/configs/visual mess before.
* glx: Delete unused glcontextmodes.c functionsKristian Høgsberg2010-07-283-101/+0
|
* glx: Stop using glcore.h and glxint.h in glxKristian Høgsberg2010-07-283-93/+90
| | | | We'd like to get rid of these header files.
* Avoid a compiler warning about a potentially unused variable.Carl Worth2010-07-281-1/+1
| | | | | | | | | There is no assignment to the "ret" variable if X_DRI2SwapBuffers is not defined. In this case, the earlier explicit "return 0" is likely to be used, but the compiler can't be sure of that, (nor can I for that matter). We cover this case by explicitly initializing "ret" to 0.
* Avoid compiler warning for unused glx_dpy variable.Carl Worth2010-07-281-1/+1
| | | | | | | | | | My earlier attempt to eliminate this warning (c0ca2bfb2ad8c) was invalid as it removed the variable declaration. Jerome correctly reverted that (600c85efdb0ff) since the variable is used when X_DRI2SwapBuffers is defined. Here, instead of removing the declaration, we move it to inside the correct #ifdef.
* glx: Remove function prototypes no longer necessaryKristian Høgsberg2010-07-261-7/+0
|
* glx: Drop debug fprintf that snug in with the previous commitKristian Høgsberg2010-07-261-2/+0
|
* glx: Enable copy subbuffer patch when GLX_DIRECT_RENDERING is #definedKristian Høgsberg2010-07-261-1/+3
| | | | | | | Depending on __DRI_COPY_SUB_BUFFER doesn't work when we no longer include dri_interface.h. https://bugs.freedesktop.org/show_bug.cgi?id=29264
* glx: Drop duplicate psc field in dri context structKristian Høgsberg2010-07-261-3/+2
| | | | Same problem as fixed for drisw in 4d58b5b482d06ab8d4c4b2db33d0b48b7c82d064.
* glx: Drop duplicate psc field in drisw context structKristian Høgsberg2010-07-241-3/+3
| | | | Causing a crash in drisw MakeCurrent.
* glx: Fix another case of confusing driContext and dri2_context *Eric Anholt2010-07-231-1/+1
|
* glx: Correctly look up the dri2 context pointer for SetTexBuffer.Eric Anholt2010-07-231-1/+1
| | | | | | | gc->driContext points at the second member of the dri2 context. The dri2 context is just a subclass of the GLX context. Fixes piglit tfp testcase.
* glx: Don't destroy context with XID 0Kristian Høgsberg2010-07-234-4/+9
| | | | | We use XID 0 to indicate the context has already been destroyed, but it's currently bound.
* glx: Fix use after free case when destroying screensKristian Høgsberg2010-07-231-1/+0
|
* glx: zero out drawable structs after allocationKristian Høgsberg2010-07-233-0/+3
|
* glx: Refactor and simplify context creationKristian Høgsberg2010-07-236-205/+185
| | | | | This lets us better separate context creation between the different backends.
* glx: Fix indirect screen initializationKristian Høgsberg2010-07-231-2/+7
| | | | https://bugs.freedesktop.org/show_bug.cgi?id=29225
* glx: Move context destroy to context vtableKristian Høgsberg2010-07-237-105/+83
|
* glx: Don't try to swap a front buffer if we don't have one.Eric Anholt2010-07-231-1/+2
| | | | Fixes glean glsl1 since 7b7845a076c933e096ac511b4184141ba194449a