diff options
author | Masahiro Yamada <masahiroy@kernel.org> | 2021-03-31 15:38:05 +0200 |
---|---|---|
committer | Masahiro Yamada <masahiroy@kernel.org> | 2021-04-24 22:22:42 +0200 |
commit | 1a998be620a10000c1e1240026e4bd6bc3378c96 (patch) | |
tree | 43b11c0ce0134689314cb40e03b361d0bfa730e5 /Makefile | |
parent | kbuild: show the target directory for depmod log (diff) | |
download | linux-1a998be620a10000c1e1240026e4bd6bc3378c96.tar.xz linux-1a998be620a10000c1e1240026e4bd6bc3378c96.zip |
kbuild: check module name conflict for external modules as well
If there are multiple modules with the same name in the same external
module tree, there is ambiguity about which one will be loaded, and
very likely something odd is happening.
Signed-off-by: Masahiro Yamada <masahiroy@kernel.org>
Diffstat (limited to 'Makefile')
-rw-r--r-- | Makefile | 10 |
1 files changed, 5 insertions, 5 deletions
@@ -1459,10 +1459,6 @@ endif PHONY += modules modules: $(if $(KBUILD_BUILTIN),vmlinux) modules_check modules_prepare -PHONY += modules_check -modules_check: modules.order - $(Q)$(CONFIG_SHELL) $(srctree)/scripts/modules-check.sh $< - cmd_modules_order = $(AWK) '!x[$$0]++' $(real-prereqs) > $@ modules.order: $(subdir-modorder) FORCE @@ -1775,9 +1771,13 @@ PHONY += modules modules_install ifdef CONFIG_MODULES -modules: $(MODORDER) +modules: modules_check $(Q)$(MAKE) -f $(srctree)/scripts/Makefile.modpost +PHONY += modules_check +modules_check: $(MODORDER) + $(Q)$(CONFIG_SHELL) $(srctree)/scripts/modules-check.sh $< + quiet_cmd_depmod = DEPMOD $(MODLIB) cmd_depmod = $(CONFIG_SHELL) $(srctree)/scripts/depmod.sh $(DEPMOD) \ $(KERNELRELEASE) |