Re: [PATCH 2/2] string.h: fix incompatibility between FORTIFY_SOURCE and KASAN

2020-01-16 Thread kbuild test robot
Hi Daniel, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on linux/master] [also build test WARNING on arm64/for-next/core powerpc/next s390/features tip/x86/core linus/master v5.5-rc6 next-20200110] [if your patch is applied to the wrong git tree, please drop us

Re: [PATCH 2/2] string.h: fix incompatibility between FORTIFY_SOURCE and KASAN

2020-01-15 Thread Daniel Axtens
Dmitry Vyukov writes: > On Wed, Jan 15, 2020 at 7:37 AM Daniel Axtens wrote: >> >> The memcmp KASAN self-test fails on a kernel with both KASAN and >> FORTIFY_SOURCE. >> >> When FORTIFY_SOURCE is on, a number of functions are replaced with >> fortified versions, which attempt to check the sizes

Re: [PATCH 2/2] string.h: fix incompatibility between FORTIFY_SOURCE and KASAN

2020-01-15 Thread Dmitry Vyukov
On Wed, Jan 15, 2020 at 7:37 AM Daniel Axtens wrote: > > The memcmp KASAN self-test fails on a kernel with both KASAN and > FORTIFY_SOURCE. > > When FORTIFY_SOURCE is on, a number of functions are replaced with > fortified versions, which attempt to check the sizes of the operands. > However, thes

[PATCH 2/2] string.h: fix incompatibility between FORTIFY_SOURCE and KASAN

2020-01-14 Thread Daniel Axtens
The memcmp KASAN self-test fails on a kernel with both KASAN and FORTIFY_SOURCE. When FORTIFY_SOURCE is on, a number of functions are replaced with fortified versions, which attempt to check the sizes of the operands. However, these functions often directly invoke __builtin_foo() once they have pe