diff options
author | Ingo Molnar <mingo@kernel.org> | 2023-10-06 13:25:18 +0300 |
---|---|---|
committer | Ingo Molnar <mingo@kernel.org> | 2023-10-09 13:21:23 +0300 |
commit | f4bb5705114530cd775a5a649b666755b3efe7aa (patch) | |
tree | 0c63fcda0feec0a08bae86c60ae4acbd5858f980 /kernel/sched/core.c | |
parent | b6dd6984832a2868f78879fce30d6965ae899d02 (diff) | |
download | linux-f4bb5705114530cd775a5a649b666755b3efe7aa.tar.xz |
sched/nohz: Remove unnecessarily complex error handling pattern from find_new_ilb()
find_new_ilb() returns nr_cpu_ids on failure - which is the usual
cpumask bitops return pattern, but is weird & unnecessary in this
context: not only is it a global variable, it it is a +1 out of
bounds CPU index and also has different signedness ...
Its only user, kick_ilb(), then checks the return against nr_cpu_ids
to decide to return. There's no other use.
So instead of this, use a standard -1 return on failure to find an
idle CPU, as the argument is signed already.
Signed-off-by: Ingo Molnar <mingo@kernel.org>
Reviewed-by: Joel Fernandes (Google) <joel@joelfernandes.org>
Link: https://lore.kernel.org/r/20231006102518.2452758-4-mingo@kernel.org
Diffstat (limited to 'kernel/sched/core.c')
0 files changed, 0 insertions, 0 deletions