Re: [PATCH 5/5] powerpc/64s: fix POWER9 machine check handler from stop state

2017-03-10 Thread Gautham R Shenoy
On Wed, Mar 08, 2017 at 10:19:19PM +1000, Nicholas Piggin wrote: > Hi Gautham, > > I'm just getting back to this. Sorry for the late reply, and > thanks for the reviews. No problems. > [..snip..] > > > +#ifdef CONFIG_PPC_P7_NAP > > > +EXC_COMMON_BEGIN(machine_check_idle_common) > > > + bl

Re: [PATCH 5/5] powerpc/64s: fix POWER9 machine check handler from stop state

2017-03-08 Thread Nicholas Piggin
Hi Gautham, I'm just getting back to this. Sorry for the late reply, and thanks for the reviews. On Tue, 28 Feb 2017 22:45:46 +0530 Gautham R Shenoy wrote: > Hi Nick, > > On Fri, Feb 17, 2017 at 12:09 AM, Nicholas Piggin wrote: > > The ISA specifies power save wakeup can cause a machine check

Re: [PATCH 5/5] powerpc/64s: fix POWER9 machine check handler from stop state

2017-02-28 Thread Gautham R Shenoy
Hi Nick, On Fri, Feb 17, 2017 at 12:09 AM, Nicholas Piggin wrote: > The ISA specifies power save wakeup can cause a machine check interrupt. > The machine check handler currently has code to handle that for POWER8, > but POWER9 crashes when trying to execute the P8 style sleep > instructions. > >

[PATCH 5/5] powerpc/64s: fix POWER9 machine check handler from stop state

2017-02-16 Thread Nicholas Piggin
The ISA specifies power save wakeup can cause a machine check interrupt. The machine check handler currently has code to handle that for POWER8, but POWER9 crashes when trying to execute the P8 style sleep instructions. So queue up the machine check, then call into the idle code to wake up as the