Messages in this thread Patch in this message | | | From | Morten Rasmussen <> | Subject | [PATCHv3 6/9] sched: Change root_domain->overload type to int | Date | Wed, 20 Jun 2018 10:05:46 +0100 |
| |
From: Valentin Schneider <valentin.schneider@arm.com>
sizeof(_Bool) is implementation defined, so let's just go with 'int' as is done for other structures e.g. sched_domain_shared->has_idle_cores.
cc: Ingo Molnar <mingo@redhat.com> cc: Peter Zijlstra <peterz@infradead.org>
Signed-off-by: Valentin Schneider <valentin.schneider@arm.com> Signed-off-by: Morten Rasmussen <morten.rasmussen@arm.com> --- kernel/sched/fair.c | 7 +++---- kernel/sched/sched.h | 4 ++-- 2 files changed, 5 insertions(+), 6 deletions(-)
diff --git a/kernel/sched/fair.c b/kernel/sched/fair.c index b29d53d1e22a..545fa3600894 100644 --- a/kernel/sched/fair.c +++ b/kernel/sched/fair.c @@ -8170,7 +8170,7 @@ static bool update_nohz_stats(struct rq *rq, bool force) static inline void update_sg_lb_stats(struct lb_env *env, struct sched_group *group, int load_idx, int local_group, struct sg_lb_stats *sgs, - bool *overload) + int *overload) { unsigned long load; int i, nr_running; @@ -8195,7 +8195,7 @@ static inline void update_sg_lb_stats(struct lb_env *env, nr_running = rq->nr_running; if (nr_running > 1) - *overload = true; + *overload = 1; #ifdef CONFIG_NUMA_BALANCING sgs->nr_numa_running += rq->nr_numa_running; @@ -8354,8 +8354,7 @@ static inline void update_sd_lb_stats(struct lb_env *env, struct sd_lb_stats *sd struct sched_group *sg = env->sd->groups; struct sg_lb_stats *local = &sds->local_stat; struct sg_lb_stats tmp_sgs; - int load_idx, prefer_sibling = 0; - bool overload = false; + int load_idx, prefer_sibling, overload = 0; if (child && child->flags & SD_PREFER_SIBLING) prefer_sibling = 1; diff --git a/kernel/sched/sched.h b/kernel/sched/sched.h index 5ed67122cf59..f945edadd2c5 100644 --- a/kernel/sched/sched.h +++ b/kernel/sched/sched.h @@ -689,7 +689,7 @@ struct root_domain { cpumask_var_t online; /* Indicate more than one runnable task for any CPU */ - bool overload; + int overload; /* * The bit corresponding to a CPU gets set here if such CPU has more @@ -1666,7 +1666,7 @@ static inline void add_nr_running(struct rq *rq, unsigned count) if (prev_nr < 2 && rq->nr_running >= 2) { #ifdef CONFIG_SMP if (!rq->rd->overload) - rq->rd->overload = true; + rq->rd->overload = 1; #endif } -- 2.7.4
| |