summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--src/mesa/main/bufferobj.c607
-rw-r--r--src/mesa/main/context.c7
-rw-r--r--src/mesa/main/dd.h33
-rw-r--r--src/mesa/main/mtypes.h16
4 files changed, 583 insertions, 80 deletions
diff --git a/src/mesa/main/bufferobj.c b/src/mesa/main/bufferobj.c
index a9e66a199ae..4e67bc225c4 100644
--- a/src/mesa/main/bufferobj.c
+++ b/src/mesa/main/bufferobj.c
@@ -25,39 +25,392 @@
/**
* \file bufferobj.c
- * Functions for the GL_ARB_vertex_buffer_object extension.
+ * \brief Functions for the GL_ARB_vertex_buffer_object extension.
* \author Brian Paul
*/
#include "glheader.h"
+#include "hash.h"
#include "imports.h"
#include "context.h"
#include "bufferobj.h"
+struct gl_buffer_object {
+ GLint RefCount;
+ GLuint Name;
+ GLenum Target;
+
+ GLenum usage;
+ GLenum access;
+ GLvoid * pointer;
+ GLuint size;
+
+ void * data;
+};
+
+void _mesa_initialize_buffer_object( struct gl_buffer_object *obj,
+ GLuint name, GLenum target );
+
+
+/**
+ * Get the buffer object bound to the specified target in a GL context.
+ *
+ * \param ctx GL context
+ * \param target Buffer object target to be retrieved. Currently this must
+ * be either \c GL_ARRAY_BUFFER or \c GL_ELEMENT_ARRAY_BUFFER.
+ * \param str Name of caller for logging errors.
+ * \return A pointer to the buffer object bound to \c target in the
+ * specified context or \c NULL if \c target is invalid or no
+ * buffer object is bound.
+ */
+
+static __inline struct gl_buffer_object *
+_mesa_buffer_object_get_target( GLcontext *ctx, GLenum target,
+ const char * str )
+{
+ struct gl_buffer_object * bufObj = NULL;
+
+ switch (target) {
+ case GL_ARRAY_BUFFER_ARB:
+ bufObj = ctx->ArrayBuffer;
+ break;
+ case GL_ELEMENT_ARRAY_BUFFER_ARB:
+ bufObj = ctx->ElementArrayBuffer;
+ break;
+ default:
+ _mesa_error(ctx, GL_INVALID_ENUM, "gl%s(target)", str);
+ break;
+ }
+
+ return bufObj;
+}
+
+
+/**
+ * Tests the subdata range parameters and sets the GL error code for
+ * \c glBufferSubDataARB and \c glGetBufferSubDataARB.
+ *
+ * \param ctx GL context.
+ * \param target Buffer object target on which to operate.
+ * \param offset Offset of the first byte of the subdata range.
+ * \param size Size, in bytes, of the subdata range.
+ * \param str Name of caller for logging errors.
+ * \return A pointer to the buffer object bound to \c target in the
+ * specified context or \c NULL if any of the parameter or state
+ * conditions for \c glBufferSubDataARB or \c glGetBufferSubDataARB
+ * are invalid.
+ *
+ * \sa glBufferSubDataARB, glGetBufferSubDataARB
+ */
+
+static struct gl_buffer_object *
+_mesa_buffer_object_subdata_range_good( GLcontext * ctx, GLenum target,
+ GLintptrARB offset, GLsizeiptrARB size,
+ const char * str )
+{
+ struct gl_buffer_object *bufObj;
+
+ if (size < 0) {
+ _mesa_error(ctx, GL_INVALID_VALUE, "gl%s(size < 0)", str);
+ return NULL;
+ }
+
+ if (offset < 0) {
+ _mesa_error(ctx, GL_INVALID_VALUE, "gl%s(offset < 0)", str);
+ return NULL;
+ }
+
+ bufObj = _mesa_buffer_object_get_target( ctx, target, str );
+ if ( bufObj == NULL ) {
+ return NULL;
+ }
+
+ if ( (offset + size) > bufObj->size ) {
+ _mesa_error(ctx, GL_INVALID_VALUE,
+ "gl%s(size + offset > buffer size)", str);
+ return NULL;
+ }
+
+ if ( bufObj->pointer != NULL ) {
+ _mesa_error(ctx, GL_INVALID_OPERATION, "gl%s", str);
+ return NULL;
+ }
+
+ return bufObj;
+}
+
+
+/**
+ * Allocate and initialize a new buffer object.
+ *
+ * This function is intended to be called via
+ * \c dd_function_table::NewBufferObject.
+ */
+
+struct gl_buffer_object *
+_mesa_new_buffer_object( GLcontext *ctx, GLuint name, GLenum target )
+{
+ struct gl_buffer_object *obj;
+ obj = CALLOC_STRUCT(gl_buffer_object);
+ _mesa_initialize_buffer_object(obj, name, target);
+ return obj;
+}
+
+
+/**
+ * Initialize a buffer object to default values.
+ */
+
+void
+_mesa_initialize_buffer_object( struct gl_buffer_object *obj,
+ GLuint name, GLenum target )
+{
+ obj->RefCount = 1;
+ obj->Name = name;
+}
+
+
+/**
+ * Add the given buffer object to the buffer object pool.
+ */
+
+void
+_mesa_save_buffer_object( GLcontext *ctx, struct gl_buffer_object *obj )
+{
+ if (obj->Name > 0) {
+ /* insert into hash table */
+ _mesa_HashInsert(ctx->Shared->BufferObjects, obj->Name, obj);
+ }
+}
+
+
+/**
+ * Remove the given buffer object from the buffer object pool.
+ * Do not deallocate the buffer object though.
+ */
+
+void
+_mesa_remove_buffer_object( GLcontext *ctx, struct gl_buffer_object *bufObj )
+{
+ if (bufObj->Name > 0) {
+ /* remove from hash table */
+ _mesa_HashRemove(ctx->Shared->BufferObjects, bufObj->Name);
+ }
+}
+
+
+/**
+ * Allocate space for and store data in a buffer object. Any data that was
+ * previously stored in the buffer object is lost. If \c data is \c NULL,
+ * memory will be allocated, but no copy will occur.
+ *
+ * This function is intended to be called via
+ * \c dd_function_table::BufferData. This function need not set GL error
+ * codes. The input parameters will have been tested before calling.
+ *
+ * \param ctx GL context.
+ * \param target Buffer object target on which to operate.
+ * \param size Size, in bytes, of the new data store.
+ * \param data Pointer to the data to store in the buffer object. This
+ * pointer may be \c NULL.
+ * \param usage Hints about how the data will be used.
+ * \param bufObj Object to be used.
+ *
+ * \sa glBufferDataARB, dd_function_table::BufferData.
+ */
+
+void
+_mesa_buffer_data( GLcontext *ctx, GLenum target, GLsizeiptrARB size,
+ const GLvoid * data, GLenum usage,
+ struct gl_buffer_object * bufObj )
+{
+ void * new_data;
+
+ (void) target;
+
+ new_data = _mesa_realloc( bufObj->data, bufObj->size, size );
+ if ( new_data != NULL ) {
+ bufObj->data = new_data;
+ bufObj->size = size;
+ bufObj->usage = usage;
+
+ if ( data != NULL ) {
+ memcpy( bufObj->data, data, size );
+ }
+ }
+}
+
+
+/**
+ * Replace data in a subrange of buffer object. If the data range
+ * specified by \c size + \c offset extends beyond the end of the buffer or
+ * if \c data is \c NULL, no copy is performed.
+ *
+ * This function is intended to be called by
+ * \c dd_function_table::BufferSubData. This function need not set GL error
+ * codes. The input parameters will have been tested before calling.
+ *
+ * \param ctx GL context.
+ * \param target Buffer object target on which to operate.
+ * \param offset Offset of the first byte to be modified.
+ * \param size Size, in bytes, of the data range.
+ * \param data Pointer to the data to store in the buffer object.
+ * \param bufObj Object to be used.
+ *
+ * \sa glBufferSubDataARB, dd_function_table::BufferSubData.
+ */
+
+void
+_mesa_buffer_subdata( GLcontext *ctx, GLenum target, GLintptrARB offset,
+ GLsizeiptrARB size, const GLvoid * data,
+ struct gl_buffer_object * bufObj )
+{
+ if ( (bufObj->data != NULL)
+ && ((size + offset) <= bufObj->size) ) {
+ memcpy( (GLubyte *) bufObj->data + offset, data, size );
+ }
+}
+
+
+/**
+ * Retrieve data from a subrange of buffer object. If the data range
+ * specified by \c size + \c offset extends beyond the end of the buffer or
+ * if \c data is \c NULL, no copy is performed.
+ *
+ * This function is intended to be called by
+ * \c dd_function_table::BufferGetSubData. This function need not set GL error
+ * codes. The input parameters will have been tested before calling.
+ *
+ * \param ctx GL context.
+ * \param target Buffer object target on which to operate.
+ * \param offset Offset of the first byte to be modified.
+ * \param size Size, in bytes, of the data range.
+ * \param data Pointer to the data to store in the buffer object.
+ * \param bufObj Object to be used.
+ *
+ * \sa glBufferGetSubDataARB, dd_function_table::GetBufferSubData.
+ */
+
+void
+_mesa_buffer_get_subdata( GLcontext *ctx, GLenum target, GLintptrARB offset,
+ GLsizeiptrARB size, GLvoid * data,
+ struct gl_buffer_object * bufObj )
+{
+ if ( (bufObj->data != NULL)
+ && ((size + offset) <= bufObj->size) ) {
+ memcpy( data, (GLubyte *) bufObj->data + offset, size );
+ }
+}
+
+
+/**
+ * Maps the private data buffer into the processor's address space.
+ *
+ * This function is intended to be called by \c dd_function_table::MapBuffer.
+ * This function need not set GL error codes. The input parameters will have
+ * been tested before calling.
+ *
+ * \param ctx GL context.
+ * \param target Buffer object target on which to operate.
+ * \param access Information about how the buffer will be accessed.
+ * \param bufObj Object to be used.
+ * \return A pointer to the object's internal data store that can be accessed
+ * by the processor
+ *
+ * \sa glMapBufferARB, dd_function_table::MapBuffer
+ */
+
+void *
+_mesa_buffer_map( GLcontext *ctx, GLenum target, GLenum access,
+ struct gl_buffer_object * bufObj )
+{
+ return bufObj->data;
+}
+
void
_mesa_BindBufferARB(GLenum target, GLuint buffer)
{
GET_CURRENT_CONTEXT(ctx);
+ struct gl_buffer_object *oldBufObj;
+ struct gl_buffer_object *newBufObj = 0;
ASSERT_OUTSIDE_BEGIN_END(ctx);
- if (target == GL_ARRAY_BUFFER_ARB) {
-
- }
- else if (target == GL_ELEMENT_ARRAY_BUFFER_ARB) {
+ oldBufObj = _mesa_buffer_object_get_target( ctx, target, "BindBufferARB" );
+ if ( (oldBufObj != NULL) && (oldBufObj->Name == buffer) )
+ return; /* rebinding the same buffer object- no change */
+ /*
+ * Get pointer to new buffer object (newBufObj)
+ */
+ if ( buffer == 0 ) {
+ newBufObj = NULL;
}
else {
- _mesa_error(ctx, GL_INVALID_ENUM, "glBindBufferARB(target)");
- return;
+ /* non-default buffer object */
+ const struct _mesa_HashTable *hash = ctx->Shared->BufferObjects;
+ newBufObj = (struct gl_buffer_object *) _mesa_HashLookup(hash, buffer);
+ if ( newBufObj != NULL ) {
+ /* error checking */
+ if (newBufObj->Target != 0 && newBufObj->Target != target) {
+ /* the named buffer object's target doesn't match the target */
+ _mesa_error( ctx, GL_INVALID_OPERATION,
+ "glBindBufferARB(wrong target)" );
+ return;
+ }
+ }
+ else {
+ /* if this is a new buffer object id, allocate a buffer object now */
+ newBufObj = (*ctx->Driver.NewBufferObject)(ctx, buffer, target);
+ if (!newBufObj) {
+ _mesa_error(ctx, GL_OUT_OF_MEMORY, "glBindBufferARB");
+ return;
+ }
+ _mesa_save_buffer_object(ctx, newBufObj);
+ }
+ newBufObj->Target = target;
+ newBufObj->RefCount++;
+ }
+
+ switch (target) {
+ case GL_ARRAY_BUFFER_ARB:
+ ctx->ArrayBuffer = newBufObj;
+ break;
+ case GL_ELEMENT_ARRAY_BUFFER_ARB:
+ ctx->ElementArrayBuffer = newBufObj;
+ break;
+ }
+
+ /* Pass BindBuffer call to device driver */
+ if ( (ctx->Driver.BindBuffer != NULL) && (newBufObj != NULL) )
+ (*ctx->Driver.BindBuffer)( ctx, target, newBufObj );
+
+ if ( oldBufObj != NULL ) {
+ oldBufObj->RefCount--;
+ assert(oldBufObj->RefCount >= 0);
+ if (oldBufObj->RefCount == 0) {
+ assert(oldBufObj->Name != 0);
+ _mesa_remove_buffer_object(ctx, oldBufObj);
+ ASSERT(ctx->Driver.DeleteBuffer);
+ (*ctx->Driver.DeleteBuffer)( ctx, oldBufObj );
+ }
}
}
+
+/**
+ * Delete a set of buffer objects.
+ *
+ * \param n Number of buffer objects to delete.
+ * \param buffer Array of \c n buffer object IDs.
+ */
+
void
-_mesa_DeleteBuffersARB(GLsizei n, const GLuint *buffer)
+_mesa_DeleteBuffersARB(GLsizei n, const GLuint *ids)
{
GET_CURRENT_CONTEXT(ctx);
+ unsigned i;
ASSERT_OUTSIDE_BEGIN_END(ctx);
if (n < 0) {
@@ -65,31 +418,125 @@ _mesa_DeleteBuffersARB(GLsizei n, const GLuint *buffer)
return;
}
+ _glthread_LOCK_MUTEX(ctx->Shared->Mutex);
+
+ for (i = 0; i < n; i++) {
+ if (ids[i] != 0) {
+ struct gl_buffer_object *bufObj = (struct gl_buffer_object *)
+ _mesa_HashLookup(ctx->Shared->BufferObjects, ids[i]);
+ if (bufObj) {
+ if ( (bufObj->Target == GL_ARRAY_BUFFER_ARB)
+ || (bufObj->Target == GL_ELEMENT_ARRAY_BUFFER_ARB) ) {
+ _mesa_BindBufferARB( bufObj->Target, 0 );
+ }
+ else if (bufObj->Target == 0) {
+ /* The buffer object is not bound.
+ */
+ }
+ else {
+ _mesa_problem(ctx, "bad target in glDeleteBufferARB");
+ return;
+ }
+ bufObj->RefCount--;
+ if (bufObj->RefCount <= 0) {
+ ASSERT(bufObj->Name != 0);
+ _mesa_remove_buffer_object(ctx, bufObj);
+ ASSERT(ctx->Driver.DeleteBuffer);
+ (*ctx->Driver.DeleteBuffer)(ctx, bufObj);
+ }
+ }
+ }
+ }
+
+ _glthread_UNLOCK_MUTEX(ctx->Shared->Mutex);
}
+
+/**
+ * Generate a set of unique buffer object IDs and store them in \c buffer.
+ *
+ * \param n Number of IDs to generate.
+ * \param buffer Array of \c n locations to store the IDs.
+ */
+
void
_mesa_GenBuffersARB(GLsizei n, GLuint *buffer)
{
GET_CURRENT_CONTEXT(ctx);
+ GLuint first;
+ GLint i;
ASSERT_OUTSIDE_BEGIN_END(ctx);
if (n < 0) {
- _mesa_error(ctx, GL_INVALID_VALUE, "glGenBuffersARB(n)");
+ _mesa_error(ctx, GL_INVALID_VALUE, "glGenBuffersARB");
return;
}
+
+ if ( buffer == NULL ) {
+ return;
+ }
+
+ /*
+ * This must be atomic (generation and allocation of buffer object IDs)
+ */
+ _glthread_LOCK_MUTEX(ctx->Shared->Mutex);
+
+ first = _mesa_HashFindFreeKeyBlock(ctx->Shared->BufferObjects, n);
+
+ /* Return the buffer names */
+ for (i=0;i<n;i++) {
+ buffer[i] = first + i;
+ }
+
+ /* Allocate new, empty buffer objects */
+ for (i = 0; i < n; i++) {
+ struct gl_buffer_object *bufObj;
+ GLuint name = first + i;
+ GLenum target = 0;
+ bufObj = (*ctx->Driver.NewBufferObject)( ctx, name, target );
+ if ( bufObj != NULL ) {
+ _mesa_error(ctx, GL_OUT_OF_MEMORY, "glGenBuffersARB");
+ return;
+ }
+ _mesa_save_buffer_object(ctx, bufObj);
+ }
+
+ _glthread_UNLOCK_MUTEX(ctx->Shared->Mutex);
}
+
+/**
+ * Determine if ID is the name of a buffer object.
+ *
+ * \param id ID of the potential buffer object.
+ * \return \c GL_TRUE if \c id is the name of a buffer object,
+ * \c GL_FALSE otherwise.
+ */
+
GLboolean
-_mesa_IsBufferARB(GLuint buffer)
+_mesa_IsBufferARB(GLuint id)
{
- return GL_FALSE;
+ struct gl_buffer_object * bufObj;
+ GET_CURRENT_CONTEXT(ctx);
+ ASSERT_OUTSIDE_BEGIN_END_WITH_RETVAL(ctx, GL_FALSE);
+
+ if (id == 0)
+ return GL_FALSE;
+
+ _glthread_LOCK_MUTEX(ctx->Shared->Mutex);
+ bufObj = (struct gl_buffer_object *) _mesa_HashLookup(ctx->Shared->BufferObjects, id);
+ _glthread_UNLOCK_MUTEX(ctx->Shared->Mutex);
+
+ return (bufObj != NULL);
}
+
void
_mesa_BufferDataARB(GLenum target, GLsizeiptrARB size,
const GLvoid * data, GLenum usage)
{
GET_CURRENT_CONTEXT(ctx);
+ struct gl_buffer_object *bufObj;
ASSERT_OUTSIDE_BEGIN_END(ctx);
if (size < 0) {
@@ -114,16 +561,15 @@ _mesa_BufferDataARB(GLenum target, GLsizeiptrARB size,
return;
}
- if (target == GL_ARRAY_BUFFER_ARB) {
-
- }
- else if (target == GL_ELEMENT_ARRAY_BUFFER_ARB) {
-
- }
- else {
- _mesa_error(ctx, GL_INVALID_ENUM, "glBufferDataARB(target)");
+ bufObj = _mesa_buffer_object_get_target( ctx, target, "BufferDataARB" );
+ if ( bufObj == NULL ) {
return;
}
+
+ ASSERT(ctx->Driver.BufferData);
+
+ /* Give the buffer object to the driver! <data> may be null! */
+ (*ctx->Driver.BufferData)( ctx, target, size, data, usage, bufObj );
}
void
@@ -131,22 +577,14 @@ _mesa_BufferSubDataARB(GLenum target, GLintptrARB offset,
GLsizeiptrARB size, const GLvoid * data)
{
GET_CURRENT_CONTEXT(ctx);
+ struct gl_buffer_object *bufObj;
ASSERT_OUTSIDE_BEGIN_END(ctx);
- if (size < 0) {
- _mesa_error(ctx, GL_INVALID_VALUE, "glBufferSubDataARB(size < 0)");
- return;
- }
-
- if (target == GL_ARRAY_BUFFER_ARB) {
-
- }
- else if (target == GL_ELEMENT_ARRAY_BUFFER_ARB) {
-
- }
- else {
- _mesa_error(ctx, GL_INVALID_ENUM, "glBufferSubDataARB(target)");
- return;
+ bufObj = _mesa_buffer_object_subdata_range_good( ctx, target, offset, size,
+ "BufferSubDataARB" );
+ if ( bufObj != NULL ) {
+ ASSERT(ctx->Driver.BufferSubData);
+ (*ctx->Driver.BufferSubData)( ctx, target, offset, size, data, bufObj );
}
}
@@ -155,22 +593,14 @@ _mesa_GetBufferSubDataARB(GLenum target, GLintptrARB offset,
GLsizeiptrARB size, void * data)
{
GET_CURRENT_CONTEXT(ctx);
+ struct gl_buffer_object *bufObj;
ASSERT_OUTSIDE_BEGIN_END(ctx);
- if (size < 0) {
- _mesa_error(ctx, GL_INVALID_VALUE, "glGetBufferSubDataARB(size < 0)");
- return;
- }
-
- if (target == GL_ARRAY_BUFFER_ARB) {
-
- }
- else if (target == GL_ELEMENT_ARRAY_BUFFER_ARB) {
-
- }
- else {
- _mesa_error(ctx, GL_INVALID_ENUM, "glGetBufferSubDataARB(target)");
- return;
+ bufObj = _mesa_buffer_object_subdata_range_good( ctx, target, offset, size,
+ "GetBufferSubDataARB" );
+ if ( bufObj != NULL ) {
+ ASSERT(ctx->Driver.GetBufferSubData);
+ (*ctx->Driver.GetBufferSubData)( ctx, target, offset, size, data, bufObj );
}
}
@@ -178,6 +608,7 @@ void *
_mesa_MapBufferARB(GLenum target, GLenum access)
{
GET_CURRENT_CONTEXT(ctx);
+ struct gl_buffer_object * bufObj;
ASSERT_OUTSIDE_BEGIN_END_WITH_RETVAL(ctx, NULL);
switch (access) {
@@ -191,73 +622,92 @@ _mesa_MapBufferARB(GLenum target, GLenum access)
return NULL;
}
- if (target == GL_ARRAY_BUFFER_ARB) {
-
+ bufObj = _mesa_buffer_object_get_target( ctx, target,
+ "MapBufferARB" );
+ if ( bufObj == NULL ) {
+ return NULL;
}
- else if (target == GL_ELEMENT_ARRAY_BUFFER_ARB) {
- }
- else {
- _mesa_error(ctx, GL_INVALID_ENUM, "glMapBufferARB(target)");
+ if ( bufObj->pointer != NULL ) {
+ _mesa_error(ctx, GL_INVALID_OPERATION, "glMapBufferARB");
return NULL;
}
- return NULL;
+ ASSERT(ctx->Driver.MapBuffer);
+ bufObj->pointer = (*ctx->Driver.MapBuffer)( ctx, target, access, bufObj );
+ if ( bufObj->pointer == NULL ) {
+ _mesa_error(ctx, GL_OUT_OF_MEMORY, "glMapBufferARB(access)");
+ }
+
+ return bufObj->pointer;
}
GLboolean
_mesa_UnmapBufferARB(GLenum target)
{
GET_CURRENT_CONTEXT(ctx);
+ struct gl_buffer_object *bufObj;
+ GLboolean status = GL_TRUE;
ASSERT_OUTSIDE_BEGIN_END_WITH_RETVAL(ctx, GL_FALSE);
- if (target == GL_ARRAY_BUFFER_ARB) {
+ bufObj = _mesa_buffer_object_get_target( ctx, target,
+ "UnmapBufferARB" );
+ if ( bufObj == NULL ) {
+ return GL_FALSE;
}
- else if (target == GL_ELEMENT_ARRAY_BUFFER_ARB) {
- }
- else {
- _mesa_error(ctx, GL_INVALID_ENUM, "glUnmapBufferARB(target)");
+ if ( bufObj->pointer == NULL ) {
+ _mesa_error(ctx, GL_INVALID_OPERATION, "glUnmapBufferARB");
return GL_FALSE;
}
- return GL_FALSE;
+
+ if ( ctx->Driver.UnmapBuffer != NULL ) {
+ status = (*ctx->Driver.UnmapBuffer)( ctx, target, bufObj );
+ }
+
+ bufObj->pointer = NULL;
+
+ return status;
}
void
_mesa_GetBufferParameterivARB(GLenum target, GLenum pname, GLint *params)
{
GET_CURRENT_CONTEXT(ctx);
+ struct gl_buffer_object *bufObj;
ASSERT_OUTSIDE_BEGIN_END(ctx);
+ bufObj = _mesa_buffer_object_get_target( ctx, target,
+ "GetBufferParameterivARB" );
+ if ( bufObj == NULL ) {
+ return;
+ }
+
switch (pname) {
case GL_BUFFER_SIZE_ARB:
+ *params = bufObj->size;
+ break;
case GL_BUFFER_USAGE_ARB:
+ *params = bufObj->usage;
+ break;
case GL_BUFFER_ACCESS_ARB:
+ *params = bufObj->access;
+ break;
case GL_BUFFER_MAPPED_ARB:
- /* ok */
+ *params = (bufObj->pointer != NULL);
break;
default:
_mesa_error(ctx, GL_INVALID_ENUM, "glGetBufferParameterivARB(pname)");
return;
}
-
- if (target == GL_ARRAY_BUFFER_ARB) {
-
- }
- else if (target == GL_ELEMENT_ARRAY_BUFFER_ARB) {
-
- }
- else {
- _mesa_error(ctx, GL_INVALID_ENUM, "glGetBufferParameterARB(target)");
- return;
- }
}
void
_mesa_GetBufferPointervARB(GLenum target, GLenum pname, GLvoid **params)
{
GET_CURRENT_CONTEXT(ctx);
+ struct gl_buffer_object * bufObj;
ASSERT_OUTSIDE_BEGIN_END(ctx);
if (pname != GL_BUFFER_MAP_POINTER_ARB) {
@@ -265,14 +715,11 @@ _mesa_GetBufferPointervARB(GLenum target, GLenum pname, GLvoid **params)
return;
}
- if (target == GL_ARRAY_BUFFER_ARB) {
-
- }
- else if (target == GL_ELEMENT_ARRAY_BUFFER_ARB) {
-
- }
- else {
- _mesa_error(ctx, GL_INVALID_ENUM, "glGetBufferPointervARB(target)");
+ bufObj = _mesa_buffer_object_get_target( ctx, target,
+ "GetBufferPointervARB" );
+ if ( bufObj == NULL ) {
return;
}
+
+ *params = bufObj->pointer;
}
diff --git a/src/mesa/main/context.c b/src/mesa/main/context.c
index 9151c74899f..dea778c39a9 100644
--- a/src/mesa/main/context.c
+++ b/src/mesa/main/context.c
@@ -840,6 +840,8 @@ alloc_shared_state( GLcontext *ctx )
goto cleanup;
#endif
+ ss->BufferObjects = _mesa_NewHashTable();
+
ss->Default1D = (*ctx->Driver.NewTextureObject)(ctx, 0, GL_TEXTURE_1D);
if (!ss->Default1D)
goto cleanup;
@@ -895,6 +897,9 @@ alloc_shared_state( GLcontext *ctx )
if (ss->DefaultFragmentProgram)
_mesa_delete_program(ctx, ss->DefaultFragmentProgram);
#endif
+ if (ss->BufferObjects)
+ _mesa_DeleteHashTable(ss->BufferObjects);
+
if (ss->Default1D)
(*ctx->Driver.DeleteTexture)(ctx, ss->Default1D);
if (ss->Default2D)
@@ -972,6 +977,8 @@ free_shared_state( GLcontext *ctx, struct gl_shared_state *ss )
_mesa_DeleteHashTable(ss->Programs);
#endif
+ _mesa_DeleteHashTable(ss->BufferObjects);
+
_glthread_DESTROY_MUTEX(ss->Mutex);
FREE(ss);
diff --git a/src/mesa/main/dd.h b/src/mesa/main/dd.h
index 1829bbc6524..09a53e557cc 100644
--- a/src/mesa/main/dd.h
+++ b/src/mesa/main/dd.h
@@ -829,6 +829,39 @@ struct dd_function_table {
*/
void (*UnlockArraysEXT)( GLcontext *ctx );
/*@}*/
+
+
+ /**
+ * \name Vertex buffer object functions
+ */
+ /*@{*/
+
+ void (*BindBuffer)( GLcontext *ctx, GLenum target,
+ struct gl_buffer_object *obj );
+
+ struct gl_buffer_object * (*NewBufferObject)( GLcontext *ctx, int buffer,
+ GLenum target );
+
+ void (*DeleteBuffer)( GLcontext *ctx, struct gl_buffer_object *obj );
+
+ void (*BufferData)( GLcontext *ctx, GLenum target, GLsizeiptrARB size,
+ const GLvoid *data, GLenum usage,
+ struct gl_buffer_object *obj );
+
+ void (*BufferSubData)( GLcontext *ctx, GLenum target, GLintptrARB offset,
+ GLsizeiptrARB size, const GLvoid *data,
+ struct gl_buffer_object *obj );
+
+ void (*GetBufferSubData)( GLcontext *ctx, GLenum target,
+ GLintptrARB offset, GLsizeiptrARB size,
+ GLvoid *data, struct gl_buffer_object *obj );
+
+ void * (*MapBuffer)( GLcontext *ctx, GLenum target, GLenum access,
+ struct gl_buffer_object *obj );
+
+ GLboolean (*UnmapBuffer)( GLcontext *ctx, GLenum target,
+ struct gl_buffer_object *obj );
+ /*@}*/
};
diff --git a/src/mesa/main/mtypes.h b/src/mesa/main/mtypes.h
index a51a463602c..3abf6ea4acc 100644
--- a/src/mesa/main/mtypes.h
+++ b/src/mesa/main/mtypes.h
@@ -1598,6 +1598,9 @@ struct gl_shared_state
#endif
/*@}*/
+ /** GL_ARB_vertex_buffer_objects */
+ struct _mesa_HashTable *BufferObjects;
+
void *DriverData; /**< Device driver shared state */
};
@@ -1985,6 +1988,7 @@ typedef union node Node;
/* This has to be included here. */
+struct gl_buffer_object;
#include "dd.h"
@@ -2221,6 +2225,18 @@ struct __GLcontextRec {
struct gl_tnl_module TnlModule;
/**
+ * \name GL_ARB_vertex_buffer_object state
+ *
+ * These pointers track the buffer objects last bound via
+ * \c glBindBufferObjectARB. If it the last bound object ID was 0 for a
+ * given target, the pointer will be \c NULL.
+ */
+ /*@{*/
+ struct gl_buffer_object * ArrayBuffer;
+ struct gl_buffer_object * ElementArrayBuffer;
+ /*@}*/
+
+ /**
* \name Hooks for module contexts.
*
* These will eventually live in the driver or elsewhere.