sched: fix: move the CPU check into ->task_new_fair()

noticed by Peter Zijlstra:

fix: move the CPU check into ->task_new_fair(), this way we
can call place_entity() and get child ->vruntime right at
initial wakeup time.

(without this there can be large latencies)

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

1 files changed, 1 insertions, 4 deletions

diff --git a/kernel/sched.c b/kernel/sched.c
index f2b8db4d6802..b41ef663b993 100644
--- a/kernel/sched.c
+++ b/kernel/sched.c
@@ -1660,17 +1660,14 @@ void fastcall wake_up_new_task(struct task_struct *p, unsigned long clone_flags)
 {
         unsigned long flags;
         struct rq *rq;
-        int this_cpu;
 
         rq = task_rq_lock(p, &flags);
         BUG_ON(p->state != TASK_RUNNING);
-        this_cpu = smp_processor_id(); /* parent's CPU */
         update_rq_clock(rq);
 
         p->prio = effective_prio(p);
 
-        if (task_cpu(p) != this_cpu || !p->sched_class->task_new ||
-                                                        !current->se.on_rq) {
+        if (!p->sched_class->task_new || !current->se.on_rq || !rq->cfs.curr) {
                 activate_task(rq, p, 0);
         } else {
                 /*