diff options
author | Vincent Guittot <vincent.guittot@linaro.org> | 2014-03-11 12:26:06 -0400 |
---|---|---|
committer | Ingo Molnar <mingo@kernel.org> | 2014-03-12 05:49:00 -0400 |
commit | a2cd42601b474b957e1a5fe3692bcf7f9363bd51 (patch) | |
tree | d551c6e8329b99ce163959f8b7281488da582c3a | |
parent | 383afd0971538b3d77532a56404b24cfe967b5dd (diff) |
sched: Remove double calculation in fix_small_imbalance()
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 <vincent.guittot@linaro.org>
Signed-off-by: Peter Zijlstra <peterz@infradead.org>
Link: http://lkml.kernel.org/r/1394555166-22894-1-git-send-email-vincent.guittot@linaro.org
Signed-off-by: Ingo Molnar <mingo@kernel.org>
-rw-r--r-- | kernel/sched/fair.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/kernel/sched/fair.c b/kernel/sched/fair.c index f1eedae1e83e..b301918ed510 100644 --- a/kernel/sched/fair.c +++ b/kernel/sched/fair.c | |||
@@ -6061,12 +6061,10 @@ void fix_small_imbalance(struct lb_env *env, struct sd_lb_stats *sds) | |||
6061 | pwr_now /= SCHED_POWER_SCALE; | 6061 | pwr_now /= SCHED_POWER_SCALE; |
6062 | 6062 | ||
6063 | /* Amount of load we'd subtract */ | 6063 | /* Amount of load we'd subtract */ |
6064 | tmp = (busiest->load_per_task * SCHED_POWER_SCALE) / | 6064 | if (busiest->avg_load > scaled_busy_load_per_task) { |
6065 | busiest->group_power; | ||
6066 | if (busiest->avg_load > tmp) { | ||
6067 | pwr_move += busiest->group_power * | 6065 | pwr_move += busiest->group_power * |
6068 | min(busiest->load_per_task, | 6066 | min(busiest->load_per_task, |
6069 | busiest->avg_load - tmp); | 6067 | busiest->avg_load - scaled_busy_load_per_task); |
6070 | } | 6068 | } |
6071 | 6069 | ||
6072 | /* Amount of load we'd add */ | 6070 | /* Amount of load we'd add */ |