Re: [PATCH] powerpc/traps: restore recoverability of machine_check interrupts
From: Nicholas Piggin <npiggin@gmail.com>
Date: 2018-10-16 01:10:18
Also in:
lkml, stable
From: Nicholas Piggin <npiggin@gmail.com>
Date: 2018-10-16 01:10:18
Also in:
lkml, stable
On Mon, 15 Oct 2018 09:40:50 +0200 Christophe LEROY [off-list ref] wrote:
Cc: stable@vger.kernel.org <redacted> Le 13/10/2018 à 11:16, Christophe Leroy a écrit :quoted
commit b96672dd840f ("powerpc: Machine check interrupt is a non- maskable interrupt") added a call to nmi_enter() at the beginning of machine check restart exception handler. Due to that, in_interrupt() always returns true regardless of the state before entering the exception, and die() panics even when the system was not already in interrupt. This patch calls nmi_exit() before calling die() in order to restore the interrupt state we had before calling nmi_enter() Fixes: b96672dd840f ("powerpc: Machine check interrupt is a non-maskable interrupt") Signed-off-by: Christophe Leroy <redacted>
Reviewed-by: Nicholas Piggin <npiggin@gmail.com> This looks good to me and probably the simplest fix. powernv will need a fix on top of this, to remove the die and let it just fall through unrecovered to the traps.c code. pseries seems to be okay. But this patch looks good. Thanks, Nick