diff options
| author | Masahiro Yamada <masahiroy@kernel.org> | 2021-03-31 22:38:09 +0900 |
|---|---|---|
| committer | Masahiro Yamada <masahiroy@kernel.org> | 2021-04-25 05:23:57 +0900 |
| commit | 961ab4a3cd66c285951cf4c8ec10bc8d9a4b0232 (patch) | |
| tree | 1b15c4aa877e5d7976660b4618a2b7aa113e7b03 /scripts/Makefile.modinst | |
| parent | 65ce9c38326e2588fcd1a3a4817c14b4660f430b (diff) | |
| download | lwn-961ab4a3cd66c285951cf4c8ec10bc8d9a4b0232.tar.gz lwn-961ab4a3cd66c285951cf4c8ec10bc8d9a4b0232.zip | |
kbuild: merge scripts/Makefile.modsign to scripts/Makefile.modinst
scripts/Makefile.modsign is a subset of scripts/Makefile.modinst,
and duplicates the code. Let's merge them.
By the way, you do not need to run 'make modules_sign' explicitly
because modules are signed as a part of 'make modules_install' when
CONFIG_MODULE_SIG_ALL=y. If CONFIG_MODULE_SIG_ALL=n, mod_sign_cmd is
set to 'true', so 'make modules_sign' is not functional.
In my understanding, the reason of still keeping this is to handle
corner cases like commit 64178cb62c32 ("builddeb: fix stripped module
signatures if CONFIG_DEBUG_INFO and CONFIG_MODULE_SIG_ALL are set").
Signed-off-by: Masahiro Yamada <masahiroy@kernel.org>
Diffstat (limited to 'scripts/Makefile.modinst')
| -rw-r--r-- | scripts/Makefile.modinst | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/scripts/Makefile.modinst b/scripts/Makefile.modinst index 84696ef99df7..191408f7a91a 100644 --- a/scripts/Makefile.modinst +++ b/scripts/Makefile.modinst @@ -74,11 +74,20 @@ quiet_cmd_sign := cmd_sign := : endif +ifeq ($(modules_sign_only),) + $(dst)/%.ko: $(extmod_prefix)%.ko FORCE $(call cmd,install) $(call cmd,strip) $(call cmd,sign) +else + +$(dst)/%.ko: FORCE + $(call cmd,sign) + +endif + # # Compression # |