Changeset 57344 in vbox for trunk/src/VBox/Additions/x11/VBoxClient/seamless.cpp

Timestamp:

Aug 14, 2015 9:22:27 AM (9 years ago)

Author:

vboxsync

Message:

Additions/x11/VBoxClient: clean up seamless code slightly and handle pausing and resuming internally to service.

File:

• trunk/src/VBox/Additions/x11/VBoxClient/seamless.cpp (modified) (9 diffs)

trunk/src/VBox/Additions/x11/VBoxClient/seamless.cpp

@@ -41 +41 @@
     mX11MonitorThreadStopping = false;
     mMode = VMMDev_Seamless_Disabled;
-    mfPaused = false;
+    mfPaused = true;
 }
 
@@ -87 +87 @@
         if (RT_FAILURE(rc))
             break;
+        rc = startX11MonitorThread();
+        if (RT_FAILURE(rc))
+            break;
     } while(0);
     if (RT_FAILURE(rc))
-        LogRel(("VBoxClient (seamless): failed to start.  Stage: \"%s\"  Error: %Rrc\n",
+        VBClFatalError(("VBoxClient (seamless): failed to start.  Stage: \"%s\"  Error: %Rrc\n",
                 pcszStage, rc));
     return rc;
@@ -170 +173 @@
     if (RT_SUCCESS(rc) || rc == VERR_TRY_AGAIN)
     {
-        if (mMode == VMMDev_Seamless_Visible_Region && !mfPaused)
-            /* This does it's own logging on failure. */
-            rc = startX11MonitorThread();
+        if (mMode == VMMDev_Seamless_Visible_Region)
+            mfPaused = false;
         else
-            /* This does it's own logging on failure. */
-            rc = stopX11MonitorThread();
+            mfPaused = true;
+        mX11Monitor.interruptEventWait();
     }
     else
@@ -183 +185 @@
     LogRelFlowFunc(("returning %Rrc\n", rc));
     return rc;
-}
-
-int SeamlessMain::cancelEvent(void)
-{
-    return VbglR3InterruptEventWaits();
 }
 
@@ -199 +196 @@
 
     LogRelFlowFunc(("\n"));
-    rc = pHost->mX11Monitor.start();
-    if (RT_SUCCESS(rc))
-    {
-        while (!pHost->mX11MonitorThreadStopping)
-            pHost->mX11Monitor.nextConfigurationEvent();
-        pHost->mX11Monitor.stop();
+    while (!pHost->mX11MonitorThreadStopping)
+    {
+        if (!pHost->mfPaused)
+        {
+            rc = pHost->mX11Monitor.start();
+            if (RT_FAILURE(rc))
+                VBClFatalError(("Failed to change the X11 seamless service state, mfPaused=%RTbool, rc=%Rrc\n",
+                                pHost->mfPaused, rc));
+        }
+        pHost->mX11Monitor.nextConfigurationEvent();
+        if (pHost->mfPaused || pHost->mX11MonitorThreadStopping)
+            pHost->mX11Monitor.stop();
     }
     LogRelFlowFunc(("returning %Rrc\n", rc));
@@ -248 +251 @@
 }
 
-/** Pause the service loop. */
-int SeamlessMain::pause()
-{
-    int rc;
-    const char *pcszStage;
-
-    LogRelFlowFunc(("\n"));
-    mfPaused = true;
-    do {
-        pcszStage = "Reporting end of support for seamless capability";
-        rc = VbglR3SeamlessSetCap(false);
-        if (RT_FAILURE(rc))
-            break;
-        pcszStage = "Interrupting the event loop";
-        rc = cancelEvent();
-        if (RT_FAILURE(rc))
-            break;
-    } while (0);
-    if (RT_FAILURE(rc))
-        LogRelFunc(("Failure.  Stage: \"%s\"  Error: %Rrc (VBoxClient)\n",
-                    pcszStage, rc));
-    return rc;
-}
-
-/** Resume after pausing. */
-int SeamlessMain::resume()
-{
-    int rc;
-    const char *pcszStage;
-
-    LogRelFlowFunc(("\n"));
-    mfPaused = false;
-    do {
-        pcszStage = "Reporting support for seamless capability";
-        rc = VbglR3SeamlessSetCap(true);
-        if (RT_FAILURE(rc))
-            break;
-        pcszStage = "Interrupting the event loop";
-        rc = cancelEvent();
-        if (RT_FAILURE(rc))
-            break;
-    } while (0);
-    if (RT_FAILURE(rc))
-        LogRelFunc(("Failure.  Stage: \"%s\"  Error: %Rrc (VBoxClient)\n",
-                    pcszStage, rc));
-    return rc;
-}
-
 /** @todo Expand this? */
 int SeamlessMain::selfTest()
@@ -382 +337 @@
     if (!pSelf->mIsInitialised)
         return VERR_INTERNAL_ERROR;
-    rc = VBClStartVTMonitor();
-    if (RT_FAILURE(rc))
-        VBClFatalError(("Failed to start the VT monitor thread: %Rrc\n", rc));
     /* This only exits on error. */
     rc = pSelf->mSeamless.run();
@@ -391 +343 @@
 }
 
-static int pause(struct VBCLSERVICE **ppInterface)
-{
-    struct SEAMLESSSERVICE *pSelf = getClassFromInterface(ppInterface);
-
-    return pSelf->mSeamless.pause();
-}
-
-static int resume(struct VBCLSERVICE **ppInterface)
-{
-    struct SEAMLESSSERVICE *pSelf = getClassFromInterface(ppInterface);
-
-    return pSelf->mSeamless.resume();
-}
-
 static void cleanup(struct VBCLSERVICE **ppInterface)
 {
@@ -417 +355 @@
     init,
     run,
-    pause,
-    resume,
+    VBClServiceDefaultHandler, /* pause */
+    VBClServiceDefaultHandler, /* resume */
     cleanup
 };