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

Timestamp:

Jul 19, 2018 2:56:24 PM (7 years ago)

Author:

vboxsync

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

123846

Message:

Audio/PDM: Added (tweakable) parameters to the audio stream configuration and give the audio connector(s) a scheduling hint from the device emulation(s), if available.

Location:

trunk/src/VBox/Devices/Audio

Files:

• DevIchAc97.cpp (modified) (2 diffs)
• DevSB16.cpp (modified) (1 diff)
• HDAStream.cpp (modified) (1 diff)
• HDAStream.h (modified) (2 diffs)

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

@@ -312 +312 @@
 #endif
     /** The stream's current configuration. */
-    PDMAUDIOSTREAMCFG     Cfg; //+96
+    PDMAUDIOSTREAMCFG     Cfg; //+100
+    uint32_t              Padding;
 #ifdef VBOX_WITH_AUDIO_AC97_ASYNC_IO
     /** Asynchronous I/O state members. */
@@ -1687 +1688 @@
     AssertCompile(sizeof(pCfg->szName) >= 8);
 
+    /* Set scheduling hint (if available). */
+    if (pThis->cTimerTicks)
+        pCfg->Device.uSchedulingHintMs = 1000 /* ms */ / (TMTimerGetFreq(pThis->pTimerR3) / pThis->cTimerTicks);
+
     switch (pStream->u8SD)
     {

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

@@ -2144 +2144 @@
     if (RT_FAILURE(rc))
         return rc;
+
+    /* Set scheduling hint (if available). */
+    if (pThis->cTimerTicksIO)
+        pThis->Out.Cfg.Device.uSchedulingHintMs = 1000 /* ms */ / (TMTimerGetFreq(pThis->pTimerIO) / pThis->cTimerTicksIO);
 
     PSB16DRIVER pDrv;

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

@@ -194 +194 @@
         return rc;
     }
+
+    /* Set scheduling hint (if available). */
+    if (pThis->u16TimerHz)
+        pCfg->Device.uSchedulingHintMs = 1000 /*ms */ / pThis->u16TimerHz;
 
     /* (Re-)Allocate the stream's internal DMA buffer, based on the PCM  properties we just got above. */

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

@@ -165 +165 @@
      *  Should match SDFMT. */
     PDMAUDIOSTREAMCFG       Cfg;
+    uint32_t                aPadding3;
 #ifdef HDA_USE_DMA_ACCESS_HANDLER
     /** List of DMA handlers. */
@@ -175 +176 @@
     uint16_t                cbDMALeft;
     /** Unused, padding. */
-    uint8_t                 abPadding3[2+4];
+    uint8_t                 abPadding4[2+4];
 } HDASTREAMSTATE;
 AssertCompileSizeAlignment(HDASTREAMSTATE, 8);