Changeset 16074 in vbox for trunk/src/VBox/Additions/WINNT/Graphics/crOpenGL/array/arrayspu.c

Timestamp:

Jan 20, 2009 10:00:38 AM (16 years ago)

Author:

vboxsync

Message:

crOpenGL: fix guest app crashes with vertexarrays for multithreaded apps

File:

• trunk/src/VBox/Additions/WINNT/Graphics/crOpenGL/array/arrayspu.c (modified) (7 diffs)

trunk/src/VBox/Additions/WINNT/Graphics/crOpenGL/array/arrayspu.c

@@ -13 +13 @@
 ArraySPU array_spu;
 
+#ifdef CHROMIUM_THREADSAFE
+CRmutex _ArrayMutex;
+#endif
+
 static void ARRAYSPU_APIENTRY arrayspu_ArrayElement( GLint index )
 {
-    const CRClientState *c = &(array_spu.ctx->client);
+    const CRClientState *c = &(crStateGetCurrent()->client);
     const CRVertexArrays *array = &(c->array);
-    const GLboolean vpEnabled = array_spu.ctx->program.vpEnabled;
+    const GLboolean vpEnabled = crStateGetCurrent()->program.vpEnabled;
     unsigned char *p;
     unsigned int unit, attr;
@@ -103 +107 @@
 
     /* Now do conventional arrays, unless overriden by generic arrays above */
-    for (unit = 0 ; unit < array_spu.ctx->limits.maxTextureUnits ; unit++)
+    for (unit = 0 ; unit < crStateGetCurrent()->limits.maxTextureUnits ; unit++)
     {
         if (array->t[unit].enabled && !(vpEnabled && array->a[VERT_ATTRIB_TEX0+unit].enabled))
@@ -450 +454 @@
     GLubyte *p = (GLubyte *)indices;
 #ifdef CR_ARB_vertex_buffer_object
-    CRBufferObject *elementsBuffer = array_spu.ctx->bufferobject.elementsBuffer;
+    CRBufferObject *elementsBuffer = crStateGetCurrent()->bufferobject.elementsBuffer;
 #endif
 
@@ -622 +626 @@
 {
      if (cap == GL_VERTEX_PROGRAM_NV) {
-            array_spu.ctx->program.vpEnabled = GL_TRUE;
+            crStateGetCurrent()->program.vpEnabled = GL_TRUE;
      }
      array_spu.child.Enable(cap);
@@ -631 +635 @@
 {
      if (cap == GL_VERTEX_PROGRAM_NV) {
-            array_spu.ctx->program.vpEnabled = GL_FALSE;
+            crStateGetCurrent()->program.vpEnabled = GL_FALSE;
      }
      array_spu.child.Disable(cap);
@@ -754 +758 @@
 }
 
+static GLint ARRAYSPU_APIENTRY
+arrayspu_CreateContext( const char *dpyName, GLint visual, GLint shareCtx )
+{
+    GLint ctx, slot;
+
+#ifdef CHROMIUM_THREADSAFE
+    crLockMutex(&_ArrayMutex);
+#endif
+
+    ctx = array_spu.child.CreateContext(dpyName, visual, shareCtx);
+
+    /* find an empty context slot */
+    for (slot = 0; slot < array_spu.numContexts; slot++) {
+        if (!array_spu.context[slot].clientState) {
+            /* found empty slot */
+            break;
+        }
+    }
+    if (slot == array_spu.numContexts) {
+        array_spu.numContexts++;
+    }
+
+    array_spu.context[slot].clientState = crStateCreateContext(NULL, visual, NULL);
+    array_spu.context[slot].clientCtx = ctx;
+#ifdef CR_ARB_vertex_buffer_object
+    array_spu.context[slot].clientState->bufferobject.retainBufferData = GL_TRUE;
+#endif
+
+#ifdef CHROMIUM_THREADSAFE
+    crUnlockMutex(&_ArrayMutex);
+#endif
+
+    return ctx;
+}
+
+static void ARRAYSPU_APIENTRY
+arrayspu_MakeCurrent( GLint window, GLint nativeWindow, GLint ctx )
+{
+#ifdef CHROMIUM_THREADSAFE
+    crLockMutex(&_ArrayMutex);
+#endif
+    array_spu.child.MakeCurrent(window, nativeWindow, ctx);
+
+    if (ctx) {
+        int slot;
+
+        for (slot=0; slot<array_spu.numContexts; ++slot)
+            if (array_spu.context[slot].clientCtx == ctx) break;
+        CRASSERT(slot < array_spu.numContexts);
+
+        crStateMakeCurrent(array_spu.context[slot].clientState);
+    }
+    else 
+    {
+        crStateMakeCurrent(NULL);
+    }
+
+#ifdef CHROMIUM_THREADSAFE
+    crUnlockMutex(&_ArrayMutex);
+#endif
+}
+
+static void ARRAYSPU_APIENTRY
+arrayspu_DestroyContext( GLint ctx )
+{
+#ifdef CHROMIUM_THREADSAFE
+    crLockMutex(&_ArrayMutex);
+#endif
+    array_spu.child.DestroyContext(ctx);
+
+    if (ctx) {
+        int slot;
+
+        for (slot=0; slot<array_spu.numContexts; ++slot)
+            if (array_spu.context[slot].clientCtx == ctx) break;
+        CRASSERT(slot < array_spu.numContexts);
+
+        crStateDestroyContext(array_spu.context[slot].clientState);
+
+        array_spu.context[slot].clientState = NULL;
+        array_spu.context[slot].clientCtx = 0;
+    }
+
+#ifdef CHROMIUM_THREADSAFE
+    crUnlockMutex(&_ArrayMutex);
+#endif
+}
 
 SPUNamedFunctionTable _cr_array_table[] = {
@@ -793 +884 @@
     { "GetBufferPointervARB", (SPUGenericFunction) arrayspu_GetBufferPointervARB},
     { "InterleavedArrays", (SPUGenericFunction) arrayspu_InterleavedArrays},
+    { "CreateContext", (SPUGenericFunction) arrayspu_CreateContext},
+    { "MakeCurrent", (SPUGenericFunction) arrayspu_MakeCurrent},
+    { "DestroyContext", (SPUGenericFunction) arrayspu_DestroyContext},
     { NULL, NULL }
 };