Changeset 59283 in vbox for trunk/src/VBox/Devices/Audio/DevIchHda.cpp

Timestamp:

Jan 7, 2016 7:24:00 PM (9 years ago)

Author:

vboxsync

Message:

HDA: Try to only initialize streams if we have a valid DMA base address.

File:

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

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

@@ -1485 +1485 @@
     pStrmSt->State.cBDLE = 0;
 
-    if (pStrmSt->u16LVI) /* Any BDLEs to fetch? */
-    {
-        AssertMsg(pStrmSt->u64BaseDMA, ("No base DMA address set for stream %RU8\n", u8Strm));
-
+    if (/* Is there a base DMA address set? */
+           pStrmSt->u64BaseDMA
+        /* Any BDLEs to fetch? */
+        && pStrmSt->u16LVI)
+    {
         uint32_t cbBDLE = 0;
 
@@ -1975 +1976 @@
 #ifdef IN_RING3
         /*
-         * Only (re-)initialize the stream when not running.
-         */
-        if (!fRun && !fInRun)
-        {
-            int rc2 = hdaStreamInit(pThis, pStrmSt, u8Strm);
-            AssertRC(rc2);
-        }
-
-        /*
          * We enter here to change DMA states only.
          */
         if (fInRun != fRun)
         {
+            if (fRun)
+            {
+                int rc2 = hdaStreamInit(pThis, pStrmSt, u8Strm);
+                AssertRC(rc2);
+            }
+
             Assert(!fReset && !fInReset);
             LogFunc(("[SD%RU8]: fRun=%RTbool\n", u8Strm, fRun));