Re: [RFT][PATCH v4 4/7] cpuidle: Return nohz hint from cpuidle_select()

2018-03-15 Thread Rafael J. Wysocki
On Wednesday, March 14, 2018 1:59:29 PM CET Peter Zijlstra wrote: > On Mon, Mar 12, 2018 at 10:54:18AM +0100, Rafael J. Wysocki wrote: > > @@ -378,6 +384,26 @@ static int menu_select(struct cpuidle_dr > > if (idx == -1) > > idx = 0; /* No states enabled. Must use 0. */ > > > > +

Re: [RFT][PATCH v4 4/7] cpuidle: Return nohz hint from cpuidle_select()

2018-03-15 Thread Rafael J. Wysocki
On Wednesday, March 14, 2018 1:59:29 PM CET Peter Zijlstra wrote: > On Mon, Mar 12, 2018 at 10:54:18AM +0100, Rafael J. Wysocki wrote: > > @@ -378,6 +384,26 @@ static int menu_select(struct cpuidle_dr > > if (idx == -1) > > idx = 0; /* No states enabled. Must use 0. */ > > > > +

Re: [RFT][PATCH v4 4/7] cpuidle: Return nohz hint from cpuidle_select()

2018-03-14 Thread Peter Zijlstra
On Mon, Mar 12, 2018 at 10:54:18AM +0100, Rafael J. Wysocki wrote: > @@ -378,6 +384,26 @@ static int menu_select(struct cpuidle_dr > if (idx == -1) > idx = 0; /* No states enabled. Must use 0. */ > > + if (drv->states[idx].flags & CPUIDLE_FLAG_POLLING) { > +

Re: [RFT][PATCH v4 4/7] cpuidle: Return nohz hint from cpuidle_select()

2018-03-14 Thread Peter Zijlstra
On Mon, Mar 12, 2018 at 10:54:18AM +0100, Rafael J. Wysocki wrote: > @@ -378,6 +384,26 @@ static int menu_select(struct cpuidle_dr > if (idx == -1) > idx = 0; /* No states enabled. Must use 0. */ > > + if (drv->states[idx].flags & CPUIDLE_FLAG_POLLING) { > +

[RFT][PATCH v4 4/7] cpuidle: Return nohz hint from cpuidle_select()

2018-03-12 Thread Rafael J. Wysocki
From: Rafael J. Wysocki Add a new pointer argument to cpuidle_select() and to the ->select cpuidle governor callback to allow a boolean value indicating whether or not the tick should be stopped before entering the selected state to be returned from there. Make the

[RFT][PATCH v4 4/7] cpuidle: Return nohz hint from cpuidle_select()

2018-03-12 Thread Rafael J. Wysocki
From: Rafael J. Wysocki Add a new pointer argument to cpuidle_select() and to the ->select cpuidle governor callback to allow a boolean value indicating whether or not the tick should be stopped before entering the selected state to be returned from there. Make the ladder governor ignore that