OpenHarmony/third_party_mesa3d
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

mesa: Reads must also be done with lock held.

Browse Source 
Otherwise two threads might think each made the refcount go zero.
This commit is contained in:
José Fonseca
2009-03-06 14:02:42 +00:00
parent f6159ba4d5
commit 8bbb6b352a
1 changed files with 5 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
src/mesa/main/context.c
View File

@@ -1310,6 +1310,8 @@ _mesa_create_context(const GLvisual *visual,
void void
_mesa_free_context_data( GLcontext *ctx ) _mesa_free_context_data( GLcontext *ctx )
{ {
GLint RefCount;
if (!_mesa_get_current_context()){ if (!_mesa_get_current_context()){
/* No current context, but we may need one in order to delete /* No current context, but we may need one in order to delete
* texture objs, etc. So temporarily bind the context now. * texture objs, etc. So temporarily bind the context now.
@@ -1361,10 +1363,10 @@ _mesa_free_context_data( GLcontext *ctx )
/* Shared context state (display lists, textures, etc) */ /* Shared context state (display lists, textures, etc) */
_glthread_LOCK_MUTEX(ctx->Shared->Mutex); _glthread_LOCK_MUTEX(ctx->Shared->Mutex);
ctx->Shared->RefCount--; RefCount = --ctx->Shared->RefCount;
assert(ctx->Shared->RefCount >= 0);
_glthread_UNLOCK_MUTEX(ctx->Shared->Mutex); _glthread_UNLOCK_MUTEX(ctx->Shared->Mutex);
if (ctx->Shared->RefCount == 0) { assert(RefCount >= 0);
if (RefCount == 0) {
/* free shared state */ /* free shared state */
free_shared_state( ctx, ctx->Shared ); free_shared_state( ctx, ctx->Shared );
} }