diff options
author | Brian Paul <[email protected]> | 2004-02-11 22:06:05 +0000 |
---|---|---|
committer | Brian Paul <[email protected]> | 2004-02-11 22:06:05 +0000 |
commit | 2020278d06f927eed0bcba919f70846df090fc45 (patch) | |
tree | 697316b48af48bd9d59d178408692f4c1d318487 /src | |
parent | 840e82163d0efb6275ccfe766f063a4e1079c26f (diff) |
Do more bookkeeping of vertex buffer object reference counts.
Incr/decr counts when doing glPush/PopClientAttrib(GL_CLIENT_VERTEX_ARRAY_BIT).
Diffstat (limited to 'src')
-rw-r--r-- | src/mesa/main/attrib.c | 34 | ||||
-rw-r--r-- | src/mesa/main/bufferobj.c | 6 | ||||
-rw-r--r-- | src/mesa/main/varray.c | 4 |
3 files changed, 40 insertions, 4 deletions
diff --git a/src/mesa/main/attrib.c b/src/mesa/main/attrib.c index c5339fdbf35..e3423c74170 100644 --- a/src/mesa/main/attrib.c +++ b/src/mesa/main/attrib.c @@ -1,9 +1,8 @@ - /* * Mesa 3-D graphics library - * Version: 5.1 + * Version: 6.1 * - * Copyright (C) 1999-2003 Brian Paul All Rights Reserved. + * Copyright (C) 1999-2004 Brian Paul All Rights Reserved. * * Permission is hereby granted, free of charge, to any person obtaining a * copy of this software and associated documentation files (the "Software"), @@ -1143,6 +1142,31 @@ _mesa_PopAttrib(void) } +/** + * Helper for incrementing/decrementing vertex buffer object reference + * counts when pushing/popping the GL_CLIENT_VERTEX_ARRAY_BIT attribute group. + */ +static void +adjust_buffer_object_ref_counts(struct gl_array_attrib *array, GLint step) +{ + GLuint i; + array->Vertex.BufferObj->RefCount += step; + array->Normal.BufferObj->RefCount += step; + array->Color.BufferObj->RefCount += step; + array->SecondaryColor.BufferObj->RefCount += step; + array->FogCoord.BufferObj->RefCount += step; + array->Index.BufferObj->RefCount += step; + array->EdgeFlag.BufferObj->RefCount += step; + for (i = 0; i < MAX_TEXTURE_COORD_UNITS; i++) + array->TexCoord[i].BufferObj->RefCount += step; + for (i = 0; i < VERT_ATTRIB_MAX; i++) + array->VertexAttrib[i].BufferObj->RefCount += step; + + array->ArrayBufferObj->RefCount += step; + array->ElementArrayBufferObj->RefCount += step; +} + + #define GL_CLIENT_PACK_BIT (1<<20) #define GL_CLIENT_UNPACK_BIT (1<<21) @@ -1190,6 +1214,8 @@ _mesa_PushClientAttrib(GLbitfield mask) newnode->data = attr; newnode->next = head; head = newnode; + /* bump reference counts on buffer objects */ + adjust_buffer_object_ref_counts(&ctx->Array, 1); } ctx->ClientAttribStack[ctx->ClientAttribStackDepth] = head; @@ -1228,8 +1254,10 @@ _mesa_PopClientAttrib(void) ctx->NewState |= _NEW_PACKUNPACK; break; case GL_CLIENT_VERTEX_ARRAY_BIT: + adjust_buffer_object_ref_counts(&ctx->Array, -1); MEMCPY( &ctx->Array, attr->data, sizeof(struct gl_array_attrib) ); + /* decrement reference counts on buffer objects */ ctx->NewState |= _NEW_ARRAY; break; default: diff --git a/src/mesa/main/bufferobj.c b/src/mesa/main/bufferobj.c index 25457183431..97893b721db 100644 --- a/src/mesa/main/bufferobj.c +++ b/src/mesa/main/bufferobj.c @@ -329,7 +329,13 @@ _mesa_init_buffer_objects( GLcontext *ctx ) { GLuint i; + /* Allocate the default buffer object and set refcount so high that + * it never gets deleted. + */ ctx->Array.NullBufferObj = _mesa_new_buffer_object(ctx, 0, 0); + if (ctx->Array.NullBufferObj) + ctx->Array.NullBufferObj->RefCount = 1000; + ctx->Array.ArrayBufferObj = ctx->Array.NullBufferObj; ctx->Array.ElementArrayBufferObj = ctx->Array.NullBufferObj; diff --git a/src/mesa/main/varray.c b/src/mesa/main/varray.c index c9896ae3ca6..c0b7bcfabe9 100644 --- a/src/mesa/main/varray.c +++ b/src/mesa/main/varray.c @@ -58,7 +58,9 @@ update_array(GLcontext *ctx, struct gl_client_array *array, array->Ptr = (const GLubyte *) ptr; #if FEATURE_ARB_vertex_buffer_object array->BufferObj->RefCount--; - /* XXX free buffer object if RefCount == 0 ? */ + if (array->BufferObj->RefCount <= 0) { + (*ctx->Driver.DeleteBuffer)( ctx, array->BufferObj ); + } array->BufferObj = ctx->Array.ArrayBufferObj; array->BufferObj->RefCount++; /* Compute the index of the last array element that's inside the buffer. |