Changeset 80686 in vbox for trunk/src/VBox/Devices/Audio

Timestamp:

Sep 10, 2019 7:54:05 AM (5 years ago)

Author:

vboxsync

Message:

Audio/DrvHostDSound: Fixed "double buffering" size of a stream's internal ring buffer, adjusted logging.

File:

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

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

@@ -746 +746 @@
         pStreamDS->cbBufSize = bc.dwBufferBytes;
 
-        rc = RTCircBufCreate(&pStreamDS->pCircBuf, pStreamDS->cbBufSize) * 2; /* Use "double buffering" */
+        rc = RTCircBufCreate(&pStreamDS->pCircBuf, pStreamDS->cbBufSize * 2 /* Use "double buffering" */);
         AssertRC(rc);
 
@@ -1227 +1227 @@
         && pStreamDS->In.cOverruns < 32) /** @todo Make this configurable. */
     {
-        DSLOG(("DSound: Warning: Ring buffer full, skipping to record data (overflow #%RU32)\n", pStreamDS->In.cOverruns));
-        DSLOG(("DSound: DSound capture buffer currently uses %RU32/%RU32 bytes\n", cbUsed, pStreamDS->cbBufSize));
+        DSLOG(("DSound: Warning: Internal buffer full (size is %zu bytes), skipping to record data (overflow #%RU32)\n",
+               RTCircBufSize(pCircBuf), pStreamDS->In.cOverruns));
+        DSLOG(("DSound: Warning: DSound capture buffer currently uses %RU32/%RU32 bytes\n", cbUsed, pStreamDS->cbBufSize));
         pStreamDS->In.cOverruns++;
     }
@@ -1491 +1492 @@
         pStreamDS->cbBufSize     = bc.dwBufferBytes;
 
-        rc = RTCircBufCreate(&pStreamDS->pCircBuf, pStreamDS->cbBufSize) * 2; /* Use "double buffering". */
+        rc = RTCircBufCreate(&pStreamDS->pCircBuf, pStreamDS->cbBufSize * 2 /* Use "double buffering" */);
         AssertRC(rc);