Changeset 64687 in vbox for trunk/src/VBox/Devices

Timestamp:

Nov 17, 2016 10:02:07 AM (8 years ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

111961

Message:

Audio/DrvHostPulseAudio: added some temporary logging to debug a testbox issue

File:

• trunk/src/VBox/Devices/Audio/DrvHostPulseAudio.cpp (modified) (8 diffs)

trunk/src/VBox/Devices/Audio/DrvHostPulseAudio.cpp

@@ -176 +176 @@
 {
     if (!pThis)
+    {
+        LogRel(("DEBUG: paSignalWaiter return because of !pThis\n"));
         return;
-
+    }
+
+    LogRel(("DEBUG: paSignalWaiter set fLoopWait=true and calling pa_threaded_mainloop_signal()\n"));
     pThis->fLoopWait = true;
     pa_threaded_mainloop_signal(pThis->pMainLoop, 0);
@@ -260 +264 @@
 
     uint64_t u64StartMs = RTTimeMilliTS();
+    LogRel(("DEBUG: entering paWaitForEx\n"));
     while (pa_operation_get_state(pOP) == PA_OPERATION_RUNNING)
     {
@@ -265 +270 @@
         {
             AssertPtr(pThis->pMainLoop);
+            LogRel(("DEBUG: entering pa_threaded_mainloop_wait()\n"));
             pa_threaded_mainloop_wait(pThis->pMainLoop);
+            LogRel(("DEBUG: leaving pa_threaded_mainloop_wait()\n"));
         }
         pThis->fLoopWait = false;
@@ -275 +282 @@
             break;
         }
+        LogRel(("DEBUG: paWaitForEx next turn (elapsed=%RU64ms)\n", u64ElapsedMs));
     }
 
@@ -985 +993 @@
 {
     if (eol != 0)
+    {
+        LogRel(("DEBUG: paEnumSinkCb return EOL=1\n"));
+        return;
+    }
+    if (!pCtx)
+        LogRel(("DEBUG: paEnumSinkCb return because of !pCtx\n"));
+    AssertPtrReturnVoid(pCtx);
+    if (!pInfo)
+        LogRel(("DEBUG: paEnumSinkCb return because of !pInfo\n"));
+    AssertPtrReturnVoid(pInfo);
+
+    PPULSEAUDIOENUMCBCTX pCbCtx = (PPULSEAUDIOENUMCBCTX)pvUserData;
+    if (!pCbCtx)
+        LogRel(("DEBUG: paEnumSinkCb return because of !pCbCtx\n"));
+    AssertPtrReturnVoid(pCbCtx);
+    if (!pCbCtx->pDrv)
+        LogRel(("DEBUG: paEnumSinkCb return because of !pCbCtx->pDrv\n"));
+    AssertPtrReturnVoid(pCbCtx->pDrv);
+
+    LogRel2(("PulseAudio: Using output sink '%s'\n", pInfo->name));
+
+    /** @todo Store sinks + channel mapping in callback context as soon as we have surround support. */
+    pCbCtx->cDevOut++;
+
+    LogRel(("DEBUG: pa_threaded_mainloop_signal() from paEnumSinkCb\n"));
+    pa_threaded_mainloop_signal(pCbCtx->pDrv->pMainLoop, 0);
+}
+
+
+static void paEnumSourceCb(pa_context *pCtx, const pa_source_info *pInfo, int eol, void *pvUserData)
+{
+    if (eol != 0)
         return;
 
@@ -994 +1034 @@
     AssertPtrReturnVoid(pCbCtx->pDrv);
 
-    LogRel2(("PulseAudio: Using output sink '%s'\n", pInfo->name));
-
-    /** @todo Store sinks + channel mapping in callback context as soon as we have surround support. */
-    pCbCtx->cDevOut++;
-
+    LogRel2(("PulseAudio: Using input source '%s'\n", pInfo->name));
+
+    /** @todo Store sources + channel mapping in callback context as soon as we have surround support. */
+    pCbCtx->cDevIn++;
+
+    LogRel(("DEBUG: pa_threaded_mainloop_signal() from paEnumSourceCb\n"));
     pa_threaded_mainloop_signal(pCbCtx->pDrv->pMainLoop, 0);
 }
 
 
-static void paEnumSourceCb(pa_context *pCtx, const pa_source_info *pInfo, int eol, void *pvUserData)
-{
-    if (eol != 0)
-        return;
-
+static void paEnumServerCb(pa_context *pCtx, const pa_server_info *pInfo, void *pvUserData)
+{
     AssertPtrReturnVoid(pCtx);
     AssertPtrReturnVoid(pInfo);
@@ -1013 +1051 @@
     PPULSEAUDIOENUMCBCTX pCbCtx = (PPULSEAUDIOENUMCBCTX)pvUserData;
     AssertPtrReturnVoid(pCbCtx);
-    AssertPtrReturnVoid(pCbCtx->pDrv);
-
-    LogRel2(("PulseAudio: Using input source '%s'\n", pInfo->name));
-
-    /** @todo Store sources + channel mapping in callback context as soon as we have surround support. */
-    pCbCtx->cDevIn++;
-
-    pa_threaded_mainloop_signal(pCbCtx->pDrv->pMainLoop, 0);
-}
-
-
-static void paEnumServerCb(pa_context *pCtx, const pa_server_info *pInfo, void *pvUserData)
-{
-    AssertPtrReturnVoid(pCtx);
-    AssertPtrReturnVoid(pInfo);
-
-    PPULSEAUDIOENUMCBCTX pCbCtx = (PPULSEAUDIOENUMCBCTX)pvUserData;
-    AssertPtrReturnVoid(pCbCtx);
 
     PDRVHOSTPULSEAUDIO pThis    = pCbCtx->pDrv;
@@ -1047 +1067 @@
     }
 
+    LogRel(("DEBUG: pa_threaded_mainloop_signal() from paEnumServerCb\n"));
     pa_threaded_mainloop_signal(pThis->pMainLoop, 0);
 }