Re: [PATCH -v3] perf, x86: try to handle unknown nmis with running perfctrs

From: Ingo Molnar <mingo@elte.hu>
To: Don Zickus <dzickus@redhat.com>
Cc: Peter Zijlstra <peterz@infradead.org>,
	Robert Richter <robert.richter@amd.com>,
	Cyrill Gorcunov <gorcunov@gmail.com>,
	Lin Ming <ming.m.lin@intel.com>,
	"fweisbec@gmail.com" <fweisbec@gmail.com>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"Huang, Ying" <ying.huang@intel.com>,
	Yinghai Lu <yinghai@kernel.org>, Andi Kleen <andi@firstfloor.org>
Subject: Re: [PATCH -v3] perf, x86: try to handle unknown nmis with running perfctrs
Date: Mon, 23 Aug 2010 10:53:39 +0200	[thread overview]
Message-ID: <20100823085339.GA26713@elte.hu> (raw)
In-Reply-To: <20100820152510.GA4167@elte.hu>

* Ingo Molnar <mingo@elte.hu> wrote:

> 
> * Don Zickus <dzickus@redhat.com> wrote:
> 
> > I'll test tip later today to see if I can reproduce it.
> > 
> > Cheers,
> > Don
> > 
> > Ingo Molnar <mingo@elte.hu> wrote:
> > 
> > >
> > >it's not working so well, i'm getting:
> > >
> > > Uhhuh. NMI received for unknown reason 00 on CPU 9.
> > > Do you have a strange power saving mode enabled?
> > > Dazed and confused, but trying to continue
> > >
> > >on a nehalem box, after a perf top and perf stat run.
> 
> FYI, it does not trigger on an AMD box.

Ok, to not hold up the perf/urgent flow i zapped these two commits for 
the time being:

 4a31beb: perf, x86: Fix handle_irq return values
 8e3e42b: perf, x86: Try to handle unknown nmis with an enabled PMU

We can apply them if they take a form that dont introduce a different 
kind of (and more visible) regression.

Thanks,

	Ingo