diff options
| author |   Hou Tao <houtao1@huawei.com> | 2022-09-01 14:19:36 +0800 |
|---|---|---|
| committer |   Martin KaFai Lau <martin.lau@kernel.org> | 2022-09-01 12:16:18 -0700 |
| commit | c89e843a11f1075d27684f6b42256213e4592383 (patch) | |
| tree | 22c31daa6100b62e1d048af31c9c813cdf072bf0 /tools/testing/selftests/bpf | |
| parent | bpf: Use this_cpu_{inc|dec|inc_return} for bpf_task_storage_busy (diff) | |
| download | linux-dev-c89e843a11f1075d27684f6b42256213e4592383.tar.xz linux-dev-c89e843a11f1075d27684f6b42256213e4592383.zip | |
bpf: Use this_cpu_{inc_return|dec} for prog->active
Both __this_cpu_inc_return() and __this_cpu_dec() are not preemption
safe and now migrate_disable() doesn't disable preemption, so the update
of prog-active is not atomic and in theory under fully preemptible kernel
recurisve prevention may do not work.
Fixing by using the preemption-safe and IRQ-safe variants.
Fixes: ca06f55b9002 ("bpf: Add per-program recursion prevention mechanism")
Signed-off-by: Hou Tao <houtao1@huawei.com>
Acked-by: Alexei Starovoitov <ast@kernel.org>
Link: https://lore.kernel.org/r/20220901061938.3789460-3-houtao@huaweicloud.com
Signed-off-by: Martin KaFai Lau <martin.lau@kernel.org>
Diffstat (limited to 'tools/testing/selftests/bpf')
0 files changed, 0 insertions, 0 deletions