diff mbox series

[v2,11/28] module: Use RCU in __is_module_percpu_address().

Message ID	20241220174731.514432-12-bigeasy@linutronix.de (mailing list archive)
State	New
Headers	show Received: from galois.linutronix.de (Galois.linutronix.de [193.142.43.55]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id E0DFA225A36; Fri, 20 Dec 2024 17:47:42 +0000 (UTC) From: Sebastian Andrzej Siewior <bigeasy@linutronix.de> To: linux-modules@vger.kernel.org, linux-kernel@vger.kernel.org Cc: Daniel Gomez <da.gomez@samsung.com>, Luis Chamberlain <mcgrof@kernel.org>, "Paul E . McKenney" <paulmck@kernel.org>, Peter Zijlstra <peterz@infradead.org>, Petr Pavlu <petr.pavlu@suse.com>, Sami Tolvanen <samitolvanen@google.com>, Thomas Gleixner <tglx@linutronix.de>, Sebastian Andrzej Siewior <bigeasy@linutronix.de> Subject: [PATCH v2 11/28] module: Use RCU in __is_module_percpu_address(). Date: Fri, 20 Dec 2024 18:41:25 +0100 Message-ID: <20241220174731.514432-12-bigeasy@linutronix.de> In-Reply-To: <20241220174731.514432-1-bigeasy@linutronix.de> References: <20241220174731.514432-1-bigeasy@linutronix.de> Precedence: bulk MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable
Series	module: Use RCU instead of RCU-sched. \| expand [v2,00/28] module: Use RCU instead of RCU-sched. [v2,01/28] module: Extend the preempt disabled section in dereference_symbol_descriptor(). [v2,02/28] module: Begin to move from RCU-sched to RCU. [v2,03/28] module: Use proper RCU assignment in add_kallsyms(). [v2,04/28] module: Use RCU in find_kallsyms_symbol(). [v2,05/28] module: Use RCU in module_get_kallsym(). [v2,06/28] module: Use RCU in find_module_all(). [v2,07/28] module: Use RCU in __find_kallsyms_symbol_value(). [v2,08/28] module: Use RCU in module_kallsyms_on_each_symbol(). [v2,09/28] module: Remove module_assert_mutex_or_preempt() from try_add_tainted_module(). [v2,10/28] module: Use RCU in find_symbol(). [v2,11/28] module: Use RCU in __is_module_percpu_address(). [v2,12/28] module: Allow __module_address() to be called from RCU section. [v2,13/28] module: Use RCU in search_module_extables(). [v2,14/28] module: Use RCU in all users of __module_address(). [v2,15/28] module: Use RCU in all users of __module_text_address(). [v2,16/28] ARM: module: Use RCU in all users of __module_text_address(). [v2,17/28] arm64: module: Use RCU in all users of __module_text_address(). [v2,18/28] LoongArch/orc: Use RCU in all users of __module_address(). [v2,19/28] LoongArch: ftrace: Use RCU in all users of __module_text_address(). [v2,20/28] powerpc/ftrace: Use RCU in all users of __module_text_address(). [v2,21/28] x86: Use RCU in all users of __module_address(). [v2,22/28] jump_label: Use RCU in all users of __module_address(). [v2,23/28] jump_label: Use RCU in all users of __module_text_address(). [v2,24/28] bpf: Use RCU in all users of __module_text_address(). [v2,25/28] kprobes: Use RCU in all users of __module_text_address(). [v2,26/28] static_call: Use RCU in all users of __module_text_address(). [v2,27/28] bug: Use RCU instead RCU-sched to protect module_bug_list. [v2,28/28] cfi: Use RCU while invoking __module_address().

Commit Message

Sebastian Andrzej Siewior Dec. 20, 2024, 5:41 p.m. UTC

The modules list can be accessed under RCU assumption.

Use RCU protection instead preempt_disable().

Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
---
 kernel/module/main.c | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

diff mbox series

Patch

diff --git a/kernel/module/main.c b/kernel/module/main.c
index 71e73deed76c0..126f7f05dedf8 100644
--- a/kernel/module/main.c
+++ b/kernel/module/main.c
@@ -450,8 +450,7 @@  bool __is_module_percpu_address(unsigned long addr, unsigned long *can_addr)
 	struct module *mod;
 	unsigned int cpu;
 
-	preempt_disable();
-
+	guard(rcu)();
 	list_for_each_entry_rcu(mod, &modules, list) {
 		if (mod->state == MODULE_STATE_UNFORMED)
 			continue;
@@ -468,13 +467,10 @@  bool __is_module_percpu_address(unsigned long addr, unsigned long *can_addr)
 						per_cpu_ptr(mod->percpu,
 							    get_boot_cpu_id());
 				}
-				preempt_enable();
 				return true;
 			}
 		}
 	}
-
-	preempt_enable();
 	return false;
 }