]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
tracing: Replace MAX_PARAM_PREFIX_LEN with MODULE_NAME_LEN
authorPetr Pavlu <petr.pavlu@suse.com>
Mon, 30 Jun 2025 14:32:35 +0000 (16:32 +0200)
committerDaniel Gomez <da.gomez@samsung.com>
Thu, 31 Jul 2025 11:57:44 +0000 (13:57 +0200)
Use the MODULE_NAME_LEN definition in module_exists() to obtain the maximum
size of a module name, instead of using MAX_PARAM_PREFIX_LEN. The values
are the same but MODULE_NAME_LEN is more appropriate in this context.
MAX_PARAM_PREFIX_LEN was added in commit 730b69d22525 ("module: check
kernel param length at compile time, not runtime") only to break a circular
dependency between module.h and moduleparam.h, and should mostly be limited
to use in moduleparam.h.

Signed-off-by: Petr Pavlu <petr.pavlu@suse.com>
Cc: Steven Rostedt <rostedt@goodmis.org>
Cc: Masami Hiramatsu <mhiramat@kernel.org>
Reviewed-by: Daniel Gomez <da.gomez@samsung.com>
Acked-by: Steven Rostedt (Google) <rostedt@goodmis.org>
Link: https://lore.kernel.org/r/20250630143535.267745-5-petr.pavlu@suse.com
Signed-off-by: Daniel Gomez <da.gomez@samsung.com>
kernel/trace/trace.c

index 7996f26c3f46d2844f7ed443b8dfaeefb5ea8e81..3112ac128145355231cc0b7cae87cbee34f1881d 100644 (file)
@@ -10367,7 +10367,7 @@ bool module_exists(const char *module)
 {
        /* All modules have the symbol __this_module */
        static const char this_mod[] = "__this_module";
-       char modname[MAX_PARAM_PREFIX_LEN + sizeof(this_mod) + 2];
+       char modname[MODULE_NAME_LEN + sizeof(this_mod) + 2];
        unsigned long val;
        int n;