2024-01-08 03:32:09

by WANG Rui

Subject: [PATCH] scripts/min-tool-version.sh: Raise min clang version to 18.0.0 for loongarch

The existing mainline clang development version encounters diffculties
compiling the LoongArch kernel module. It is anticipated that this issue
will be resolved in the upcoming 18.0.0 release. To prevent user
confusion arising from broken builds, it is advisable to raise the
minimum required clang version for LoongArch to 18.0.0.

Suggested-by: Nathan Chancellor <[email protected]>
Signed-off-by: Tiezhu Yang <[email protected]>
Signed-off-by: WANG Rui <[email protected]>
---
scripts/min-tool-version.sh | 2 ++
1 file changed, 2 insertions(+)

diff --git a/scripts/min-tool-version.sh b/scripts/min-tool-version.sh
index c62066825f53..9faa4d3d91e3 100755
--- a/scripts/min-tool-version.sh
+++ b/scripts/min-tool-version.sh
@@ -26,6 +26,8 @@ gcc)
llvm)
if [ "$SRCARCH" = s390 ]; then
echo 15.0.0
+ elif [ "$SRCARCH" = loongarch ]; then
+ echo 18.0.0
else
echo 11.0.0
fi
--
2.43.0

2024-01-08 16:32:11

by Nick Desaulniers

Subject: Re: [PATCH] scripts/min-tool-version.sh: Raise min clang version to 18.0.0 for loongarch

On Sun, Jan 7, 2024 at 7:32 PM WANG Rui <[email protected]> wrote:
>
> The existing mainline clang development version encounters diffculties

s/diffculties/difficulties/

> compiling the LoongArch kernel module. It is anticipated that this issue
> will be resolved in the upcoming 18.0.0 release. To prevent user
> confusion arising from broken builds, it is advisable to raise the
> minimum required clang version for LoongArch to 18.0.0.
>
> Suggested-by: Nathan Chancellor <[email protected]>
> Signed-off-by: Tiezhu Yang <[email protected]>
> Signed-off-by: WANG Rui <[email protected]>

Thanks for all of the work here; it's nice to observe the progress.

Acked-by: Nick Desaulniers <[email protected]>
Link: https://github.com/ClangBuiltLinux/linux/issues/1941

> ---
> scripts/min-tool-version.sh | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/scripts/min-tool-version.sh b/scripts/min-tool-version.sh
> index c62066825f53..9faa4d3d91e3 100755
> --- a/scripts/min-tool-version.sh
> +++ b/scripts/min-tool-version.sh
> @@ -26,6 +26,8 @@ gcc)
> llvm)
> if [ "$SRCARCH" = s390 ]; then
> echo 15.0.0
> + elif [ "$SRCARCH" = loongarch ]; then
> + echo 18.0.0
> else
> echo 11.0.0
> fi
> --
> 2.43.0
>
>


--
Thanks,
~Nick Desaulniers

2024-01-08 16:38:50

by Nathan Chancellor

Subject: Re: [PATCH] scripts/min-tool-version.sh: Raise min clang version to 18.0.0 for loongarch

On Mon, Jan 08, 2024 at 11:31:38AM +0800, WANG Rui wrote:
> The existing mainline clang development version encounters diffculties
> compiling the LoongArch kernel module. It is anticipated that this issue
> will be resolved in the upcoming 18.0.0 release. To prevent user
> confusion arising from broken builds, it is advisable to raise the
> minimum required clang version for LoongArch to 18.0.0.
>
> Suggested-by: Nathan Chancellor <[email protected]>
> Signed-off-by: Tiezhu Yang <[email protected]>
> Signed-off-by: WANG Rui <[email protected]>

Reviewed-by: Nathan Chancellor <[email protected]>

Thanks a lot for all the work driving getting everything to work
properly with LLVM!

> ---
> scripts/min-tool-version.sh | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/scripts/min-tool-version.sh b/scripts/min-tool-version.sh
> index c62066825f53..9faa4d3d91e3 100755
> --- a/scripts/min-tool-version.sh
> +++ b/scripts/min-tool-version.sh
> @@ -26,6 +26,8 @@ gcc)
> llvm)
> if [ "$SRCARCH" = s390 ]; then
> echo 15.0.0
> + elif [ "$SRCARCH" = loongarch ]; then
> + echo 18.0.0
> else
> echo 11.0.0
> fi
> --
> 2.43.0
>
>

2024-01-09 01:40:28

by Huacai Chen

Subject: Re: [PATCH] scripts/min-tool-version.sh: Raise min clang version to 18.0.0 for loongarch

Applied with R-b/A-b tags and fix typo together, thanks.

Huacai

On Tue, Jan 9, 2024 at 12:38 AM Nathan Chancellor <[email protected]> wrote:
>
> On Mon, Jan 08, 2024 at 11:31:38AM +0800, WANG Rui wrote:
> > The existing mainline clang development version encounters diffculties
> > compiling the LoongArch kernel module. It is anticipated that this issue
> > will be resolved in the upcoming 18.0.0 release. To prevent user
> > confusion arising from broken builds, it is advisable to raise the
> > minimum required clang version for LoongArch to 18.0.0.
> >
> > Suggested-by: Nathan Chancellor <[email protected]>
> > Signed-off-by: Tiezhu Yang <[email protected]>
> > Signed-off-by: WANG Rui <[email protected]>
>
> Reviewed-by: Nathan Chancellor <[email protected]>
>
> Thanks a lot for all the work driving getting everything to work
> properly with LLVM!
>
> > ---
> > scripts/min-tool-version.sh | 2 ++
> > 1 file changed, 2 insertions(+)
> >
> > diff --git a/scripts/min-tool-version.sh b/scripts/min-tool-version.sh
> > index c62066825f53..9faa4d3d91e3 100755
> > --- a/scripts/min-tool-version.sh
> > +++ b/scripts/min-tool-version.sh
> > @@ -26,6 +26,8 @@ gcc)
> > llvm)
> > if [ "$SRCARCH" = s390 ]; then
> > echo 15.0.0
> > + elif [ "$SRCARCH" = loongarch ]; then
> > + echo 18.0.0
> > else
> > echo 11.0.0
> > fi
> > --
> > 2.43.0
> >
> >
>