Commit cdd2ab3d authored Sep 07, 2009 by Peter Zijlstra Committed by Ingo Molnar Sep 07, 2009

sched: Remove short cut from select_task_rq_fair()



select_task_rq_fair() incorrectly skips the wake_affine()
logic, remove this.

When prev_cpu == this_cpu, the code jumps straight to the
wake_idle() logic, this doesn't give the wake_affine() logic
the chance to pin the task to this cpu.

Signed-off-by: Peter Zijlstra <a.p.zijlstra@chello.nl>
LKML-Reference: <new-submission>
Signed-off-by: Ingo Molnar <mingo@elte.hu>

parent 840a0653

kernel/sched_fair.c

+0 −2

Original line number	Diff line number	Diff line
		@@ -1305,8 +1305,6 @@ static int select_task_rq_fair(struct task_struct *p, int sync)
		this_rq = cpu_rq(this_cpu);
		new_cpu = prev_cpu;

		if (prev_cpu == this_cpu)
		goto out;
		/*
		* 'this_sd' is the first domain that both
		* this_cpu and prev_cpu are present in: