From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755825AbaEOU3e (ORCPT ); Thu, 15 May 2014 16:29:34 -0400 Received: from mx1.redhat.com ([209.132.183.28]:27625 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755762AbaEOU3c (ORCPT ); Thu, 15 May 2014 16:29:32 -0400 Date: Thu, 15 May 2014 16:28:48 -0400 From: Don Zickus To: x86@kernel.org Cc: Peter Zijlstra , Andi Kleen , gong.chen@linux.intel.com, LKML , Elliott@hp.com Subject: Re: [PATCH 0/6 V2] x86, nmi: Various fixes and cleanups Message-ID: <20140515202848.GJ50500@redhat.com> References: <1400181949-157170-1-git-send-email-dzickus@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1400181949-157170-1-git-send-email-dzickus@redhat.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, May 15, 2014 at 03:25:43PM -0400, Don Zickus wrote: > I started this patch by fixing a performance problem with the GHES > NMI handler and then things evolved to more patches as I was poking > around in the code. > > The main focus was moving the GHES NMI driver to its own NMI subtype > to avoid slowing down perf handling. Then I decided to move the > default external NMI handler to its own routine. Then finally, I > needed to see which NMI handlers were registered so I hacked up > /proc/interrupts to show me. > > Tested mostly on HP boxes that have GHES enabled and having the iLO > send NMIs to panic the box (using hpwdt driver). Ran perf on other > GHES enabled boxes to test performance results. > > V2: adding irq_work items to handled possible lost NMIs (new patch 1) > modified output of /proc/interrupts based on feedback (patch 6) > ugh. Forgot to snip below.. > Don Zickus (5): > x86, nmi: Add new nmi type 'external' > x86, nmi: Add boot line option 'panic_on_unrecovered_nmi' and > 'panic_on_io_nmi' > x86, nmi: Remove 'reason' value from unknown nmi output > x86, nmi: Move default external NMI handler to its own routine > x86, nmi: Add better NMI stats to /proc/interrupts and show handlers > > Documentation/kernel-parameters.txt | 9 ++ > arch/x86/include/asm/nmi.h | 5 + > arch/x86/kernel/irq.c | 3 + > arch/x86/kernel/nmi.c | 199 ++++++++++++++++++++++++++-------- > drivers/acpi/apei/ghes.c | 4 +- > drivers/watchdog/hpwdt.c | 24 +++-- > 6 files changed, 187 insertions(+), 57 deletions(-) > > *** BLURB HERE *** .. to here. Cheers, Don > > Don Zickus (6): > x86, nmi: Implement delayed irq_work mechanism to handle lost NMIs > x86, nmi: Add new nmi type 'external' > x86, nmi: Add boot line option 'panic_on_unrecovered_nmi' and > 'panic_on_io_nmi' > x86, nmi: Remove 'reason' value from unknown nmi output > x86, nmi: Move default external NMI handler to its own routine > x86, nmi: Add better NMI stats to /proc/interrupts and show handlers > > Documentation/kernel-parameters.txt | 9 + > arch/x86/include/asm/nmi.h | 5 + > arch/x86/kernel/irq.c | 3 + > arch/x86/kernel/nmi.c | 302 +++++++++++++++++++++++++++++----- > drivers/acpi/apei/ghes.c | 4 +- > drivers/watchdog/hpwdt.c | 24 ++- > 6 files changed, 292 insertions(+), 55 deletions(-) >