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

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

Message-ID: <87o7w2j9jn.fsf@mpe.ellerman.id.au>
Date:   Tue, 30 Aug 2022 20:43:56 +1000
From:   Michael Ellerman <mpe@...erman.id.au>
To:     Christophe Leroy <christophe.leroy@...roup.eu>,
        Masahiro Yamada <masahiroy@...nel.org>
Cc:     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

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