diff --git a/arch/x86/oprofile/op_model_ppro.c b/arch/x86/oprofile/op_model_ppro.c index 3f1b81a..8484528 100644 --- a/arch/x86/oprofile/op_model_ppro.c +++ b/arch/x86/oprofile/op_model_ppro.c @@ -126,6 +126,12 @@ static int ppro_check_ctrs(struct pt_regs * const regs, u64 val; int i; + /* + * We need to unmask the apic vector *before* writing reset_value + * to msr counter, because we use edge trigger + */ + apic_write(APIC_LVTPC, apic_read(APIC_LVTPC) & ~APIC_LVT_MASKED); + for (i = 0 ; i < num_counters; ++i) { if (!reset_value[i]) continue; @@ -136,10 +142,6 @@ static int ppro_check_ctrs(struct pt_regs * const regs, } } - /* Only P6 based Pentium M need to re-unmask the apic vector but it - * doesn't hurt other P6 variant */ - apic_write(APIC_LVTPC, apic_read(APIC_LVTPC) & ~APIC_LVT_MASKED); - /* We can't work out if we really handled an interrupt. We * might have caught a *second* counter just after overflowing * the interrupt for this counter then arrives