Commit 73feb8d5 authored by Jiri Olsa's avatar Jiri Olsa Committed by Alexei Starovoitov
Browse files

kallsyms: Make module_kallsyms_on_each_symbol generally available 



Making module_kallsyms_on_each_symbol generally available, so it
can be used outside CONFIG_LIVEPATCH option in following changes.

Rather than adding another ifdef option let's make the function
generally available (when CONFIG_KALLSYMS and CONFIG_MODULES
options are defined).

Cc: Christoph Hellwig <hch@lst.de>
Acked-by: default avatarSong Liu <song@kernel.org>
Signed-off-by: default avatarJiri Olsa <jolsa@kernel.org>
Link: https://lore.kernel.org/r/20221025134148.3300700-2-jolsa@kernel.org


Signed-off-by: default avatarAlexei Starovoitov <ast@kernel.org>
parent 152e60e3

include/linux/module.h

+9 −0
Original line number Diff line number Diff line
@@ -879,8 +879,17 @@ static inline bool module_sig_ok(struct module *module) 
}

#endif	/* CONFIG_MODULE_SIG */



#if defined(CONFIG_MODULES) && defined(CONFIG_KALLSYMS)

int module_kallsyms_on_each_symbol(int (*fn)(void *, const char *,

					     struct module *, unsigned long),

				   void *data);

#else

static inline int module_kallsyms_on_each_symbol(int (*fn)(void *, const char *,

						 struct module *, unsigned long),

						 void *data)

{

	return -EOPNOTSUPP;

}

#endif  /* CONFIG_MODULES && CONFIG_KALLSYMS */



#endif /* _LINUX_MODULE_H */

kernel/module/kallsyms.c

+0 −2
Original line number Diff line number Diff line
@@ -494,7 +494,6 @@ unsigned long module_kallsyms_lookup_name(const char *name) 
	return ret;

}



#ifdef CONFIG_LIVEPATCH

int module_kallsyms_on_each_symbol(int (*fn)(void *, const char *,

					     struct module *, unsigned long),

				   void *data)
@@ -531,4 +530,3 @@ int module_kallsyms_on_each_symbol(int (*fn)(void *, const char *, 
	mutex_unlock(&module_mutex);

	return ret;

}
 
#endif /* CONFIG_LIVEPATCH */