Commit 1e684172 authored Mar 19, 2023 by Luis Chamberlain

module: add a for_each_modinfo_entry()



Add a for_each_modinfo_entry() to make it easier to read and use.
This produces no functional changes but makes this code easiert
to read as we are used to with loops in the kernel and trims more
lines of code.

Signed-off-by: Luis Chamberlain <mcgrof@kernel.org>

parent feb5b784

kernel/module/internal.h

+3 −0

Original line number	Diff line number	Diff line
		@@ -98,6 +98,9 @@ size_t module_flags_taint(unsigned long taints, char *buf);

		char module_next_tag_pair(char string, unsigned long *secsize);

		#define for_each_modinfo_entry(entry, info, name) \
		for (entry = get_modinfo(info, name); entry; entry = get_next_modinfo(info, name, entry))

		static inline void module_assert_mutex_or_preempt(void)
		{
		#ifdef CONFIG_LOCKDEP

kernel/module/main.c

+1 −4

Original line number	Diff line number	Diff line
		@@ -1075,12 +1075,9 @@ static int verify_namespace_is_imported(const struct load_info *info,

		namespace = kernel_symbol_namespace(sym);
		if (namespace && namespace[0]) {
		imported_namespace = get_modinfo(info, "import_ns");
		while (imported_namespace) {
		for_each_modinfo_entry(imported_namespace, info, "import_ns") {
		if (strcmp(namespace, imported_namespace) == 0)
		return 0;
		imported_namespace = get_next_modinfo(
		info, "import_ns", imported_namespace);
		}
		#ifdef CONFIG_MODULE_ALLOW_MISSING_NAMESPACE_IMPORTS
		pr_warn(