ARM: 9151/1: Thumb2: avoid __builtin_thread_pointer() on Clang
authorArd Biesheuvel <ardb@kernel.org>
Fri, 29 Oct 2021 16:49:37 +0000 (17:49 +0100)
committerRussell King (Oracle) <rmk+kernel@armlinux.org.uk>
Sat, 30 Oct 2021 10:24:37 +0000 (11:24 +0100)
commitc1e42efacb9bb4ae873e9b1cf249fa4fb6ef7f84
tree4fc038347e172360bf7ab4be79ce8d28da891ccc
parentfa191b711c32ba107cf8d3474cd860407b7e997a
ARM: 9151/1: Thumb2: avoid __builtin_thread_pointer() on Clang

If available, we use the __builtin_thread_pointer() helper to get the
value of the TLS register, to help the compiler understand that it
doesn't need to reload it every time we access 'current'.

Unfortunately, Clang fails to emit the MRC system register read
directly when building for Thumb2, and instead, it issues a call to the
__aeabi_read_tp helper, which the kernel does not provide, and so this
result in link failures at build time.

So create a special case for this, and emit the MRC directly using an
asm() block, just like we do when the helper is not available to begin
with.

Link: https://github.com/ClangBuiltLinux/linux/issues/1485
Reviewed-by: Nick Desaulniers <ndesaulniers@google.com>
Reviewed-by: Nathan Chancellor <nathan@kernel.org>
Tested-by: Nathan Chancellor <nathan@kernel.org>
Reviewed-by: Kees Cook <keescook@chromium.org>
Signed-off-by: Ard Biesheuvel <ardb@kernel.org>
Signed-off-by: Russell King (Oracle) <rmk+kernel@armlinux.org.uk>
arch/arm/include/asm/current.h