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
| ||
|
Date: Fri, 6 Apr 2018 09:59:59 +0900 From: Masahiro Yamada <yamada.masahiro@...ionext.com> To: Jiri Olsa <jolsa@...hat.com> Cc: Jiri Olsa <jolsa@...nel.org>, Alexei Starovoitov <ast@...nel.org>, Daniel Borkmann <daniel@...earbox.net>, lkml <linux-kernel@...r.kernel.org>, netdev@...r.kernel.org, Linux Kbuild mailing list <linux-kbuild@...r.kernel.org>, Quentin Monnet <quentin.monnet@...ronome.com>, Eugene Syromiatnikov <esyr@...hat.com>, Jiri Benc <jbenc@...hat.com>, Stanislav Kozina <skozina@...hat.com>, Jerome Marchand <jmarchan@...hat.com>, Arnaldo Carvalho de Melo <acme@...hat.com>, Michal Marek <michal.lkml@...kovi.net>, Jiri Kosina <jkosina@...e.cz> Subject: Re: [PATCH 3/9] kbuild: Do not pass arguments to link-vmlinux.sh 2018-04-06 3:59 GMT+09:00 Jiri Olsa <jolsa@...hat.com>: > On Fri, Apr 06, 2018 at 12:50:00AM +0900, Masahiro Yamada wrote: >> 2018-04-06 0:16 GMT+09:00 Jiri Olsa <jolsa@...nel.org>: >> > There's no need to pass LD* arguments to link-vmlinux.sh, >> > because they are passed as variables. The only argument >> > the link-vmlinux.sh supports is the 'clean' argument. >> > >> > Signed-off-by: Jiri Olsa <jolsa@...nel.org> >> > --- >> >> Wrong. >> >> $(LD) $(LDFLAGS) $(LDFLAGS_vmlinux) >> exist here so that any change in them >> invokes scripts/linkk-vmlinux.sh > > sry, I can't see that.. but it's just a side fix, > which is actually not needed for the rest > > I'll check on more and address this separately The link command is recorded in .vmlinux.cmd then, it is checked by if_changed in the next rebuild because link-vmlinux is invoked by $(call if_changed,...) +$(call if_changed,link-vmlinux) > thanks, > jirka > >> >> >> >> >> > Makefile | 2 +- >> > 1 file changed, 1 insertion(+), 1 deletion(-) >> > >> > diff --git a/Makefile b/Makefile >> > index d3300e46f925..a65a3919c6ad 100644 >> > --- a/Makefile >> > +++ b/Makefile >> > @@ -1027,7 +1027,7 @@ ARCH_POSTLINK := $(wildcard $(srctree)/arch/$(SRCARCH)/Makefile.postlink) >> > >> > # Final link of vmlinux with optional arch pass after final link >> > cmd_link-vmlinux = \ >> > - $(CONFIG_SHELL) $< $(LD) $(LDFLAGS) $(LDFLAGS_vmlinux) ; \ >> > + $(CONFIG_SHELL) $< ; \ >> > $(if $(ARCH_POSTLINK), $(MAKE) -f $(ARCH_POSTLINK) $@, true) >> > >> > vmlinux: scripts/link-vmlinux.sh vmlinux_prereq $(vmlinux-deps) FORCE >> >> >> >> >> >> >> -- >> Best Regards >> Masahiro Yamada > -- > To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in > the body of a message to majordomo@...r.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- Best Regards Masahiro Yamada
Powered by blists - more mailing lists