Wim: Don is correct. We did have a separate discussion about this after I denied the original patch. It is all good now. And thanks for cutting in the port 0x72 patch. Thanks, Tom -----Original Message----- From: Don Zickus [mailto:dzickus@xxxxxxxxxx] Sent: Thursday, April 26, 2012 7:27 AM To: Wim Van Sebroeck Cc: mingo@xxxxxxxxxx; hpa@xxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; torvalds@xxxxxxxxxxxxxxxxxxxx; peterz@xxxxxxxxxxxxx; Mingarelli, Thomas; akpm@xxxxxxxxxxxxxxxxxxxx; tglx@xxxxxxxxxxxxx; linux-tip-commits@xxxxxxxxxxxxxxx Subject: Re: [tip:core/locking] watchdog, hpwdt: Remove priority option for NMI callback On Thu, Apr 26, 2012 at 09:13:39AM +0200, Wim Van Sebroeck wrote: > > > > Therefore hpwdt's priority mechanism doesn't make sense any > > more. They will be always first on the NMI_UNKNOWN queue, if > > they register. > > > > Removing this parameter cleans up the code and simplifies things > > for the next patch which changes how nmis are registered. > > > > Signed-off-by: Don Zickus <dzickus@xxxxxxxxxx> > > Cc: Thomas Mingarelli <thomas.mingarelli@xxxxxx> > > Cc: Wim Van Sebroeck <wim@xxxxxxxxx> > > Cc: Peter Zijlstra <peterz@xxxxxxxxxxxxx> > > Cc: Linus Torvalds <torvalds@xxxxxxxxxxxxxxxxxxxx> > > Cc: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> > > Link: http://lkml.kernel.org/r/1333051877-15755-2-git-send-email-dzickus@xxxxxxxxxx > > Signed-off-by: Ingo Molnar <mingo@xxxxxxxxxx> > > This is the feedback I have from Tom which he discussed with Don: > > I don't like this patch because the Virtual NMI button doesn't come through the pretimeout routine. It is taken by the > system as an IOCK NMI error and no log messages in our IML. > > Our BIOS is not able to source the NMI. > > And since then it became quiet. Imho: this needs more discussion... Tom and I discussed this offline. The result was patch 2 of this series. The problem he had, had nothing to do with this patch (which was just a cleanup really). Tom tested the second patch and was happy with the results. If there is any other issues, I am assuming Tom would have let me know a while ago. But I believe all his issues are addressed. Tom? Cheers, Don -- 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