Re: [PATCH 04/16] sched/fair: Optimize find_idlest_cpu() when there is no choice

2016-06-07 Thread Morten Rasmussen
On Wed, Jun 01, 2016 at 09:59:20PM +0200, Peter Zijlstra wrote: > On Mon, May 23, 2016 at 11:58:46AM +0100, Morten Rasmussen wrote: > > In the current find_idlest_group()/find_idlest_cpu() search we end up > > calling find_idlest_cpu() in a sched_group containing only one cpu in > > the end. Checki

Re: [PATCH 04/16] sched/fair: Optimize find_idlest_cpu() when there is no choice

2016-06-01 Thread Peter Zijlstra
On Mon, May 23, 2016 at 11:58:46AM +0100, Morten Rasmussen wrote: > In the current find_idlest_group()/find_idlest_cpu() search we end up > calling find_idlest_cpu() in a sched_group containing only one cpu in > the end. Checking idle-states becomes pointless when there is no > alternative, so bail

Re: [PATCH 04/16] sched/fair: Optimize find_idlest_cpu() when there is no choice

2016-05-24 Thread Mike Galbraith
On Tue, 2016-05-24 at 09:05 +0100, Morten Rasmussen wrote: > On Tue, May 24, 2016 at 08:29:05AM +0200, Mike Galbraith wrote: > > > > > +> > > > > > > > /* Check if we have any choice */ > > > +> > > > > > > > if (group->group_weight == 1) { > > > +> > > > > > > > > > > > > > > return > > >

Re: [PATCH 04/16] sched/fair: Optimize find_idlest_cpu() when there is no choice

2016-05-24 Thread Morten Rasmussen
On Tue, May 24, 2016 at 08:29:05AM +0200, Mike Galbraith wrote: > On Mon, 2016-05-23 at 11:58 +0100, Morten Rasmussen wrote: > > In the current find_idlest_group()/find_idlest_cpu() search we end up > > calling find_idlest_cpu() in a sched_group containing only one cpu in > > the end. Checking idle

Re: [PATCH 04/16] sched/fair: Optimize find_idlest_cpu() when there is no choice

2016-05-23 Thread Mike Galbraith
On Mon, 2016-05-23 at 11:58 +0100, Morten Rasmussen wrote: > In the current find_idlest_group()/find_idlest_cpu() search we end up > calling find_idlest_cpu() in a sched_group containing only one cpu in > the end. Checking idle-states becomes pointless when there is no > alternative, so bail out in