Re: [PATCH] cpuidle: menu: Retain tick when shallow state is selected

2018-08-28 Thread Rafael J. Wysocki
On Sunday, August 26, 2018 9:37:09 PM CEST Doug Smythies wrote: > On 2018.08.25 04:22 Rafael J. Wysocki wrote: > > On Fri, Aug 24, 2018 at 5:52 PM Doug Smythies wrote: > >> On 2018.08.24 02:44 Rafael J. Wysocki wrote: > >>> On Tuesday, August 21, 2018 10:44:10 AM CEST Rafael J. Wysocki wrote: >

Re: [PATCH] cpuidle: menu: Retain tick when shallow state is selected

2018-08-28 Thread Rafael J. Wysocki
On Sunday, August 26, 2018 9:37:09 PM CEST Doug Smythies wrote: > On 2018.08.25 04:22 Rafael J. Wysocki wrote: > > On Fri, Aug 24, 2018 at 5:52 PM Doug Smythies wrote: > >> On 2018.08.24 02:44 Rafael J. Wysocki wrote: > >>> On Tuesday, August 21, 2018 10:44:10 AM CEST Rafael J. Wysocki wrote: >

RE: [PATCH] cpuidle: menu: Retain tick when shallow state is selected

2018-08-26 Thread Doug Smythies
On 2018.08.25 04:22 Rafael J. Wysocki wrote: > On Fri, Aug 24, 2018 at 5:52 PM Doug Smythies wrote: >> On 2018.08.24 02:44 Rafael J. Wysocki wrote: >>> On Tuesday, August 21, 2018 10:44:10 AM CEST Rafael J. Wysocki wrote: From: Rafael J. Wysocki The case addressed by commit

RE: [PATCH] cpuidle: menu: Retain tick when shallow state is selected

2018-08-26 Thread Doug Smythies
On 2018.08.25 04:22 Rafael J. Wysocki wrote: > On Fri, Aug 24, 2018 at 5:52 PM Doug Smythies wrote: >> On 2018.08.24 02:44 Rafael J. Wysocki wrote: >>> On Tuesday, August 21, 2018 10:44:10 AM CEST Rafael J. Wysocki wrote: From: Rafael J. Wysocki The case addressed by commit

Re: [PATCH] cpuidle: menu: Retain tick when shallow state is selected

