Changeset 70436 in vbox

Timestamp:

Jan 2, 2018 4:56:54 PM (7 years ago)

Author:

vboxsync

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

120041

Message:

Audio/HDA: Only re-arm the timer in hdaWriteRegSDSTS() if there were pending transfer interrupts left -- it could happen that we land in here if a guest writes to SDnSTS unconditionally.

File:

• trunk/src/VBox/Devices/Audio/DevHDA.cpp (modified) (1 diff)

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

@@ -1560 +1560 @@
         pStream->State.tsTransferNext      = tsNow + cTicksToNext;
 
+        /* Only re-arm the timer if there were pending transfer interrupts left
+         *  -- it could happen that we land in here if a guest writes to SDnSTS
+         * unconditionally. */
         if (pStream->State.cTransferPendingInterrupts)
+        {
             pStream->State.cTransferPendingInterrupts--;
 
-        /* Re-arm the timer. */
-        hdaTimerSet(pThis, tsNow + cTicksToNext, false /* fForce */);
+            /* Re-arm the timer. */
+            hdaTimerSet(pThis, tsNow + cTicksToNext, false /* fForce */);
+        }
     }