linux-kernel - Re: [PATCH 1/1] kbuild: recursive build of external kernel modules

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Tue, 13 Aug 2019 00:23:38 +0900
From:   Masahiro Yamada <yamada.masahiro@...ionext.com>
To:     Shaun Tancheff <shaun@...cheff.com>
Cc:     Shaun Tancheff <stancheff@...y.com>,
        Linux Kbuild mailing list <linux-kbuild@...r.kernel.org>,
        Joe Lawrence <joe.lawrence@...hat.com>,
        "James E . J . Bottomley" <jejb@...ux.ibm.com>,
        Jonathan Corbet <corbet@....net>,
        "Martin K . Petersen" <martin.petersen@...cle.com>,
        Michal Marek <michal.lkml@...kovi.net>,
        Shuah Khan <shuah@...nel.org>,
        Thomas Renninger <trenn@...e.com>,
        "open list:DOCUMENTATION" <linux-doc@...r.kernel.org>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
        Linux PM mailing list <linux-pm@...r.kernel.org>,
        linux-scsi <linux-scsi@...r.kernel.org>
Subject: Re: [PATCH 1/1] kbuild: recursive build of external kernel modules

On Fri, Aug 9, 2019 at 9:21 AM Shaun Tancheff <shaun@...cheff.com> wrote:
>
> When building a tree of external modules stage 2 fails
> silently as the root modules.order is empty.
>
> Modify the modules.order location to be fixed to the
> root when KBUILD_EXTMOD is specified and write all
> module paths to the single modules.order file.

Could you try v5.3-rc4 please?




> Signed-off-by: Shaun Tancheff <stancheff@...y.com>
> ---
>  Makefile               | 1 +
>  scripts/Makefile.build | 8 +++++++-
>  2 files changed, 8 insertions(+), 1 deletion(-)
>
> diff --git a/Makefile b/Makefile
> index 23cdf1f41364..a9964492f47e 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -1622,6 +1622,7 @@ $(module-dirs): prepare $(objtree)/Module.symvers
>
>  modules: $(module-dirs)
>         @$(kecho) '  Building modules, stage 2.';
> +       $(Q)$rm -f $(KBUILD_EXTMOD)/modules.order
>         $(Q)$(MAKE) -f $(srctree)/scripts/Makefile.modpost
>
>  PHONY += modules_install
> diff --git a/scripts/Makefile.build b/scripts/Makefile.build
> index 0d434d0afc0b..f9908b3d59e0 100644
> --- a/scripts/Makefile.build
> +++ b/scripts/Makefile.build
> @@ -64,7 +64,13 @@ builtin-target := $(obj)/built-in.a
>  endif
>
>  ifeq ($(CONFIG_MODULES)$(need-modorder),y1)
> +ifneq ($(KBUILD_EXTMOD),)
> +modorder-target := $(KBUILD_EXTMOD)/modules.order
> +modorder-add := >>
> +else
>  modorder-target := $(obj)/modules.order
> +modorder-add := >
> +endif
>  endif
>
>  mod-targets := $(patsubst %.o, %.mod, $(obj-m))
> @@ -423,7 +429,7 @@ endif # builtin-target
>  $(modorder-target): $(subdir-ym) FORCE
>         $(Q){ $(foreach m, $(modorder), \
>         $(if $(filter %/modules.order, $m), cat $m, echo $m);) :; } \
> -       | $(AWK) '!x[$$0]++' - > $@
> +       | $(AWK) '!x[$$0]++' - $(modorder-add) $@
>
>  #
>  # Rule to compile a set of .o files into one .a file (with symbol table)
> --
> 2.20.1
>


-- 
Best Regards
Masahiro Yamada