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

Timestamp:

Feb 9, 2018 10:35:53 AM (7 years ago)

Author:

vboxsync

Message:

Audio/DrvHostDSound.cpp: Added underrun / overrun counters.

File:

• trunk/src/VBox/Devices/Audio/DrvHostDSound.cpp (modified) (6 diffs)

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

@@ -136 +136 @@
             /** Current read offset (in bytes) within the DSB. */
             DWORD                       offReadPos;
+            /** Number of buffer overruns happened. Used for logging. */
+            uint8_t                     cOverruns;
         } In;
         struct
@@ -154 +156 @@
             /** Timestamp (in ms) of When the last playback has happened. */
             uint64_t                    tsLastPlayMs;
+            /** Number of buffer underruns happened. Used for logging. */
+            uint8_t                     cUnderruns;
         } Out;
     };
@@ -922 +926 @@
     Assert(pStreamDS->fEnabled == false);
 
+    pStreamDS->fEnabled           = true;
+
     pStreamDS->Out.fFirstPlayback = true;
-    pStreamDS->fEnabled           = true;
+    pStreamDS->Out.cUnderruns     = 0;
 
     DSLOG(("DSound: Playback started\n"));
@@ -1317 +1323 @@
                 {
                     pStreamDS->fEnabled = true;
+
+                    pStreamDS->In.cOverruns = 0;
                 }
                 else
@@ -1968 +1976 @@
 
                     uint32_t cbFree = (uint32_t)RTCircBufFree(pCircBuf);
-                    if (!cbFree)
+                    if (   !cbFree
+                        || pStreamDS->In.cOverruns < 32) /** @todo Make this configurable. */
+                    {
                         DSLOG(("DSound: Warning: Capture buffer full, skipping to record data (%RU32 bytes)\n", cbUsed));
+                        pStreamDS->In.cOverruns++;
+                    }
 
                     DWORD cbToCapture = RT_MIN(cbUsed, cbFree);
@@ -2066 +2078 @@
 
                     DWORD cbUsed   = (uint32_t)RTCircBufUsed(pCircBuf);
-                    if (!cbUsed)
+                    if (   !cbUsed
+                        || pStreamDS->Out.cUnderruns < 32) /** @todo Make this configurable. */
+                    {
                         DSLOG(("DSound: Warning: No more playback data available within time (%RU32 bytes free)\n", cbFree));
+                        pStreamDS->Out.cUnderruns++;
+                    }
 
                     DWORD cbToPlay = RT_MIN(cbFree, cbUsed);