The modules list can be accessed under RCU assumption.

Use RCU protection instead preempt_disable().

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

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;
 }
 
-- 
2.45.2


Reply via email to