Re: [PATCH] sched: fix: remove double calculation in fix_small_imbalance

2014-03-12 Thread Vincent Guittot
On 12 March 2014 10:00, Peter Zijlstra wrote: > On Tue, Mar 11, 2014 at 05:26:06PM +0100, Vincent Guittot wrote: >> The tmp value has been already calculated in: >> scaled_busy_load_per_task = >> (busiest->load_per_task * SCHED_POWER_SCALE) / >> busiest->group_power; >

Re: [PATCH] sched: fix: remove double calculation in fix_small_imbalance

2014-03-12 Thread Peter Zijlstra
On Tue, Mar 11, 2014 at 05:26:06PM +0100, Vincent Guittot wrote: > The tmp value has been already calculated in: > scaled_busy_load_per_task = > (busiest->load_per_task * SCHED_POWER_SCALE) / > busiest->group_power; You created a little confusion in my brain by saying i

[PATCH] sched: fix: remove double calculation in fix_small_imbalance

2014-03-11 Thread Vincent Guittot
The tmp value has been already calculated in: scaled_busy_load_per_task = (busiest->load_per_task * SCHED_POWER_SCALE) / busiest->group_power; Signed-off-by: Vincent Guittot --- kernel/sched/fair.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff