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

Timestamp:

Apr 5, 2018 3:11:16 PM (7 years ago)

Author:

vboxsync

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

121775

Message:

HDA: Removed redundant and wrong STATESTS interrupt check to fix FreeBSD guest interrupt storm (bugref:9147).

File:

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

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

@@ -449 +449 @@
 
     /* Check controller interrupts (RIRB, STATEST). */
-    if (   (HDA_REG(pThis, RIRBSTS) & HDA_REG(pThis, RIRBCTL) & (HDA_RIRBCTL_ROIC | HDA_RIRBCTL_RINTCTL))
-        /* SDIN State Change Status Flags (SCSF). */
-        || (HDA_REG(pThis, STATESTS) & HDA_STATESTS_SCSF_MASK))
+    if (HDA_REG(pThis, RIRBSTS) & HDA_REG(pThis, RIRBCTL) & (HDA_RIRBCTL_ROIC | HDA_RIRBCTL_RINTCTL))
     {
         intSts |= HDA_INTSTS_CIS; /* Set the Controller Interrupt Status (CIS). */
     }
 
+    /* Check SDIN State Change Status Flags. */
     if (HDA_REG(pThis, STATESTS) & HDA_REG(pThis, WAKEEN))
     {