On 10/29/2018 01:29 PM, Will Deacon wrote: > On Thu, Sep 20, 2018 at 04:56:30PM +0300, Andrey Ryabinin wrote: >> ARM64 has asm implementation of memchr(), memcmp(), str[r]chr(), >> str[n]cmp(), str[n]len(). KASAN don't see memory accesses in asm >> code, thus it can potentially miss many bugs. >> >> Ifdef out __HAVE_ARCH_* defines of these functions when KASAN is >> enabled, so the generic implementations from lib/string.c will be used. >> >> We can't just remove the asm functions because efistub uses them. >> And we can't have two non-weak functions either, so declare the asm >> functions as weak. >> >> Reported-by: Kyeongdon Kim <[email protected]> >> Signed-off-by: Andrey Ryabinin <[email protected]> >> --- >> Changes since v1: >> - Use WEAK() instead of .weak >> >> arch/arm64/include/asm/string.h | 14 ++++++++------ >> arch/arm64/kernel/arm64ksyms.c | 7 +++++-- >> arch/arm64/lib/memchr.S | 2 +- >> arch/arm64/lib/memcmp.S | 2 +- >> arch/arm64/lib/strchr.S | 2 +- >> arch/arm64/lib/strcmp.S | 2 +- >> arch/arm64/lib/strlen.S | 2 +- >> arch/arm64/lib/strncmp.S | 2 +- >> arch/arm64/lib/strnlen.S | 2 +- >> arch/arm64/lib/strrchr.S | 2 +- >> 10 files changed, 21 insertions(+), 16 deletions(-) > > Acked-by: Will Deacon <[email protected]> > > Please post these again after the merge window and we'll figure out how to > get them queued. >
Andrew sent these patches to Linus couple days ago, so they are in tree already. Something went wrong with mail notification though. I didn't even realize that they were in -mm tree, because I didn't receive the usual 'the patch has been added to -mm tree' email. But I did receive email that was sent to Linus. Also there was no you or Catalin in Cc tags in 2,3 patches, and in the first patch, the Cc tags were corrupted: From: Andrey Ryabinin <[email protected]> Subject: include/linux/linkage.h: align weak symbols Since WEAK() supposed to be used instead of ENTRY() to define weak symbols, but unlike ENTRY() it doesn't have ALIGN directive. It seems there is no actual reason to not have, so let's add ALIGN to WEAK() too. Link: http://lkml.kernel.org/r/[email protected] Signed-off-by: Andrey Ryabinin <[email protected]> Will Deacon <[email protected]>, Catalin Marinas <[email protected]> Cc: Kyeongdon Kim <[email protected]> Cc: Ard Biesheuvel <[email protected]> Cc: Alexander Potapenko <[email protected]> Cc: Dmitry Vyukov <[email protected]> Cc: Mark Rutland <[email protected]> Signed-off-by: Andrew Morton <[email protected]>
