Re: [PATCH v2 2/5] sched/fair: Remove unnecessary comparison with -1

2017-08-25 Thread Vincent Guittot
On 25 August 2017 at 12:16, Brendan Jackman wrote: > Since commit 83a0a96a5f26 ("sched/fair: Leverage the idle state info > when choosing the "idlest" cpu") find_idlest_grou_cpu (formerly > find_idlest_cpu) no longer returns -1. > > Signed-off-by: Brendan Jackman

Re: [PATCH v2 2/5] sched/fair: Remove unnecessary comparison with -1

2017-08-25 Thread Vincent Guittot
On 25 August 2017 at 12:16, Brendan Jackman wrote: > Since commit 83a0a96a5f26 ("sched/fair: Leverage the idle state info > when choosing the "idlest" cpu") find_idlest_grou_cpu (formerly > find_idlest_cpu) no longer returns -1. > > Signed-off-by: Brendan Jackman > Cc: Dietmar Eggemann > Cc:

Re: [PATCH v2 2/5] sched/fair: Remove unnecessary comparison with -1

2017-08-25 Thread Josef Bacik
On Fri, Aug 25, 2017 at 11:16:29AM +0100, Brendan Jackman wrote: > Since commit 83a0a96a5f26 ("sched/fair: Leverage the idle state info > when choosing the "idlest" cpu") find_idlest_grou_cpu (formerly > find_idlest_cpu) no longer returns -1. > Reviewed-by: Josef Bacik Thanks,

Re: [PATCH v2 2/5] sched/fair: Remove unnecessary comparison with -1

2017-08-25 Thread Josef Bacik
On Fri, Aug 25, 2017 at 11:16:29AM +0100, Brendan Jackman wrote: > Since commit 83a0a96a5f26 ("sched/fair: Leverage the idle state info > when choosing the "idlest" cpu") find_idlest_grou_cpu (formerly > find_idlest_cpu) no longer returns -1. > Reviewed-by: Josef Bacik Thanks, Josef

[PATCH v2 2/5] sched/fair: Remove unnecessary comparison with -1

2017-08-25 Thread Brendan Jackman
Since commit 83a0a96a5f26 ("sched/fair: Leverage the idle state info when choosing the "idlest" cpu") find_idlest_grou_cpu (formerly find_idlest_cpu) no longer returns -1. Signed-off-by: Brendan Jackman Cc: Dietmar Eggemann Cc: Vincent Guittot

[PATCH v2 2/5] sched/fair: Remove unnecessary comparison with -1

2017-08-25 Thread Brendan Jackman
Since commit 83a0a96a5f26 ("sched/fair: Leverage the idle state info when choosing the "idlest" cpu") find_idlest_grou_cpu (formerly find_idlest_cpu) no longer returns -1. Signed-off-by: Brendan Jackman Cc: Dietmar Eggemann Cc: Vincent Guittot Cc: Josef Bacik Cc: Ingo Molnar Cc: Morten