Re: [PATCH 1/2] powerpc: fix graceful debugger recovery

2016-11-10 Thread Michael Ellerman
Nicholas Piggin writes: > On Thu, 10 Nov 2016 12:35:59 +1100 > Michael Ellerman wrote: > >> Nicholas Piggin writes: >> >> > When exiting xmon with 'x' (exit and recover), oops_begin bails >> > out immediately, but die then calls __die() and oops_end(), which >> > cause a lot of bad things to h

Re: [PATCH 1/2] powerpc: fix graceful debugger recovery

2016-11-09 Thread Nicholas Piggin
On Thu, 10 Nov 2016 12:35:59 +1100 Michael Ellerman wrote: > Nicholas Piggin writes: > > > When exiting xmon with 'x' (exit and recover), oops_begin bails > > out immediately, but die then calls __die() and oops_end(), which > > cause a lot of bad things to happen. > > In fact oops_begin() r

Re: [PATCH 1/2] powerpc: fix graceful debugger recovery

2016-11-09 Thread Michael Ellerman
Nicholas Piggin writes: > When exiting xmon with 'x' (exit and recover), oops_begin bails > out immediately, but die then calls __die() and oops_end(), which > cause a lot of bad things to happen. In fact oops_begin() returns 1, which oops_end() then passes directly to raw_local_irq_restore() as

[PATCH 1/2] powerpc: fix graceful debugger recovery

2016-11-08 Thread Nicholas Piggin
When exiting xmon with 'x' (exit and recover), oops_begin bails out immediately, but die then calls __die() and oops_end(), which cause a lot of bad things to happen. If the debugger was attached then went to graceful recovery, exit from die() immediately. Signed-off-by: Nicholas Piggin --- arc