diff options
author | Peter Zijlstra <a.p.zijlstra@chello.nl> | 2007-08-24 14:39:10 -0400 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2007-08-24 14:39:10 -0400 |
commit | a6f2994042cc2db9e507dc702ed0b5e2cc5890fe (patch) | |
tree | 2fcfab62367e086e50f3561a7dbcae6a63ef952b | |
parent | b2133c8b1e270b4a7c36f70e29be8738d09e850b (diff) |
sched: simplify bonus calculation #1
current code:
delta = min(cfs_rq->sleeper_bonus, (u64)delta_exec);
delta = calc_delta_mine(delta, curr->load.weight, lw);
delta = min((u64)delta, cfs_rq->sleeper_bonus);
drop the first min(), because we clip against sleeper_bonus in the 3rd line
again. That gives:
delta = calc_delta_mine(delta_exec, curr->load.weight, lw);
delta = min((u64)delta, cfs_rq->sleeper_bonus);
Signed-off-by: Peter Zijlstra <a.p.zijlstra@chello.nl>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
-rw-r--r-- | kernel/sched_fair.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/kernel/sched_fair.c b/kernel/sched_fair.c index c578370cd693..5b2d97fcd80c 100644 --- a/kernel/sched_fair.c +++ b/kernel/sched_fair.c | |||
@@ -303,8 +303,7 @@ __update_curr(struct cfs_rq *cfs_rq, struct sched_entity *curr) | |||
303 | delta_mine = calc_delta_mine(delta_exec, curr->load.weight, lw); | 303 | delta_mine = calc_delta_mine(delta_exec, curr->load.weight, lw); |
304 | 304 | ||
305 | if (cfs_rq->sleeper_bonus > sysctl_sched_granularity) { | 305 | if (cfs_rq->sleeper_bonus > sysctl_sched_granularity) { |
306 | delta = min(cfs_rq->sleeper_bonus, (u64)delta_exec); | 306 | delta = calc_delta_mine(delta_exec, curr->load.weight, lw); |
307 | delta = calc_delta_mine(delta, curr->load.weight, lw); | ||
308 | delta = min((u64)delta, cfs_rq->sleeper_bonus); | 307 | delta = min((u64)delta, cfs_rq->sleeper_bonus); |
309 | delta = min(delta, (unsigned long)( | 308 | delta = min(delta, (unsigned long)( |
310 | (long)sysctl_sched_runtime_limit - curr->wait_runtime)); | 309 | (long)sysctl_sched_runtime_limit - curr->wait_runtime)); |