diff options
author | Leo Yan <leo.yan@linaro.org> | 2015-09-15 06:56:45 -0400 |
---|---|---|
committer | Ingo Molnar <mingo@kernel.org> | 2015-09-18 03:23:16 -0400 |
commit | 79a89f92cbe31ba6bc50caf211a7ac4d97d0f35f (patch) | |
tree | 0a3cb38754e4f138f47e5867ec540590f67500c6 /kernel/sched/fair.c | |
parent | 84fb5a182d39221b89f205365386df243135d622 (diff) |
sched/fair: Remove unnecessary parameter for group_classify()
The group_classify() function does not use the "env" parameter, so remove it.
Also unify code to always use group_classify() to calculate group's
load type.
Signed-off-by: Leo Yan <leo.yan@linaro.org>
Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
Cc: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Thomas Gleixner <tglx@linutronix.de>
Link: http://lkml.kernel.org/r/1442314605-14838-1-git-send-email-leo.yan@linaro.org
Signed-off-by: Ingo Molnar <mingo@kernel.org>
Diffstat (limited to 'kernel/sched/fair.c')
-rw-r--r-- | kernel/sched/fair.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/kernel/sched/fair.c b/kernel/sched/fair.c index 80c62bf778a9..4df37a48f499 100644 --- a/kernel/sched/fair.c +++ b/kernel/sched/fair.c | |||
@@ -6273,9 +6273,9 @@ group_is_overloaded(struct lb_env *env, struct sg_lb_stats *sgs) | |||
6273 | return false; | 6273 | return false; |
6274 | } | 6274 | } |
6275 | 6275 | ||
6276 | static enum group_type group_classify(struct lb_env *env, | 6276 | static inline enum |
6277 | struct sched_group *group, | 6277 | group_type group_classify(struct sched_group *group, |
6278 | struct sg_lb_stats *sgs) | 6278 | struct sg_lb_stats *sgs) |
6279 | { | 6279 | { |
6280 | if (sgs->group_no_capacity) | 6280 | if (sgs->group_no_capacity) |
6281 | return group_overloaded; | 6281 | return group_overloaded; |
@@ -6340,7 +6340,7 @@ static inline void update_sg_lb_stats(struct lb_env *env, | |||
6340 | sgs->group_weight = group->group_weight; | 6340 | sgs->group_weight = group->group_weight; |
6341 | 6341 | ||
6342 | sgs->group_no_capacity = group_is_overloaded(env, sgs); | 6342 | sgs->group_no_capacity = group_is_overloaded(env, sgs); |
6343 | sgs->group_type = group_classify(env, group, sgs); | 6343 | sgs->group_type = group_classify(group, sgs); |
6344 | } | 6344 | } |
6345 | 6345 | ||
6346 | /** | 6346 | /** |
@@ -6474,7 +6474,7 @@ static inline void update_sd_lb_stats(struct lb_env *env, struct sd_lb_stats *sd | |||
6474 | group_has_capacity(env, &sds->local_stat) && | 6474 | group_has_capacity(env, &sds->local_stat) && |
6475 | (sgs->sum_nr_running > 1)) { | 6475 | (sgs->sum_nr_running > 1)) { |
6476 | sgs->group_no_capacity = 1; | 6476 | sgs->group_no_capacity = 1; |
6477 | sgs->group_type = group_overloaded; | 6477 | sgs->group_type = group_classify(sg, sgs); |
6478 | } | 6478 | } |
6479 | 6479 | ||
6480 | if (update_sd_pick_busiest(env, sds, sg, sgs)) { | 6480 | if (update_sd_pick_busiest(env, sds, sg, sgs)) { |