diff options
author | Ingo Molnar <mingo@elte.hu> | 2007-11-09 22:39:39 +0100 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2007-11-09 22:39:39 +0100 |
commit | 502d26b524d8980f3ed80d9aec398e85671a8160 (patch) | |
tree | 7a377cb8a688dbfe98c9d0b00ed957c2df816ef4 | |
parent | 77d9cc44b543fa831169e54c495ad06ef3a0c726 (diff) | |
download | linux-stable-502d26b524d8980f3ed80d9aec398e85671a8160.tar.gz linux-stable-502d26b524d8980f3ed80d9aec398e85671a8160.tar.bz2 linux-stable-502d26b524d8980f3ed80d9aec398e85671a8160.zip |
sched: clean up the wakeup preempt check, #2
clean up the preemption check to not use unnecessary 64-bit
variables. This improves code size:
text data bss dec hex filename
44227 3326 36 47589 b9e5 sched.o.before
44201 3326 36 47563 b9cb sched.o.after
Signed-off-by: Ingo Molnar <mingo@elte.hu>
-rw-r--r-- | kernel/sched_fair.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/kernel/sched_fair.c b/kernel/sched_fair.c index d558716a9add..6c361472cc74 100644 --- a/kernel/sched_fair.c +++ b/kernel/sched_fair.c @@ -837,7 +837,7 @@ static void check_preempt_wakeup(struct rq *rq, struct task_struct *p) struct task_struct *curr = rq->curr; struct cfs_rq *cfs_rq = task_cfs_rq(curr); struct sched_entity *se = &curr->se, *pse = &p->se; - s64 delta, gran; + unsigned long gran; if (unlikely(rt_prio(p->prio))) { update_rq_clock(rq); @@ -860,12 +860,11 @@ static void check_preempt_wakeup(struct rq *rq, struct task_struct *p) pse = parent_entity(pse); } - delta = se->vruntime - pse->vruntime; gran = sysctl_sched_wakeup_granularity; if (unlikely(se->load.weight != NICE_0_LOAD)) gran = calc_delta_fair(gran, &se->load); - if (delta > gran) + if (pse->vruntime + gran < se->vruntime) resched_task(curr); } |