While benign as long as it's using a direct-APIC-vector (handlers for which are called with IRQs off), follow the more usual pattern of ack-ing the IRQ only after handling it.
Requested-by: Andrew Cooper <[email protected]> Signed-off-by: Jan Beulich <[email protected]> --- v2: New. --- a/xen/arch/x86/apic.c +++ b/xen/arch/x86/apic.c @@ -1319,8 +1319,8 @@ static void cf_check error_interrupt(voi static void cf_check pmu_interrupt(void) { - ack_APIC_irq(); vpmu_do_interrupt(); + ack_APIC_irq(); } void __init apic_intr_init(void)