2018-08-25 Thread Rafael J. Wysocki
On Fri, Aug 24, 2018 at 5:52 PM Doug Smythies wrote: > > On 2018.08.24 02:44 Rafael J. Wysocki wrote: > > On Tuesday, August 21, 2018 10:44:10 AM CEST Rafael J. Wysocki wrote: > >> From: Rafael J. Wysocki > >> > >> The case addressed by commit 5ef499cd571c (cpuidle: menu: Handle > >> stopped

Re: [PATCH] cpuidle: menu: Retain tick when shallow state is selected

2018-08-25 Thread Rafael J. Wysocki
On Fri, Aug 24, 2018 at 5:52 PM Doug Smythies wrote: > > On 2018.08.24 02:44 Rafael J. Wysocki wrote: > > On Tuesday, August 21, 2018 10:44:10 AM CEST Rafael J. Wysocki wrote: > >> From: Rafael J. Wysocki > >> > >> The case addressed by commit 5ef499cd571c (cpuidle: menu: Handle > >> stopped

RE: [PATCH] cpuidle: menu: Retain tick when shallow state is selected

2018-08-24 Thread Doug Smythies
On 2018.08.24 02:44 Rafael J. Wysocki wrote: > On Tuesday, August 21, 2018 10:44:10 AM CEST Rafael J. Wysocki wrote: >> From: Rafael J. Wysocki >> >> The case addressed by commit 5ef499cd571c (cpuidle: menu: Handle >> stopped tick more aggressively) in the stopped tick case is present >> when

RE: [PATCH] cpuidle: menu: Retain tick when shallow state is selected

2018-08-24 Thread Doug Smythies
On 2018.08.24 02:44 Rafael J. Wysocki wrote: > On Tuesday, August 21, 2018 10:44:10 AM CEST Rafael J. Wysocki wrote: >> From: Rafael J. Wysocki >> >> The case addressed by commit 5ef499cd571c (cpuidle: menu: Handle >> stopped tick more aggressively) in the stopped tick case is present >> when

Re: [PATCH] cpuidle: menu: Retain tick when shallow state is selected

2018-08-24 Thread Rafael J. Wysocki
On Tuesday, August 21, 2018 10:44:10 AM CEST Rafael J. Wysocki wrote: > From: Rafael J. Wysocki > > The case addressed by commit 5ef499cd571c (cpuidle: menu: Handle > stopped tick more aggressively) in the stopped tick case is present > when the tick has not been stopped yet too. Namely, if

Re: [PATCH] cpuidle: menu: Retain tick when shallow state is selected

2018-08-24 Thread Rafael J. Wysocki
On Tuesday, August 21, 2018 10:44:10 AM CEST Rafael J. Wysocki wrote: > From: Rafael J. Wysocki > > The case addressed by commit 5ef499cd571c (cpuidle: menu: Handle > stopped tick more aggressively) in the stopped tick case is present > when the tick has not been stopped yet too. Namely, if

Re: [PATCH] cpuidle: menu: Retain tick when shallow state is selected

2018-08-22 Thread Rafael J. Wysocki
On Wed, Aug 22, 2018 at 2:02 PM wrote: > > On Tue, Aug 21, 2018 at 10:44:10AM +0200, Rafael J . Wysocki wrote: > > From: Rafael J. Wysocki > > > > The case addressed by commit 5ef499cd571c (cpuidle: menu: Handle > > stopped tick more aggressively) in the stopped tick case is present > > when the

Re: [PATCH] cpuidle: menu: Retain tick when shallow state is selected

2018-08-22 Thread Rafael J. Wysocki
On Wed, Aug 22, 2018 at 2:02 PM wrote: > > On Tue, Aug 21, 2018 at 10:44:10AM +0200, Rafael J . Wysocki wrote: > > From: Rafael J. Wysocki > > > > The case addressed by commit 5ef499cd571c (cpuidle: menu: Handle > > stopped tick more aggressively) in the stopped tick case is present > > when the

Re: [PATCH] cpuidle: menu: Retain tick when shallow state is selected

2018-08-22 Thread leo . yan
On Wed, Aug 22, 2018 at 08:02:00PM +0800, Leo Yan wrote: [...] > > + if (!tick_nohz_tick_stopped()) { > > + /* > > +* If the state selected so far is shallow, > > +* waking up early won't hurt, so

Re: [PATCH] cpuidle: menu: Retain tick when shallow state is selected

2018-08-22 Thread leo . yan
On Wed, Aug 22, 2018 at 08:02:00PM +0800, Leo Yan wrote: [...] > > + if (!tick_nohz_tick_stopped()) { > > + /* > > +* If the state selected so far is shallow, > > +* waking up early won't hurt, so

Re: [PATCH] cpuidle: menu: Retain tick when shallow state is selected

2018-08-22 Thread leo . yan
On Tue, Aug 21, 2018 at 10:44:10AM +0200, Rafael J . Wysocki wrote: > From: Rafael J. Wysocki > > The case addressed by commit 5ef499cd571c (cpuidle: menu: Handle > stopped tick more aggressively) in the stopped tick case is present > when the tick has not been stopped yet too. Namely, if only

Re: [PATCH] cpuidle: menu: Retain tick when shallow state is selected

2018-08-22 Thread leo . yan
On Tue, Aug 21, 2018 at 10:44:10AM +0200, Rafael J . Wysocki wrote: > From: Rafael J. Wysocki > > The case addressed by commit 5ef499cd571c (cpuidle: menu: Handle > stopped tick more aggressively) in the stopped tick case is present > when the tick has not been stopped yet too. Namely, if only

[PATCH] cpuidle: menu: Retain tick when shallow state is selected

2018-08-21 Thread Rafael J. Wysocki
From: Rafael J. Wysocki The case addressed by commit 5ef499cd571c (cpuidle: menu: Handle stopped tick more aggressively) in the stopped tick case is present when the tick has not been stopped yet too. Namely, if only two CPU idle states, shallow state A with target residency significantly below

[PATCH] cpuidle: menu: Retain tick when shallow state is selected

2018-08-21 Thread Rafael J. Wysocki
From: Rafael J. Wysocki The case addressed by commit 5ef499cd571c (cpuidle: menu: Handle stopped tick more aggressively) in the stopped tick case is present when the tick has not been stopped yet too. Namely, if only two CPU idle states, shallow state A with target residency significantly below