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: Tue, 23 Feb 2021 19:12:44 -0600 From: Rob Herring <robh@...nel.org> To: Masahiro Yamada <masahiroy@...nel.org> Cc: Michal Marek <michal.lkml@...kovi.net>, Linux Kbuild mailing list <linux-kbuild@...r.kernel.org>, Linus Torvalds <torvalds@...ux-foundation.org>, Andy Lutomirski <luto@...nel.org>, Thomas Gleixner <tglx@...utronix.de>, Ingo Molnar <mingo@...hat.com>, Borislav Petkov <bp@...en8.de>, X86 ML <x86@...nel.org>, "H. Peter Anvin" <hpa@...or.com>, Linux Kernel Mailing List <linux-kernel@...r.kernel.org> Subject: Re: [PATCH 1/3] kbuild: Make old-atomics and missing-syscalls phony targets On Tue, Feb 23, 2021 at 6:47 PM Masahiro Yamada <masahiroy@...nel.org> wrote: > > On Wed, Feb 24, 2021 at 3:14 AM Rob Herring <robh@...nel.org> wrote: > > > > The old-atomics and missing-syscalls targets are not files, so they > > should be marked as PHONY. > > > > Cc: Masahiro Yamada <masahiroy@...nel.org> > > Cc: Michal Marek <michal.lkml@...kovi.net> > > Signed-off-by: Rob Herring <robh@...nel.org> > > --- > > Kbuild | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/Kbuild b/Kbuild > > index fa441b98c9f6..032157c3ffd2 100644 > > --- a/Kbuild > > +++ b/Kbuild > > @@ -44,6 +44,7 @@ always-y += missing-syscalls > > quiet_cmd_syscalls = CALL $< > > cmd_syscalls = $(CONFIG_SHELL) $< $(CC) $(c_flags) $(missing_syscalls_flags) > > > > +PHONY += missing-syscalls > > missing-syscalls: scripts/checksyscalls.sh $(offsets-file) FORCE > > $(call cmd,syscalls) > > > > @@ -55,5 +56,6 @@ always-y += old-atomics > > quiet_cmd_atomics = CALL $< > > cmd_atomics = $(CONFIG_SHELL) $< > > > > +PHONY += old-atomics > > > I do not think this is the right fix. > > always-y (specified a few lines above) adds $(obj)/ prefix, > and is not supposed to work with PHONY. > > > It is wrong to blindly eliminate > the errors detected by your 3/3 What about checking just hostprogs and userprogs? That's what I initially had, but thought we could widen the net a little. Rob
Powered by blists - more mailing lists