Changeset 91824 in vbox for trunk

Timestamp:

Oct 18, 2021 10:42:34 AM (3 years ago)

Author:

vboxsync

Message:

Audio/Validation Kit: Added fail-safe for audio stacks and/or implementations which mess up draining. ​bugref:10008

File:

• trunk/src/VBox/ValidationKit/utils/audio/vkatDriverStack.cpp (modified) (2 diffs)

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

@@ -1010 +1010 @@
         if (RT_SUCCESS(rc) && fSync)
         {
+            RTMSINTERVAL const msTimeout = RT_MS_5MIN; /* 5 minutes should be really enough for draining our stuff. */
+            uint64_t const     tsStart   = RTTimeMilliTS();
+
             /*
              * This is a synchronous drain, so wait for the driver to change state to inactive.
@@ -1032 +1035 @@
                     break;
                 }
+
+                /* Fail-safe for audio stacks and/or implementations which mess up draining. */
+                if (RTTimeMilliTS() - tsStart > msTimeout)
+                {
+                    RTTestFailed(g_hTest, "Draining stream took too long (timeout is %RU32ms), giving up", msTimeout);
+                    break;
+                }
             }
             if (enmHostState != PDMHOSTAUDIOSTREAMSTATE_OKAY)