diff options
| author | Zqiang <qiang.zhang@linux.dev> | 2026-04-30 16:45:43 +0800 |
|---|---|---|
| committer | Tejun Heo <tj@kernel.org> | 2026-05-04 11:00:31 -1000 |
| commit | d757ac2ee7bfda90c64b60a6593a2139a06f79b2 (patch) | |
| tree | 8e72fdaf0e028796968bc33c0564d4306fd651f4 /kernel/sched | |
| parent | 97f86c38abe62c911ff20bc3e00b0937842f79c0 (diff) | |
| download | lwn-d757ac2ee7bfda90c64b60a6593a2139a06f79b2.tar.gz lwn-d757ac2ee7bfda90c64b60a6593a2139a06f79b2.zip | |
sched_ext: Remove redundant rcu_read_lock/unlock() in sysrq_handle_sched_ext_reset()
sysrq_handle_sched_ext_reset() is called from __handle_sysrq(), which
already holds rcu_read_lock() while invoking the sysrq handler. Remove
the redundant rcu_read_lock/unlock() pair.
Signed-off-by: Zqiang <qiang.zhang@linux.dev>
Reviewed-by: Cheng-Yang Chou <yphbchou0911@gmail.com>
Reviewed-by: Andrea Righi <arighi@nvidia.com>
Signed-off-by: Tejun Heo <tj@kernel.org>
Diffstat (limited to 'kernel/sched')
| -rw-r--r-- | kernel/sched/ext.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/kernel/sched/ext.c b/kernel/sched/ext.c index f3a585e32db3..4287654c746f 100644 --- a/kernel/sched/ext.c +++ b/kernel/sched/ext.c @@ -7914,13 +7914,11 @@ static void sysrq_handle_sched_ext_reset(u8 key) { struct scx_sched *sch; - rcu_read_lock(); sch = rcu_dereference(scx_root); if (likely(sch)) scx_disable(sch, SCX_EXIT_SYSRQ); else pr_info("sched_ext: BPF schedulers not loaded\n"); - rcu_read_unlock(); } static const struct sysrq_key_op sysrq_sched_ext_reset_op = { |