mbox series

[v3,0/8] LoongArch: Preliminary ClangBuiltLinux enablement

Message ID 20230625095644.3156349-1-kernel@xen0n.name (mailing list archive)
Headers show
Series LoongArch: Preliminary ClangBuiltLinux enablement | expand

Message

WANG Xuerui June 25, 2023, 9:56 a.m. UTC
From: WANG Xuerui <git@xen0n.name>

Hi,

It's been a long time since the LoongArch port was upstreamed to LLVM,
and there seems to be evidence that Linux was successfully built with
Clang inside Loongson roughly around that time; however, a lot has
changed since then, and the Linux/LoongArch codebase now makes use of
more novel features that necessitate further work. (The enablement work
is tracked at [1].)

With this patch series and a patched LLVM/Clang/LLD ([2][3][4][5]), a
working kernel can be built with `make LLVM=1`. Although currently
support for CONFIG_RELOCATABLE and CONFIG_MODULE is still TODO, we've
decided to post the series early to hopefully reduce the rebase
burden. The series contains several useful cleanups anyway.

Regarding how to merge this: because only Patch 8 is outside
arch/loongarch, I'd prefer the series to get merged through Huacai's
tree. The series applies cleanly on top of next-20230622.

Thanks go to the ClangBuiltLinux team, and LoongArch toolchain
maintainers from Loongson and the community alike; without your help
this would come much later, if at all (my free time has been steadily
dwindling this year already).

Your comments are welcome!

[1]: https://github.com/ClangBuiltLinux/linux/issues/1787
[2]: https://reviews.llvm.org/D153609
[3]: https://reviews.llvm.org/D138135
[4]: https://reviews.llvm.org/D150196
[5]: https://reviews.llvm.org/D153707

Changes in v3:

- Squashed the two CFLAGS patches into one, and removed the CC_IS_CLANG
  check in favor of a feature detection approach (Huacai and Ruoyao)
- Removed unnecessary BUILD_BUG_ONs in the invtlb wrappers, and made
  them __always_inline (Ruoyao and Bibo)
- Linked to the explanation regarding the Clang triple's "libc" part
  (Nick)
- Fixed a wrong punctuation in commit message of Patch 4

Changes in v2:

- Merged the two FCSR-related patches, now using the same approach for
  assembly and C (given that the inline asm codepath cannot be removed
  right away), also change terminology: register "class" instead of
  "bank"
- Restored signatures of invtlb wrappers for ease of rebase (potentially
  downstream product kernels?)
- Removed -G0 switches altogether (turned out it's useless upon closer
  look)
- Fix -mabi and -msoft-float handling in the CFLAGS patch; two more LLVM
  patches are now necessary (the [4] and [5] links above) but the
  original and correct CFLAGS arrangement now works

WANG Rui (2):
  LoongArch: Calculate various sizes in the linker script
  LoongArch: extable: Also recognize ABI names of registers

WANG Xuerui (6):
  LoongArch: Prepare for assemblers with proper FCSR class support
  LoongArch: Make the CPUCFG and CSR ops simple aliases of compiler
    built-ins
  LoongArch: Simplify the invtlb wrappers
  LoongArch: Tweak CFLAGS for Clang compatibility
  LoongArch: Mark Clang LTO as working
  Makefile: Add loongarch target flag for Clang compilation

 arch/loongarch/Kconfig                 |  5 ++
 arch/loongarch/Makefile                | 21 +++++---
 arch/loongarch/include/asm/fpregdef.h  |  7 +++
 arch/loongarch/include/asm/gpr-num.h   | 30 +++++++++++
 arch/loongarch/include/asm/loongarch.h | 72 +++++++-------------------
 arch/loongarch/include/asm/percpu.h    |  6 ++-
 arch/loongarch/include/asm/tlb.h       | 43 +++++++--------
 arch/loongarch/kernel/efi-header.S     |  6 +--
 arch/loongarch/kernel/head.S           |  8 +--
 arch/loongarch/kernel/traps.c          |  2 +-
 arch/loongarch/kernel/vmlinux.lds.S    |  7 +++
 arch/loongarch/lib/dump_tlb.c          |  6 +--
 arch/loongarch/vdso/Makefile           |  2 +-
 scripts/Makefile.clang                 |  1 +
 14 files changed, 118 insertions(+), 98 deletions(-)

Comments

Huacai Chen June 27, 2023, 3:58 a.m. UTC | #1
Queued for loongarch-next, thanks.

