Commit a2cd4260 authored Mar 11, 2014 by Vincent Guittot Committed by Ingo Molnar Mar 12, 2014

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>

parent 383afd09

kernel/sched/fair.c

+2 −4

Original line number	Diff line number	Diff line
		@@ -6061,12 +6061,10 @@ void fix_small_imbalance(struct lb_env env, struct sd_lb_stats sds)
		pwr_now /= SCHED_POWER_SCALE;

		/* Amount of load we'd subtract */
		tmp = (busiest->load_per_task * SCHED_POWER_SCALE) /
		busiest->group_power;
		if (busiest->avg_load > tmp) {
		if (busiest->avg_load > scaled_busy_load_per_task) {
		pwr_move += busiest->group_power *
		min(busiest->load_per_task,
		busiest->avg_load - tmp);
		busiest->avg_load - scaled_busy_load_per_task);
		}

		/* Amount of load we'd add */

Admin message