Re: [PATCH v2 07/12] parisc/percpu: Work around the lack of __SIZEOF_INT128__

From: H. Peter Anvin
Date: Fri Jun 02 2023 - 13:02:22 EST


On June 2, 2023 7:39:12 AM PDT, Peter Zijlstra <peterz@xxxxxxxxxxxxx> wrote:
>On Thu, Jun 01, 2023 at 09:29:18AM -0400, Linus Torvalds wrote:
>
>> Right now we have that "minimum gcc version" in a somewhat annoying
>> place: it's in the ./scripts/min-tool-version.sh file as a shell
>> script.
>
>Something like so then?
>
>---
>Subject: parisc: Raise minimal GCC version
>From: Peter Zijlstra <peterz@xxxxxxxxxxxxx>
>Date: Fri Jun 2 16:33:54 CEST 2023
>
>With 64bit builds depending on __SIZEOF_INT128__ raise the parisc
>minimum compiler version to gcc-11.0.0.
>
>All other 64bit architectures provide this from GCC-5.1.0 (and
>probably before), except hppa64 which only started advertising this
>with GCC-11.
>
>Signed-off-by: Peter Zijlstra (Intel) <peterz@xxxxxxxxxxxxx>
>---
> scripts/min-tool-version.sh | 6 +++++-
> 1 file changed, 5 insertions(+), 1 deletion(-)
>
>--- a/scripts/min-tool-version.sh
>+++ b/scripts/min-tool-version.sh
>@@ -17,7 +17,11 @@ binutils)
> echo 2.25.0
> ;;
> gcc)
>- echo 5.1.0
>+ if [ "$SRCARCH" = parisc ]; then
>+ echo 11.0.0
>+ else
>+ echo 5.1.0
>+ fi
> ;;
> llvm)
> if [ "$SRCARCH" = s390 ]; then

Dumb question: is this only about the cpp macro or is it about __int128 existing at all?