Changeset 91141 in vbox for trunk/src/VBox/ValidationKit/utils

Timestamp:

Sep 7, 2021 2:37:42 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) (4 diffs)

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

@@ -287 +287 @@
 
         uint64_t        offStream          = 0;
-        uint64_t        uTimeoutNs         = (pParms->msDuration * 4) * RT_NS_1MS; /* Four times the time playback should roughly take */
-
-        while (cbToPlayTotal)
+        uint64_t        nsTimeout          = RT_MS_5MIN_64 * RT_NS_1MS;
+
+        while (cbPlayedTotal < cbToPlayTotal)
         {
             /* Pace ourselves a little. */
@@ -306 +306 @@
             if (cbCanWrite)
             {
-                uint32_t const cbToGenerate = RT_MIN(RT_MIN(cbToPlayTotal, sizeof(abBuf)), cbCanWrite);
+                uint32_t const cbToGenerate = RT_MIN(RT_MIN(cbToPlayTotal - cbPlayedTotal, sizeof(abBuf)), cbCanWrite);
                 uint32_t       cbToPlay;
                 rc = AudioTestToneGenerate(&TstTone, abBuf, cbToGenerate, &cbToPlay);
@@ -321 +321 @@
                         if (RT_SUCCESS(rc))
                         {
+                            AssertBreakStmt(cbPlayed <= cbToPlay, rc = VERR_TOO_MUCH_DATA);
+
                             offStream += cbPlayed;
 
@@ -341 +343 @@
 
             /* Fail-safe in case something screwed up while playing back. */
-            if (RTTimeNanoTS() - nsStarted > uTimeoutNs)
+            uint64_t const cNsElapsed = RTTimeNanoTS() - nsStarted;
+            if (cNsElapsed > nsTimeout)
             {
-                RTTestFailed(g_hTest, "Playback took too long (%RU32ms exceeded), aborting\n", uTimeoutNs / RT_NS_1MS);
+                RTTestFailed(g_hTest, "Playback took too long (runng %RU64 vs. timeout %RU64), aborting\n", cNsElapsed, nsTimeout);
                 rc = VERR_TIMEOUT;
             }