diff options
author | Rik van Riel <riel@surriel.com> | 2018-09-25 23:58:40 -0400 |
---|---|---|
committer | Peter Zijlstra <peterz@infradead.org> | 2018-10-09 10:51:11 -0400 |
commit | c3f7f2c7eba1a53d2e5ffbc2dcc9a20c5f094890 (patch) | |
tree | 5d44c63073ac248c29fa7708f7dee7bfe4ba8ba0 | |
parent | 12c4d978fd170ccdd7260ec11f93b11e46904228 (diff) |
smp: use __cpumask_set_cpu in on_each_cpu_cond
The code in on_each_cpu_cond sets CPUs in a locally allocated bitmask,
which should never be used by other CPUs simultaneously. There is no
need to use locked memory accesses to set the bits in this bitmap.
Switch to __cpumask_set_cpu.
Cc: npiggin@gmail.com
Cc: mingo@kernel.org
Cc: will.deacon@arm.com
Cc: songliubraving@fb.com
Cc: kernel-team@fb.com
Cc: hpa@zytor.com
Suggested-by: Peter Zijlstra <peterz@infradead.org>
Signed-off-by: Rik van Riel <riel@surriel.com>
Reviewed-by: Andy Lutomirski <luto@kernel.org>
Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
Link: http://lkml.kernel.org/r/20180926035844.1420-4-riel@surriel.com
-rw-r--r-- | kernel/smp.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/kernel/smp.c b/kernel/smp.c index d86eec5f51c1..a7d4f9f50a49 100644 --- a/kernel/smp.c +++ b/kernel/smp.c | |||
@@ -682,7 +682,7 @@ void on_each_cpu_cond(bool (*cond_func)(int cpu, void *info), | |||
682 | preempt_disable(); | 682 | preempt_disable(); |
683 | for_each_online_cpu(cpu) | 683 | for_each_online_cpu(cpu) |
684 | if (cond_func(cpu, info)) | 684 | if (cond_func(cpu, info)) |
685 | cpumask_set_cpu(cpu, cpus); | 685 | __cpumask_set_cpu(cpu, cpus); |
686 | on_each_cpu_mask(cpus, func, info, wait); | 686 | on_each_cpu_mask(cpus, func, info, wait); |
687 | preempt_enable(); | 687 | preempt_enable(); |
688 | free_cpumask_var(cpus); | 688 | free_cpumask_var(cpus); |