Changeset 21127 in vbox for trunk/src/VBox/HostServices/SharedOpenGL

Timestamp:

Jul 1, 2009 2:25:14 PM (15 years ago)

Author:

vboxsync

Message:

crOpenGL: fix window/context management for multithreaded apps

Location:

trunk/src/VBox/HostServices/SharedOpenGL/crserverlib

Files:

• server_context.c (modified) (1 diff)
• server_stream.c (modified) (3 diffs)
• server_window.c (modified) (4 diffs)

trunk/src/VBox/HostServices/SharedOpenGL/crserverlib/server_context.c

@@ -140 +140 @@
         for (pos = 0; pos < CR_MAX_CONTEXTS; ++pos)
             if (cr_server.curClient->contextList[pos] == ctx)
+            {
                 cr_server.curClient->contextList[pos] = 0;
+                break;
+            }
+        CRASSERT(pos<CR_MAX_CONTEXTS);
     }
 }

trunk/src/VBox/HostServices/SharedOpenGL/crserverlib/server_stream.c

@@ -116 +116 @@
     int i, j;
     int32_t pos;
-
-    crDebug("Deleting client %p (%d msgs left)", client,
-                    crNetNumMessages(client->conn));
+    CRClient *oldclient = cr_server.curClient;
+
+    crDebug("Deleting client %p (%d msgs left)", client, crNetNumMessages(client->conn));
 
 #if 0
@@ -142 +142 @@
         }
     }
+
+    cr_server.curClient = client;
+
+    /* Destroy any windows created by the client */
+    for (pos = 0; pos<CR_MAX_WINDOWS && client->windowList[pos]; pos++) 
+    {
+        cr_server.dispatch.WindowDestroy(client->windowList[pos]);
+    }
+
+    /* Check if we have context(s) made by this client left, could happen if client side code is lazy */
+    for (pos = 0; pos<CR_MAX_CONTEXTS && client->contextList[pos]; pos++) 
+    {
+        cr_server.dispatch.DestroyContext(client->contextList[pos]);
+    }
+
+    cr_server.curClient = oldclient;
 
     /* remove from the run queue */
@@ -179 +195 @@
 
     crNetFreeConnection(client->conn);
-
-    /* Destroy any windows created by the client */
-    for (pos = 0; pos<CR_MAX_WINDOWS && client->windowList[pos]; pos++) 
-    {
-        cr_server.dispatch.WindowDestroy(client->windowList[pos]);
-    }
-
-    /* Check if we have context(s) made by this client left, could happen if client side code is lazy */
-    for (pos = 0; pos<CR_MAX_CONTEXTS && client->contextList[pos]; pos++) 
-    {
-        cr_server.dispatch.DestroyContext(client->contextList[pos]);
-    }
 
     crFree(client);

trunk/src/VBox/HostServices/SharedOpenGL/crserverlib/server_window.c

@@ -119 +119 @@
 }
 
-
-#define EXTRA_WARN 0
+void crServerCheckCurrentCtxWindowCB(unsigned long key, void *data1, void *data2)
+{
+    CRContext *crCtx = (CRContext *) data1;
+    GLint window = *(GLint*)data2;
+
+    (void) key;
+}
 
 void SERVER_DISPATCH_APIENTRY
@@ -130 +135 @@
     mural = (CRMuralInfo *) crHashtableSearch(cr_server.muralTable, window);
     if (!mural) {
-#if EXTRA_WARN
          crWarning("CRServer: invalid window %d passed to WindowDestroy()", window);
-#endif
          return;
     }
@@ -143 +146 @@
         for (pos = 0; pos < CR_MAX_WINDOWS; ++pos)
             if (cr_server.curClient->windowList[pos] == window)
+            {
                 cr_server.curClient->windowList[pos] = 0;
+                break;
+            }
+        CRASSERT(pos<CR_MAX_WINDOWS);
 
         if (cr_server.curClient->currentMural == mural)
@@ -152 +159 @@
     }
 
+    if (cr_server.currentWindow = window)
+    {
+        cr_server.currentWindow = -1;
+    }
+
+    crHashtableWalk(cr_server.contextTable, crServerCheckCurrentCtxWindowCB, &window);
     crHashtableDelete(cr_server.pWindowCreateInfoTable, window, crServerCreateInfoDeleteCB);
-    /* @todo, can't issue it cause we could have context(s) referencing this one left */
-    /*crHashtableDelete(cr_server.muralTable, window, crFree);*/
+    crHashtableDelete(cr_server.muralTable, window, crFree);
 }