linux-kernel - Re: [PATCH] powerpc: clean up binutils version check

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] [day] [month] [year] [list]

Date:   Wed, 31 Aug 2022 01:01:25 +0900
From:   Masahiro Yamada <masahiroy@...nel.org>
To:     Michael Ellerman <mpe@...erman.id.au>
Cc:     Christophe Leroy <christophe.leroy@...roup.eu>,
        Nicholas Piggin <npiggin@...il.com>,
        "linuxppc-dev@...ts.ozlabs.org" <linuxppc-dev@...ts.ozlabs.org>,
        "linux-kbuild@...r.kernel.org" <linux-kbuild@...r.kernel.org>,
        Alexey Dobriyan <adobriyan@...il.com>,
        Ard Biesheuvel <ardb@...nel.org>,
        Daniel Axtens <dja@...ens.net>,
        Heiko Carstens <hca@...ux.ibm.com>,
        Kees Cook <keescook@...omium.org>,
        Michal Marek <michal.lkml@...kovi.net>,
        Nathan Chancellor <nathan@...nel.org>,
        Nick Desaulniers <ndesaulniers@...gle.com>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] powerpc: clean up binutils version check

On Tue, Aug 30, 2022 at 7:44 PM Michael Ellerman <mpe@...erman.id.au> wrote:
>
> Christophe Leroy <christophe.leroy@...roup.eu> writes:
> > Le 27/08/2022 à 20:03, Masahiro Yamada a écrit :
> >> On Sun, Aug 28, 2022 at 2:37 AM Christophe Leroy
> >> <christophe.leroy@...roup.eu> wrote:
> >>> Le 27/08/2022 à 18:40, Masahiro Yamada a écrit :
> >>>> The checkbin in arch/powerpc/Makefile errors out if ld <= 2.24.
> >>>> So, the requirement on PPC is binutils >= 2.25. It is cleaner to
> >>>> specify it in scripts/min-tool-version.sh. If binutils < 2.25 is
> >>>> used, the toolchain check will fail in the Kconfig stage going
> >>>> forward.
> >>>>
> >>>> Since binutils >= 2.25 is already required, another version test
> >>>> for --save-restore-funcs on PPC64 is always met.
> ...
> >>>> diff --git a/scripts/min-tool-version.sh b/scripts/min-tool-version.sh
> >>>> index 250925aab101..7df9f2150ea1 100755
> >>>> --- a/scripts/min-tool-version.sh
> >>>> +++ b/scripts/min-tool-version.sh
> >>>> @@ -14,7 +14,13 @@ fi
> >>>>
> >>>>    case "$1" in
> >>>>    binutils)
> >>>> -     echo 2.23.0
> >>>> +     if [ "$SRCARCH" = powerpc ]; then
> >>>
> >>> Isn't this limitation only for ppc64le ?
> >>>
> >>> Refer commit 60e065f70bdb ("powerpc: Reject binutils 2.24 when building
> >>> little endian")
> >>
> >> I do not see any CONFIG check in the current checkbin.
> >>
> >> Refer commit a3ad84da0760 ("powerpc/toc: Future proof
> >> kernel toc")
> >
> > That's odd. There is no toc on PPC32.
>
> I think that's just a bug in a3ad84da0760.
>
> But that means we inadvertantly dropped support for 2.24 about 8 months
> ago, and no one noticed.
>
> Let's see what the responses are to Nick's proposal to increase the
> minimum to 2.25.1.
>
> cheers



Either way is fine with me, but in that case,
do I need to get a higher-level Ack from Linus?

The current one can go to the ppc tree
since apparently it does not affect any other arches.

-- 
Best Regards
Masahiro Yamada