diff options
| author | Zhang Hang <bob.zhanghang@huawei.com> | 2013-04-10 02:04:55 -0400 |
|---|---|---|
| committer | Ingo Molnar <mingo@kernel.org> | 2013-04-10 05:15:45 -0400 |
| commit | 4e2dcb73aecbde9fe4e3137c9ea35cb6aa6cb286 (patch) | |
| tree | d3011ead0af5e93ae0312ddf448dd6caae3a0cd3 | |
| parent | 28b4a521f618d9722bc780ea38b44718ce0fe283 (diff) | |
sched: Simplify can_migrate_task()
At this point tsk_cache_hot is always true, so no need to check it.
Signed-off-by: Zhang Hang <bob.zhanghang@huawei.com>
Cc: Peter Zijlstra <peterz@infradead.org>
Link: http://lkml.kernel.org/r/51650107.9040606@huawei.com
[ Also remove unnecessary schedstat #ifdefs. ]
Signed-off-by: Ingo Molnar <mingo@kernel.org>
| -rw-r--r-- | kernel/sched/fair.c | 11 |
1 files changed, 4 insertions, 7 deletions
diff --git a/kernel/sched/fair.c b/kernel/sched/fair.c index 539760ef00c4..bf8ab4f5e603 100644 --- a/kernel/sched/fair.c +++ b/kernel/sched/fair.c | |||
| @@ -3921,20 +3921,17 @@ int can_migrate_task(struct task_struct *p, struct lb_env *env) | |||
| 3921 | tsk_cache_hot = task_hot(p, env->src_rq->clock_task, env->sd); | 3921 | tsk_cache_hot = task_hot(p, env->src_rq->clock_task, env->sd); |
| 3922 | if (!tsk_cache_hot || | 3922 | if (!tsk_cache_hot || |
| 3923 | env->sd->nr_balance_failed > env->sd->cache_nice_tries) { | 3923 | env->sd->nr_balance_failed > env->sd->cache_nice_tries) { |
| 3924 | #ifdef CONFIG_SCHEDSTATS | 3924 | |
| 3925 | if (tsk_cache_hot) { | 3925 | if (tsk_cache_hot) { |
| 3926 | schedstat_inc(env->sd, lb_hot_gained[env->idle]); | 3926 | schedstat_inc(env->sd, lb_hot_gained[env->idle]); |
| 3927 | schedstat_inc(p, se.statistics.nr_forced_migrations); | 3927 | schedstat_inc(p, se.statistics.nr_forced_migrations); |
| 3928 | } | 3928 | } |
| 3929 | #endif | 3929 | |
| 3930 | return 1; | 3930 | return 1; |
| 3931 | } | 3931 | } |
| 3932 | 3932 | ||
| 3933 | if (tsk_cache_hot) { | 3933 | schedstat_inc(p, se.statistics.nr_failed_migrations_hot); |
| 3934 | schedstat_inc(p, se.statistics.nr_failed_migrations_hot); | 3934 | return 0; |
| 3935 | return 0; | ||
| 3936 | } | ||
| 3937 | return 1; | ||
| 3938 | } | 3935 | } |
| 3939 | 3936 | ||
| 3940 | /* | 3937 | /* |
