Changeset 89734 in vbox

Timestamp:

Jun 16, 2021 8:08:42 AM (3 years ago)

Author:

vboxsync

Message:

DevIoApic: Added stats to track how long level-triggered interrupts spend in the pending (active) state on the I/O APIC.

File:

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

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

@@ -320 +320 @@
      *  APIC(s). */
     STAMCOUNTER             StatEoiReceived;
+    /** The time an interrupt level spent in the pending state. */
+    STAMPROFILEADV          aStatLevelAct[IOAPIC_NUM_INTR_PINS];
 #endif
     /** Per-vector stats. */
@@ -677 +679 @@
         pThis->au64RedirTable[idxRte] |= IOAPIC_RTE_REMOTE_IRR;
         STAM_COUNTER_INC(&pThis->StatLevelIrqSent);
+        STAM_PROFILE_ADV_START(&pThis->aStatLevelAct[idxRte], a);
     }
 }
@@ -866 +869 @@
             pThis->au64RedirTable[idxRte] &= ~IOAPIC_RTE_REMOTE_IRR;
             fRemoteIrrCleared = true;
+            STAM_PROFILE_ADV_STOP(&pThis->aStatLevelAct[idxRte], a);
             STAM_COUNTER_INC(&pThis->StatEoiReceived);
             Log2(("IOAPIC: ioapicSetEoi: Cleared remote IRR, idxRte=%u vector=%#x (%u)\n", idxRte, u8Vector, u8Vector));
@@ -1667 +1671 @@
     PDMDevHlpSTAMRegister(pDevIns, &pThis->StatLevelIrqSent, STAMTYPE_COUNTER, "LevelIntr/Sent", STAMUNIT_OCCURENCES, "Number of level-triggered interrupts sent to the local APIC(s).");
     PDMDevHlpSTAMRegister(pDevIns, &pThis->StatEoiReceived,  STAMTYPE_COUNTER, "LevelIntr/Recv", STAMUNIT_OCCURENCES, "Number of EOIs received for level-triggered interrupts from the local APIC(s).");
+
+    for (int i = 0; i < RT_ELEMENTS(pThis->aStatLevelAct); ++i)
+        PDMDevHlpSTAMRegisterF(pDevIns, &pThis->aStatLevelAct[i], STAMTYPE_PROFILE, STAMVISIBILITY_ALWAYS, STAMUNIT_TICKS_PER_CALL, "Time spent in the level active state", "IntPending/%02x", i);
 # endif
     for (size_t i = 0; i < RT_ELEMENTS(pThis->aStatVectors); i++)