Huacai

On Sun, Jun 25, 2023 at 5:56 PM WANG Xuerui <kernel@xen0n.name> wrote:
>
> From: WANG Xuerui <git@xen0n.name>
>
> Hi,
>
> It's been a long time since the LoongArch port was upstreamed to LLVM,
> and there seems to be evidence that Linux was successfully built with
> Clang inside Loongson roughly around that time; however, a lot has
> changed since then, and the Linux/LoongArch codebase now makes use of
> more novel features that necessitate further work. (The enablement work
> is tracked at [1].)
>
> With this patch series and a patched LLVM/Clang/LLD ([2][3][4][5]), a
> working kernel can be built with `make LLVM=1`. Although currently
> support for CONFIG_RELOCATABLE and CONFIG_MODULE is still TODO, we've
> decided to post the series early to hopefully reduce the rebase
> burden. The series contains several useful cleanups anyway.
>
> Regarding how to merge this: because only Patch 8 is outside
> arch/loongarch, I'd prefer the series to get merged through Huacai's
> tree. The series applies cleanly on top of next-20230622.
>
> Thanks go to the ClangBuiltLinux team, and LoongArch toolchain
> maintainers from Loongson and the community alike; without your help
> this would come much later, if at all (my free time has been steadily
> dwindling this year already).
>
> Your comments are welcome!
>
> [1]: https://github.com/ClangBuiltLinux/linux/issues/1787
> [2]: https://reviews.llvm.org/D153609
> [3]: https://reviews.llvm.org/D138135
> [4]: https://reviews.llvm.org/D150196
> [5]: https://reviews.llvm.org/D153707
>
> Changes in v3:
>
> - Squashed the two CFLAGS patches into one, and removed the CC_IS_CLANG
>   check in favor of a feature detection approach (Huacai and Ruoyao)
> - Removed unnecessary BUILD_BUG_ONs in the invtlb wrappers, and made
>   them __always_inline (Ruoyao and Bibo)
> - Linked to the explanation regarding the Clang triple's "libc" part
>   (Nick)
> - Fixed a wrong punctuation in commit message of Patch 4
>
> Changes in v2:
>
> - Merged the two FCSR-related patches, now using the same approach for
>   assembly and C (given that the inline asm codepath cannot be removed
>   right away), also change terminology: register "class" instead of
>   "bank"
> - Restored signatures of invtlb wrappers for ease of rebase (potentially
>   downstream product kernels?)
> - Removed -G0 switches altogether (turned out it's useless upon closer
>   look)
> - Fix -mabi and -msoft-float handling in the CFLAGS patch; two more LLVM
>   patches are now necessary (the [4] and [5] links above) but the
>   original and correct CFLAGS arrangement now works
>
> WANG Rui (2):
>   LoongArch: Calculate various sizes in the linker script
>   LoongArch: extable: Also recognize ABI names of registers
>
> WANG Xuerui (6):
>   LoongArch: Prepare for assemblers with proper FCSR class support
>   LoongArch: Make the CPUCFG and CSR ops simple aliases of compiler
>     built-ins
>   LoongArch: Simplify the invtlb wrappers
>   LoongArch: Tweak CFLAGS for Clang compatibility
>   LoongArch: Mark Clang LTO as working
>   Makefile: Add loongarch target flag for Clang compilation
>
>  arch/loongarch/Kconfig                 |  5 ++
>  arch/loongarch/Makefile                | 21 +++++---
>  arch/loongarch/include/asm/fpregdef.h  |  7 +++
>  arch/loongarch/include/asm/gpr-num.h   | 30 +++++++++++
>  arch/loongarch/include/asm/loongarch.h | 72 +++++++-------------------
>  arch/loongarch/include/asm/percpu.h    |  6 ++-
>  arch/loongarch/include/asm/tlb.h       | 43 +++++++--------
>  arch/loongarch/kernel/efi-header.S     |  6 +--
>  arch/loongarch/kernel/head.S           |  8 +--
>  arch/loongarch/kernel/traps.c          |  2 +-
>  arch/loongarch/kernel/vmlinux.lds.S    |  7 +++
>  arch/loongarch/lib/dump_tlb.c          |  6 +--
>  arch/loongarch/vdso/Makefile           |  2 +-
>  scripts/Makefile.clang                 |  1 +
>  14 files changed, 118 insertions(+), 98 deletions(-)
>
> --
> 2.40.0
>