ARM: 9022/1: Change arch/arm/lib/mem*.S to use WEAK instead of .weak
authorFangrui Song <maskray@google.com>
Fri, 6 Nov 2020 20:46:11 +0000 (21:46 +0100)
committerRussell King <rmk+kernel@armlinux.org.uk>
Thu, 12 Nov 2020 14:53:19 +0000 (14:53 +0000)
commit735e8d93dc2b107f7891a9c2b1c4cfbea1fcbbbc
treed37af8ed305cebd6185990a3bc124cb31efa3e36
parentfc2933c133744305236793025b00c2f7d258b687
ARM: 9022/1: Change arch/arm/lib/mem*.S to use WEAK instead of .weak

Commit d6d51a96c7d6 ("ARM: 9014/2: Replace string mem* functions for
KASan") add .weak directives to memcpy/memmove/memset to avoid collision
with KASAN interceptors.

This does not work with LLVM's integrated assembler (the assembly snippet
`.weak memcpy ... .globl memcpy` produces a STB_GLOBAL memcpy while GNU as
produces a STB_WEAK memcpy). LLVM 12 (since https://reviews.llvm.org/D90108)
will error on such an overridden symbol binding.

Use the appropriate WEAK macro instead.

Link: https://github.com/ClangBuiltLinux/linux/issues/1190
--

Fixes: d6d51a96c7d6 ("ARM: 9014/2: Replace string mem* functions for KASan")
Reported-by: Nick Desaulniers <ndesaulniers@google.com>
Signed-off-by: Fangrui Song <maskray@google.com>
Reviewed-by: Nick Desaulniers <ndesaulniers@google.com>
Tested-by: Nick Desaulniers <ndesaulniers@google.com>
Signed-off-by: Russell King <rmk+kernel@armlinux.org.uk>
arch/arm/lib/memcpy.S
arch/arm/lib/memmove.S
arch/arm/lib/memset.S