diff options
| author | Michael Wang <wangyun@linux.vnet.ibm.com> | 2012-07-03 02:34:02 -0400 |
|---|---|---|
| committer | Ingo Molnar <mingo@kernel.org> | 2012-09-04 08:31:42 -0400 |
| commit | 38b8dd6f87398524d02c21ff614c507ba8c9d295 (patch) | |
| tree | b814a4c20ac30c5d7e7c0836daf1b11be0923cb4 | |
| parent | d00535db42805e9ae5eadf1b4a86e01e85674b0c (diff) | |
sched: Remove useless code in yield_to()
It's impossible to enter the else branch if we have set
skip_clock_update in task_yield_fair(), as yield_to_task_fair()
will directly return true after invoke task_yield_fair().
Signed-off-by: Michael Wang <wangyun@linux.vnet.ibm.com>
Acked-by: Mike Galbraith <efault@gmx.de>
Signed-off-by: Peter Zijlstra <a.p.zijlstra@chello.nl>
Link: http://lkml.kernel.org/r/4FF2925A.9060005@linux.vnet.ibm.com
Signed-off-by: Ingo Molnar <mingo@kernel.org>
| -rw-r--r-- | kernel/sched/core.c | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/kernel/sched/core.c b/kernel/sched/core.c index ec0f2b81b81c..c46a011ce5db 100644 --- a/kernel/sched/core.c +++ b/kernel/sched/core.c | |||
| @@ -4348,13 +4348,6 @@ again: | |||
| 4348 | */ | 4348 | */ |
| 4349 | if (preempt && rq != p_rq) | 4349 | if (preempt && rq != p_rq) |
| 4350 | resched_task(p_rq->curr); | 4350 | resched_task(p_rq->curr); |
| 4351 | } else { | ||
| 4352 | /* | ||
| 4353 | * We might have set it in task_yield_fair(), but are | ||
| 4354 | * not going to schedule(), so don't want to skip | ||
| 4355 | * the next update. | ||
| 4356 | */ | ||
| 4357 | rq->skip_clock_update = 0; | ||
| 4358 | } | 4351 | } |
| 4359 | 4352 | ||
| 4360 | out: | 4353 | out: |
