diff options
author | Ingo Molnar <mingo@elte.hu> | 2007-11-09 16:39:39 -0500 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2007-11-09 16:39:39 -0500 |
commit | 8bc6767acb3236e0345e99cf198168e60e7ae456 (patch) | |
tree | be9119dfaf3751ca49536bd569a0fc617daabda3 /kernel | |
parent | 3e3e13f399ac8060a20d14d210a28dc02dda372e (diff) |
sched: wakeup preemption fix
wakeup preemption fix: do not make it dependent on p->prio.
Preemption purely depends on ->vruntime.
This improves preemption in mixed-nice-level workloads.
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'kernel')
-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 fbcb426029d0..a3badf52bba2 100644 --- a/kernel/sched_fair.c +++ b/kernel/sched_fair.c | |||
@@ -863,10 +863,8 @@ static void check_preempt_wakeup(struct rq *rq, struct task_struct *p) | |||
863 | if (unlikely(se->load.weight != NICE_0_LOAD)) | 863 | if (unlikely(se->load.weight != NICE_0_LOAD)) |
864 | gran = calc_delta_fair(gran, &se->load); | 864 | gran = calc_delta_fair(gran, &se->load); |
865 | 865 | ||
866 | if (delta > gran) { | 866 | if (delta > gran) |
867 | if (p->prio < curr->prio) | 867 | resched_task(curr); |
868 | resched_task(curr); | ||
869 | } | ||
870 | } | 868 | } |
871 | } | 869 | } |
872 | 870 | ||