linux-kernel - Re: [PATCH v2] MIPS: Makefile: Remove "ifdef need-compiler" for Kbuild.platforms

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: Windows password security audit tool. GUI, reports in PDF.

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

Message-ID: <CAK7LNAS7=qYs7NdDRY+4Dz5Zt0tLDo-W-EMcx+9TaNazNVcX=Q@mail.gmail.com>
Date:   Sat, 11 Dec 2021 01:39:10 +0900
From:   Masahiro Yamada <masahiroy@...nel.org>
To:     Nathan Chancellor <nathan@...nel.org>
Cc:     Tiezhu Yang <yangtiezhu@...ngson.cn>,
        Thomas Bogendoerfer <tsbogend@...ha.franken.de>,
        Jason Self <jason@...ehome.net>,
        Ryutaroh Matsumoto <ryutaroh@....e.titech.ac.jp>,
        "open list:BROADCOM NVRAM DRIVER" <linux-mips@...r.kernel.org>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v2] MIPS: Makefile: Remove "ifdef need-compiler" for Kbuild.platforms

On Sat, Dec 11, 2021 at 1:22 AM Nathan Chancellor <nathan@...nel.org> wrote:
>
> On Fri, Dec 10, 2021 at 12:05:05PM +0800, Tiezhu Yang wrote:
> > After commit 13ceb48bc19c ("MIPS: Loongson2ef: Remove unnecessary
> > {as,cc}-option calls"), no need to use "ifdef need-compiler" for
> > Kbuild.platforms, because the cause of the build issue mentioned
> > in commit 0706f74f719e ("MIPS: fix *-pkg builds for loongson2ef
> > platform") has been disappeared, so just remove it.
> >
> > Signed-off-by: Tiezhu Yang <yangtiezhu@...ngson.cn>
>
> Reviewed-by: Nathan Chancellor <nathan@...nel.org>
>
> > ---
> >
> > v2: fix the typo "casue" --> "cause" in the commit message,
> >     sorry for that.
> >
> >  arch/mips/Makefile | 2 --
> >  1 file changed, 2 deletions(-)
> >
> > diff --git a/arch/mips/Makefile b/arch/mips/Makefile
> > index ace7f03..e036fc0 100644
> > --- a/arch/mips/Makefile
> > +++ b/arch/mips/Makefile
> > @@ -253,9 +253,7 @@ endif
> >  #
> >  # Board-dependent options and extra files
> >  #
> > -ifdef need-compiler
> >  include $(srctree)/arch/mips/Kbuild.platforms
> > -endif
> >
> >  ifdef CONFIG_PHYSICAL_START
> >  load-y                                       = $(CONFIG_PHYSICAL_START)
> > --
> > 2.1.0
> >


Nathan's patch cleaned up  arch/mips/loognson2ef/Platform,
but I still see similar code in arch/mips/sgi-ip22/Platform.


ifdef CONFIG_SGI_IP28
  ifeq ($(call cc-option-yn,-march=r10000 -mr10k-cache-barrier=store), n)
      $(error gcc doesn't support needed option -mr10k-cache-barrier=store)
  endif
endif


Doesn't this cause a failure of *-pkg builds for sgi-ip22 platform?






-- 
Best Regards
Masahiro Yamada