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

Timestamp:

Dec 14, 2017 11:02:53 AM (7 years ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

119679

Message:

Audio/HDA: Use LVI + 1 foor the stream periods and stream transfer fragments (as LVI is zero-based, 2 entries minimum).

Location:

trunk/src/VBox/Devices/Audio

Files:

• HDAStream.cpp (modified) (1 diff)
• HDAStreamPeriod.cpp (modified) (1 diff)

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

@@ -244 +244 @@
         /** @todo Use a more dynamic fragment size? */
         Assert(pStream->u16LVI <= UINT8_MAX - 1);
-        uint8_t cFragments = pStream->u16LVI;
+        uint8_t cFragments = pStream->u16LVI + 1;
         if (cFragments <= 1)
             cFragments = 2; /* At least two fragments (BDLEs) must be present. */

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

@@ -101 +101 @@
      * 3584  bytes (CBL) / 4 (frame size) = 896 frames / 2 (LVI)  = 448 frames per period
      */
-    unsigned cTotalPeriods = u16LVI;
+    unsigned cTotalPeriods = u16LVI + 1;
 
     if (cTotalPeriods <= 1)