Re: [PATCH 2/3] powernv:idle: Decouple TB restore & Per-core SPRs restore

2017-04-13 Thread Gautham R Shenoy
On Thu, Apr 13, 2017 at 04:55:45PM +1000, Michael Neuling wrote: > On Wed, 2017-04-12 at 17:16 +0530, Gautham R. Shenoy wrote: > > From: "Gautham R. Shenoy" > > > > The idle-exit code assumes that if Timebase is not lost, then neither > > are the per-core hypervisor resources lost. > > Double n

Re: [PATCH 2/3] powernv:idle: Decouple TB restore & Per-core SPRs restore

2017-04-12 Thread Michael Neuling
On Wed, 2017-04-12 at 17:16 +0530, Gautham R. Shenoy wrote: > From: "Gautham R. Shenoy" > > The idle-exit code assumes that if Timebase is not lost, then neither > are the per-core hypervisor resources lost. Double negative! How about: The idle-exit code assumes that if the timebase is restor

[PATCH 2/3] powernv:idle: Decouple TB restore & Per-core SPRs restore

2017-04-12 Thread Gautham R. Shenoy
From: "Gautham R. Shenoy" The idle-exit code assumes that if Timebase is not lost, then neither are the per-core hypervisor resources lost. This was true on POWER8 where fast-sleep lost only TB but not per-core resources, and winkle lost both. This assumption is not true for POWER9 however, sinc