Changeset 91143 in vbox for trunk/src/VBox/ValidationKit

Timestamp:

Sep 7, 2021 5:32:09 PM (3 years ago)

Author:

vboxsync

Message:

Audio/Validation Kit: Some more diagnostics to find out why some testbox guests refuse to play any test tones. ​bugref:10008

File:

• trunk/src/VBox/ValidationKit/utils/audio/vkatCommon.cpp (modified) (3 diffs)

trunk/src/VBox/ValidationKit/utils/audio/vkatCommon.cpp

@@ -288 +288 @@
         uint64_t        offStream          = 0;
         uint64_t        nsTimeout          = RT_MS_5MIN_64 * RT_NS_1MS;
+        uint64_t        nsLastMsgCantWrite = 0; /* Timestamp (in ns) when the last message of an unwritable stream was shown. */
 
         while (cbPlayedTotal < cbToPlayTotal)
         {
+            uint64_t const nsNow = RTTimeNanoTS();
+
             /* Pace ourselves a little. */
             if (offStream >= cbPreBuffer)
             {
                 if (!nsDonePreBuffering)
-                    nsDonePreBuffering = RTTimeNanoTS();
+                    nsDonePreBuffering = nsNow;
                 uint64_t const cNsWritten = PDMAudioPropsBytesToNano64(pMix->pProps, offStream - cbPreBuffer);
-                uint64_t const cNsElapsed = RTTimeNanoTS() - nsStarted;
+                uint64_t const cNsElapsed = nsNow - nsStarted;
                 if (cNsWritten > cNsElapsed + RT_NS_10MS)
                     RTThreadSleep((cNsWritten - cNsElapsed - RT_NS_10MS / 2) / RT_NS_1MS);
@@ -333 +336 @@
                 if (RT_FAILURE(rc))
                     break;
+
+                nsLastMsgCantWrite = 0;
             }
             else if (AudioTestMixStreamIsOkay(&pStream->Mix))
+            {
+                if (!nsLastMsgCantWrite || nsNow - nsLastMsgCantWrite > RT_NS_10SEC) /* Don't spam the output too much. */
+                {
+                    RTTestPrintf(g_hTest, RTTESTLVL_ALWAYS, "Waiting for stream to be writable again ...\n");
+                    nsLastMsgCantWrite = nsNow;
+                }
                 RTThreadSleep(RT_MIN(RT_MAX(1, pStream->Cfg.Device.cMsSchedulingHint), 256));
+            }
             else
                 AssertFailedBreakStmt(rc = VERR_AUDIO_STREAM_NOT_READY);
@@ -343 +355 @@
 
             /* Fail-safe in case something screwed up while playing back. */
-            uint64_t const cNsElapsed = RTTimeNanoTS() - nsStarted;
+            uint64_t const cNsElapsed = nsNow - nsStarted;
             if (cNsElapsed > nsTimeout)
             {