lkml.org 
[lkml]   [2013]   [Feb]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 5/5] perf, x86: Move NMI clearing to end of PMI handler after the counter registers are reset

* Andi Kleen <andi@firstfloor.org> wrote:

> On Tue, Feb 12, 2013 at 09:43:46AM +0100, Ingo Molnar wrote:
> > Was this stress-tested on all affected main CPU types, or only
> > on Haswell?
>
> I tested it on Haswell and Ivy Bridge. I can also try Westmere
> and a Saltwell(Atom), but for the majority of other family 6
> systems I'll need to rely on the community.

The systems you tested should be OK.

> White listing is somewhat difficult because it affects the
> architectural mode too.

Yeah, I'd rather avoid that.

> I don't really expect problems from this change, we should
> probably have always done it like this.

I expect potential problems: the ordering of the operations in
the NMI handler was always very fragile, resulting in hard to
debug hangs - which sometimes needed hours long very intense PMU
stress-testing to trigger.

That is why I asked how heavily you've tested this. Once the
series passes review I'll keep this patch last to make it easy
to revert/zap if it causes problems.

Thanks,

Ingo


\
 
 \ /
  Last update: 2013-02-13 11:01    [W:1.142 / U:0.048 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site