Re: [Skiboot] [PATCH 1/2] SLW: Remove stop1_lite and stop0 stop states

2018-05-10 Thread Nicholas Piggin
On Thu, 10 May 2018 14:29:44 +0530 Akshay Adiga wrote: > On Thu, May 03, 2018 at 08:15:59PM +1000, Nicholas Piggin wrote: > > On Thu, 03 May 2018 20:03:55 +1000 > > Stewart Smith wrote: > > > > > Nicholas Piggin writes: > > > > On Thu, 3 May 2018 14:36:47 +0530 > > > > Akshay Adiga wrote:

Re: [Skiboot] [PATCH 1/2] SLW: Remove stop1_lite and stop0 stop states

2018-05-10 Thread Akshay Adiga
On Thu, May 03, 2018 at 08:15:59PM +1000, Nicholas Piggin wrote: > On Thu, 03 May 2018 20:03:55 +1000 > Stewart Smith wrote: > > > Nicholas Piggin writes: > > > On Thu, 3 May 2018 14:36:47 +0530 > > > Akshay Adiga wrote: > > > > > >> On Tue, May 01, 2018 at 01:47:23PM +1000, Nicholas Piggin w

Re: [Skiboot] [PATCH 1/2] SLW: Remove stop1_lite and stop0 stop states

2018-05-06 Thread Stewart Smith
Michael Ellerman writes: > Stewart Smith writes: > ... >> >> Slightly stupid question: should we be disabling these here or should >> Linux be better and deciding what states to use? >> >> I'm inclined to say this is a Linux problem as it should make the >> decision of what hardware feature to us

Re: [Skiboot] [PATCH 1/2] SLW: Remove stop1_lite and stop0 stop states

2018-05-03 Thread Michael Ellerman
Stewart Smith writes: ... > > Slightly stupid question: should we be disabling these here or should > Linux be better and deciding what states to use? > > I'm inclined to say this is a Linux problem as it should make the > decision of what hardware feature to used based on the ones OPAL says > *ca

Re: [Skiboot] [PATCH 1/2] SLW: Remove stop1_lite and stop0 stop states

2018-05-03 Thread Nicholas Piggin
On Thu, 03 May 2018 20:03:55 +1000 Stewart Smith wrote: > Nicholas Piggin writes: > > On Thu, 3 May 2018 14:36:47 +0530 > > Akshay Adiga wrote: > > > >> On Tue, May 01, 2018 at 01:47:23PM +1000, Nicholas Piggin wrote: > >> > On Mon, 30 Apr 2018 14:42:08 +0530 > >> > Akshay Adiga wrote: > >

Re: [Skiboot] [PATCH 1/2] SLW: Remove stop1_lite and stop0 stop states

2018-05-03 Thread Stewart Smith
Nicholas Piggin writes: > On Thu, 3 May 2018 14:36:47 +0530 > Akshay Adiga wrote: > >> On Tue, May 01, 2018 at 01:47:23PM +1000, Nicholas Piggin wrote: >> > On Mon, 30 Apr 2018 14:42:08 +0530 >> > Akshay Adiga wrote: >> > >> > > Powersaving for stop0_lite and stop1_lite is observed to be quit

Re: [Skiboot] [PATCH 1/2] SLW: Remove stop1_lite and stop0 stop states

2018-05-03 Thread Nicholas Piggin
On Thu, 3 May 2018 14:36:47 +0530 Akshay Adiga wrote: > On Tue, May 01, 2018 at 01:47:23PM +1000, Nicholas Piggin wrote: > > On Mon, 30 Apr 2018 14:42:08 +0530 > > Akshay Adiga wrote: > > > > > Powersaving for stop0_lite and stop1_lite is observed to be quite similar > > > and both states res

Re: [Skiboot] [PATCH 1/2] SLW: Remove stop1_lite and stop0 stop states

2018-05-03 Thread Akshay Adiga
On Tue, May 01, 2018 at 01:47:23PM +1000, Nicholas Piggin wrote: > On Mon, 30 Apr 2018 14:42:08 +0530 > Akshay Adiga wrote: > > > Powersaving for stop0_lite and stop1_lite is observed to be quite similar > > and both states resume without state loss. Using context_switch test [1] > > we observe t

Re: [Skiboot] [PATCH 1/2] SLW: Remove stop1_lite and stop0 stop states

2018-04-30 Thread Nicholas Piggin
On Mon, 30 Apr 2018 14:42:08 +0530 Akshay Adiga wrote: > Powersaving for stop0_lite and stop1_lite is observed to be quite similar > and both states resume without state loss. Using context_switch test [1] > we observe that stop0_lite has slightly lower latency, hence removing > stop1_lite. > >