Re: [tip:perf/urgent] perf, x86: Catch spurious interrupts after disabling counters
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
- Subject: Re: [tip:perf/urgent] perf, x86: Catch spurious interrupts after disabling counters
- From: Robert Richter <robert.richter@xxxxxxx>
- Date: Wed, 29 Sep 2010 14:54:53 +0200
- Cc: "mingo@xxxxxxxxxx" <mingo@xxxxxxxxxx>, "hpa@xxxxxxxxx" <hpa@xxxxxxxxx>, "linux-kernel@xxxxxxxxxxxxxxx" <linux-kernel@xxxxxxxxxxxxxxx>, "yinghai@xxxxxxxxxx" <yinghai@xxxxxxxxxx>, "andi@xxxxxxxxxxxxxx" <andi@xxxxxxxxxxxxxx>, "peterz@xxxxxxxxxxxxx" <peterz@xxxxxxxxxxxxx>, "gorcunov@xxxxxxxxx" <gorcunov@xxxxxxxxx>, "ying.huang@xxxxxxxxx" <ying.huang@xxxxxxxxx>, "fweisbec@xxxxxxxxx" <fweisbec@xxxxxxxxx>, "ming.m.lin@xxxxxxxxx" <ming.m.lin@xxxxxxxxx>, "tglx@xxxxxxxxxxxxx" <tglx@xxxxxxxxxxxxx>, "dzickus@xxxxxxxxxx" <dzickus@xxxxxxxxxx>, "mingo@xxxxxxx" <mingo@xxxxxxx>, "linux-tip-commits@xxxxxxxxxxxxxxx" <linux-tip-commits@xxxxxxxxxxxxxxx>
- In-reply-to: <20100929125301.GG13563@xxxxxxxxxxxx>
- User-agent: Mutt/1.5.20 (2009-06-14)
On 29.09.10 14:53:01, Robert Richter wrote:
> Stephane,
>
> On 29.09.10 08:26:41, Stephane Eranian wrote:
> > You've applied the fix only to the generic X86 interrupt handler
> > which is currently used by AMD64 processors.
>
> (... and P4).
>
> > It seems to me that this "in-flight interrupt after disable" problem
> > may also happen on Intel and should therefore also be added
> > to intel_pmu_handle_irq(). Don't you think so?
>
> It only happens if the active_mask is used for checking single
> counters for overflows.
>
> Systems with Intel Architectural Perfmon use the status mask msr to
> determine which counter overflowed. In intel_pmu_handle_irq() the
> handled counter is incremented in this case even if the counter is not
I mean handled count (variable 'handled').
> active, so everything should be fine here.
>
> -Robert
>
> --
> Advanced Micro Devices, Inc.
> Operating System Research Center
--
Advanced Micro Devices, Inc.
Operating System Research Center
--
To unsubscribe from this list: send the line "unsubscribe linux-tip-commits" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
[Index of Archives]
[Linux Stable Commits]
[Linux Stable Kernel]
[Linux Kernel]
[Linux USB Devel]
[Linux Video &Media]
[Linux Audio Users]
[Yosemite News]
[Linux SCSI]