Changeset 80923 in vbox for trunk/src/VBox/Devices/VirtIO

Timestamp:

Sep 20, 2019 12:01:11 PM (5 years ago)

Author:

vboxsync

Message:

Dev/Net: @bugref:9556 Lower interrupts on reset

File:

• trunk/src/VBox/Devices/VirtIO/Virtio.cpp (modified) (2 diffs)

trunk/src/VBox/Devices/VirtIO/Virtio.cpp

@@ -309 +309 @@
 }
 
+
+/**
+ * Raise interrupt.
+ *
+ * @param   pState      The device state structure.
+ * @param   rcBusy      Status code to return when the critical section is busy.
+ * @param   u8IntCause  Interrupt cause bit mask to set in PCI ISR port.
+ */
+int vpciRaiseInterrupt(VPCISTATE *pState, int rcBusy, uint8_t u8IntCause)
+{
+    RT_NOREF_PV(rcBusy);
+    // int rc = vpciCsEnter(pState, rcBusy);
+    // if (RT_UNLIKELY(rc != VINF_SUCCESS))
+    //     return rc;
+
+    STAM_COUNTER_INC(&pState->StatIntsRaised);
+    LogFlow(("%s vpciRaiseInterrupt: u8IntCause=%x\n",
+             INSTANCE(pState), u8IntCause));
+
+    pState->uISR |= u8IntCause;
+    PDMDevHlpPCISetIrq(pState->CTX_SUFF(pDevIns), 0, 1);
+    // vpciCsLeave(pState);
+    return VINF_SUCCESS;
+}
+
+/**
+ * Lower interrupt.
+ *
+ * @param   pState      The device state structure.
+ */
+static void vpciLowerInterrupt(VPCISTATE *pState)
+{
+    LogFlow(("%s vpciLowerInterrupt\n", INSTANCE(pState)));
+    PDMDevHlpPCISetIrq(pState->CTX_SUFF(pDevIns), 0, 0);
+}
+
+
 void vpciReset(PVPCISTATE pState)
 {
+    /* No interrupts should survive device reset, see @bugref(9556). */
+    if (pState->uISR)
+        vpciLowerInterrupt(pState);
+
     pState->uGuestFeatures = 0;
     pState->uQueueSelector = 0;
@@ -320 +361 @@
 }
 
-
-/**
- * Raise interrupt.
- *
- * @param   pState      The device state structure.
- * @param   rcBusy      Status code to return when the critical section is busy.
- * @param   u8IntCause  Interrupt cause bit mask to set in PCI ISR port.
- */
-int vpciRaiseInterrupt(VPCISTATE *pState, int rcBusy, uint8_t u8IntCause)
-{
-    RT_NOREF_PV(rcBusy);
-    // int rc = vpciCsEnter(pState, rcBusy);
-    // if (RT_UNLIKELY(rc != VINF_SUCCESS))
-    //     return rc;
-
-    STAM_COUNTER_INC(&pState->StatIntsRaised);
-    LogFlow(("%s vpciRaiseInterrupt: u8IntCause=%x\n",
-             INSTANCE(pState), u8IntCause));
-
-    pState->uISR |= u8IntCause;
-    PDMDevHlpPCISetIrq(pState->CTX_SUFF(pDevIns), 0, 1);
-    // vpciCsLeave(pState);
-    return VINF_SUCCESS;
-}
-
-/**
- * Lower interrupt.
- *
- * @param   pState      The device state structure.
- */
-static void vpciLowerInterrupt(VPCISTATE *pState)
-{
-    LogFlow(("%s vpciLowerInterrupt\n", INSTANCE(pState)));
-    PDMDevHlpPCISetIrq(pState->CTX_SUFF(pDevIns), 0, 0);
-}
 
 DECLINLINE(uint32_t) vpciGetHostFeatures(PVPCISTATE pState,