Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Mon, 19 Jun 2023 14:34:19 +0200
From:   Nicolas Schier <nicolas@fjasle.eu>
To:     Masahiro Yamada <masahiroy@kernel.org>
Cc:     linux-kbuild@vger.kernel.org, linux-kernel@vger.kernel.org,
        Nathan Chancellor <nathan@kernel.org>,
        Nick Desaulniers <ndesaulniers@google.com>,
        Ben Hutchings <ben@decadent.org.uk>,
        linux-snps-arc@lists.infradead.org,
        Josh Triplett <josh@joshtriplett.org>
Subject: Re: [PATCH] kbuild: make modules_install copy
 modules.builtin(.modinfo)
Message-ID: <ZJBLS+Dzcj5wOJTb@fjasle.eu>
References: <20230615111743.883891-1-masahiroy@kernel.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <20230615111743.883891-1-masahiroy@kernel.org>
UI-OutboundReport: notjunk:1;M01:P0:fJLWB0XZUWo=;aBsZVDORVX7MHFJqgDDY2IeZC6l
 7U5O9PgErYt95oFdXHkjxE76yzbCCHXlhYpjaVDmtfJD1qfjOiYpKJ+E/x0wta5sygmTMpgWH
 mcUg1uBgRgWNpq6x/5gubCm17l55pVaLgTwJV0+yb6e7t5/yxJ8/csXxr82Z0h1m68MsueBdo
 Z6szlkTkiZjkd/cX1VMgiBeQJgdXVrAMDN+SVXDeeT53uVypLRiVxrvVzC369e3aljQ8UeQeo
 PKtRkPX5jSZN8spyLFUTjBewuQoUzc3zzEcvHdJBcmfnaK15a6rjOurMVBixpSD/utXrTzyzD
 MQ8UJVLv6eNyHApP6nFMERc+6j7BgrZzgyYBBSrigImt6dJuvc9KiN4pHtckK53DZmFEXegd6
 IUIZYj8Wqke9JZdKK8U8bttdW5HxpgPRNqHIUh6MdJebPJGnfD8a2CdPyLmKVbjKNX5PjxX1T
 bte1ep+Ui8NLqGIPEg2HIoxI1TFJjKE0ZY9+byakuD0hyrzhkGaK3ra5GpgzzNCs6SqH6VvR5
 tWIKVcSQarao0ZggzHgsKs+kU5J6GwE1j8guuve5xtBo7zBx2TjbpljMRLaPOikl2vaSWw5pR
 UIYZYmKgGdIuMY2H3r6kE7ojLSZC7yc5efeqa1uS5fu6pkw8y0efKI7pKs3h99ZOJMGcpSNx+
 EtG4zDsy9rRmCotKCbChGpGSu7b8Pes5+ukVO/elew==
Precedence: bulk

On Thu, Jun 15, 2023 at 08:17:43PM +0900 Masahiro Yamada wrote:
> Josh Triplett reports that initramfs-tools needs modules.builtin and
> modules.builtin.modinfo to create a working initramfs for a non-modular
> kernel.
> 
> If this is a general tooling issue not limited to Debian, I think it
> makes sense to change modules_install.
> 
> This commit changes the targets as follows when CONFIG_MODULES=n.
> 
> In-tree builds:
>   make modules          -> no-op
>   make modules_install  -> install modules.builtin(.modinfo)
> 
> External module builds:
>   make modules          -> show error message like before
>   make modules_install  -> show error message like before
> 
> Link: https://lore.kernel.org/lkml/36a4014c73a52af27d930d3ca31d362b60f4461c.1686356364.git.josh@joshtriplett.org/
> Reported-by: Josh Triplett <josh@joshtriplett.org>
> Signed-off-by: Masahiro Yamada <masahiroy@kernel.org>
> ---

Reviewed-by: Nicolas Schier <nicolas@fjasle.eu>
Tested-by: Nicolas Schier <nicolas@fjasle.eu>

>  Makefile | 26 ++++++++++++++++----------
>  1 file changed, 16 insertions(+), 10 deletions(-)
> 
> diff --git a/Makefile b/Makefile
> index cc3fe09c4dec..f18d59c81241 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -1545,6 +1545,8 @@ modules_sign_only := y
>  endif
>  endif
>  
> +endif # CONFIG_MODULES
> +
>  modinst_pre :=
>  ifneq ($(filter modules_install,$(MAKECMDGOALS)),)
>  modinst_pre := __modinst_pre
> @@ -1555,18 +1557,18 @@ PHONY += __modinst_pre
>  __modinst_pre:
>  	@rm -rf $(MODLIB)/kernel
>  	@rm -f $(MODLIB)/source
> -	@mkdir -p $(MODLIB)/kernel
> +	@mkdir -p $(MODLIB)
> +ifdef CONFIG_MODULES
>  	@ln -s $(abspath $(srctree)) $(MODLIB)/source
>  	@if [ ! $(objtree) -ef  $(MODLIB)/build ]; then \
>  		rm -f $(MODLIB)/build ; \
>  		ln -s $(CURDIR) $(MODLIB)/build ; \
>  	fi
>  	@sed 's:^\(.*\)\.o$$:kernel/\1.ko:' modules.order > $(MODLIB)/modules.order
> +endif
>  	@cp -f modules.builtin $(MODLIB)/
>  	@cp -f $(objtree)/modules.builtin.modinfo $(MODLIB)/
>  
> -endif # CONFIG_MODULES
> -
>  ###
>  # Cleaning is done on three levels.
>  # make clean     Delete most generated files
> @@ -1908,6 +1910,13 @@ help:
>  	@echo  '  clean           - remove generated files in module directory only'
>  	@echo  ''
>  
> +__external_modules_error:
> +	@echo >&2 '***'
> +	@echo >&2 '*** The present kernel disabled CONFIG_MODULES.'
> +	@echo >&2 '*** You cannot build or install external modules.'
> +	@echo >&2 '***'
> +	@false
> +
>  endif # KBUILD_EXTMOD
>  
>  # ---------------------------------------------------------------------------
> @@ -1944,13 +1953,10 @@ else # CONFIG_MODULES
>  # Modules not configured
>  # ---------------------------------------------------------------------------
>  
> -modules modules_install:
> -	@echo >&2 '***'
> -	@echo >&2 '*** The present kernel configuration has modules disabled.'
> -	@echo >&2 '*** To use the module feature, please run "make menuconfig" etc.'
> -	@echo >&2 '*** to enable CONFIG_MODULES.'
> -	@echo >&2 '***'
> -	@exit 1
> +PHONY += __external_modules_error
> +
> +modules modules_install: __external_modules_error
> +	@:
>  
>  KBUILD_MODULES :=
>  
> -- 
> 2.39.2

-- 
epost|xmpp: nicolas@fjasle.eu          irc://oftc.net/nsc
↳ gpg: 18ed 52db e34f 860e e9fb  c82b 7d97 0932 55a0 ce7f
     -- frykten for herren er opphav til kunnskap --