diff options
author | Sebastian Andrzej Siewior <bigeasy@linutronix.de> | 2025-02-22 05:44:22 +0300 |
---|---|---|
committer | Alexei Starovoitov <ast@kernel.org> | 2025-02-27 20:29:33 +0300 |
commit | 0aaddfb06882504dded9cde57f91035ab9403b82 (patch) | |
tree | c827f7f55eba4f7f7b19cdcae10bc7db43f16133 /tools/testing/selftests/bpf/prog_tests/prog_array_init.c | |
parent | 2014c95afecee3e76ca4a56956a936e23283f05b (diff) | |
download | linux-0aaddfb06882504dded9cde57f91035ab9403b82.tar.xz |
locking/local_lock: Introduce localtry_lock_t
In !PREEMPT_RT local_lock_irqsave() disables interrupts to protect
critical section, but it doesn't prevent NMI, so the fully reentrant
code cannot use local_lock_irqsave() for exclusive access.
Introduce localtry_lock_t and localtry_lock_irqsave() that
disables interrupts and sets acquired=1, so localtry_lock_irqsave()
from NMI attempting to acquire the same lock will return false.
In PREEMPT_RT local_lock_irqsave() maps to preemptible spin_lock().
Map localtry_lock_irqsave() to preemptible spin_trylock().
When in hard IRQ or NMI return false right away, since
spin_trylock() is not safe due to explicit locking in the underneath
rt_spin_trylock() implementation. Removing this explicit locking and
attempting only "trylock" is undesired due to PI implications.
Note there is no need to use local_inc for acquired variable,
since it's a percpu variable with strict nesting scopes.
Acked-by: Davidlohr Bueso <dave@stgolabs.net>
Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
Signed-off-by: Vlastimil Babka <vbabka@suse.cz>
Link: https://lore.kernel.org/r/20250222024427.30294-2-alexei.starovoitov@gmail.com
Signed-off-by: Alexei Starovoitov <ast@kernel.org>
Diffstat (limited to 'tools/testing/selftests/bpf/prog_tests/prog_array_init.c')
0 files changed, 0 insertions, 0 deletions