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

Timestamp:

Nov 5, 2020 6:55:26 AM (5 years ago)

Author:

vboxsync

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

141246

Message:

AMD IOMMU: bugref:9654 DevIoApic: Add stat for counting interrupts and MSIs discarded due to the IOMMU.

File:

• trunk/src/VBox/Devices/PC/DevIoApic.cpp (modified) (6 diffs)

trunk/src/VBox/Devices/PC/DevIoApic.cpp

@@ -252 +252 @@
     /** Number of suppressed level-triggered interrupts (by remote IRR). */
     STAMCOUNTER             StatSuppressedLevelIntr;
+    /** Number of IOMMU discarded interrupts (signaled by RTE). */
+    STAMCOUNTER             StatIommuDiscardedIntr;
+    /** Number of IOMMU denied or failed MSIs. */
+    STAMCOUNTER             StatIommuDiscardedMsi;
     /** Number of returns to ring-3 due to EOI broadcast lock contention. */
     STAMCOUNTER             StatEoiContention;
@@ -535 +539 @@
         else
         {
+            STAM_COUNTER_INC(&pThis->StatIommuDiscardedIntr);
             Log(("IOAPIC: Interrupt (%#x) discarded (rc=%Rrc)\n", ApicIntr.u8Vector, rcRemap));
             return;
@@ -894 +899 @@
 {
     PIOAPICCC pThisCC = PDMDEVINS_2_DATA_CC(pDevIns, PIOAPICCC);
+    PIOAPIC   pThis   = PDMDEVINS_2_DATA(pDevIns, PIOAPIC);
     LogFlow(("IOAPIC: ioapicSendMsi: uBusDevFn=%#x Addr=%#RX64 Data=%#RX32\n", uBusDevFn, pMsi->Addr.u64, pMsi->Data.u32));
 
@@ -911 +917 @@
     else
     {
+        STAM_COUNTER_INC(&pThis->StatIommuDiscardedMsi);
         if (rcRemap == VERR_IOMMU_INTR_REMAP_DENIED)
             Log3(("IOAPIC: MSI (Addr=%#RX64 Data=%#RX32) remapping denied. rc=%Rrc", pMsi->Addr.u64, pMsi->Data.u32, rcRemap));
@@ -925 +932 @@
      * Deliver to the local APIC via the system/3-wire-APIC bus.
      */
-    PIOAPIC pThis = PDMDEVINS_2_DATA(pDevIns, PIOAPIC);
     STAM_REL_COUNTER_INC(&pThis->aStatVectors[ApicIntr.u8Vector]);
 
@@ -1477 +1483 @@
     PDMDevHlpSTAMRegister(pDevIns, &pThis->StatRedundantLevelIntr,  STAMTYPE_COUNTER, "RedundantLevelIntr",  STAMUNIT_OCCURENCES, "Number of redundant level-triggered interrupts (no IRR change).");
     PDMDevHlpSTAMRegister(pDevIns, &pThis->StatSuppressedLevelIntr, STAMTYPE_COUNTER, "SuppressedLevelIntr", STAMUNIT_OCCURENCES, "Number of suppressed level-triggered interrupts by remote IRR.");
+    PDMDevHlpSTAMRegister(pDevIns, &pThis->StatIommuDiscardedIntr,  STAMTYPE_COUNTER, "IommuDiscardedIntr",  STAMUNIT_OCCURENCES, "Number of interrupts discarded due to IOMMU.");
+    PDMDevHlpSTAMRegister(pDevIns, &pThis->StatIommuDiscardedMsi,   STAMTYPE_COUNTER, "IommuDiscardedMsi",   STAMUNIT_OCCURENCES, "Number of MSIs discarded due to IOMMU.");
 
     PDMDevHlpSTAMRegister(pDevIns, &pThis->StatEoiContention,    STAMTYPE_COUNTER, "CritSect/ContentionSetEoi", STAMUNIT_OCCURENCES, "Number of times the critsect is busy during EOI writes causing trips to R3.");