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

Timestamp:

Jul 18, 2018 3:23:11 PM (7 years ago)

Author:

vboxsync

Message:

Audio/HDA: Made the stream's internal ring buffer size depend on a time frame instead of a fixed (byte) value, as the final size will depend on the stream's Hz rate.

File:

• trunk/src/VBox/Devices/Audio/HDAStream.cpp (modified) (2 diffs)

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

@@ -65 +65 @@
     AssertRCReturn(rc, rc);
 
-    rc = RTCircBufCreate(&pStream->State.pCircBuf, _64K); /** @todo Make this configurable. */
-    AssertRCReturn(rc, rc);
-
     rc = hdaR3StreamPeriodCreate(&pStream->State.Period);
     AssertRCReturn(rc, rc);
@@ -197 +194 @@
         return rc;
     }
+
+    /* (Re-)Allocate the stream's internal DMA buffer, based on the PCM  properties we just got above. */
+    if (pStream->State.pCircBuf)
+    {
+        RTCircBufDestroy(pStream->State.pCircBuf);
+        pStream->State.pCircBuf = NULL;
+    }
+
+    /* By default we allocate an internal buffer of 100ms. */
+    rc = RTCircBufCreate(&pStream->State.pCircBuf, DrvAudioHlpMsToBytes(&pCfg->Props, 100 /* ms */)); /** @todo Make this configurable. */
+    AssertRCReturn(rc, rc);
 
     /* Set the stream's direction. */