Re: [PATCH] kasan: add no_sanitize attribute for clang builds

2018-04-17 Thread Andrey Konovalov
On Fri, Apr 13, 2018 at 9:16 PM, Andrey Ryabinin wrote: > However, "#ifdef CONFIG_KASAN" seems to be redundant, I'd rather remove it. Done, sent v2. Thanks!

Re: [PATCH] kasan: add no_sanitize attribute for clang builds

2018-04-13 Thread Andrey Ryabinin
On 04/13/2018 08:34 PM, Andrey Konovalov wrote: > On Fri, Apr 13, 2018 at 5:31 PM, Andrey Ryabinin > wrote: >> >> >> On 04/12/2018 08:29 PM, Andrey Konovalov wrote: >>> KASAN uses the __no_sanitize_address macro to disable instrumentation >>> of particular functions. Right now it's defined only

Re: [PATCH] kasan: add no_sanitize attribute for clang builds

2018-04-13 Thread Andrey Konovalov
On Fri, Apr 13, 2018 at 5:31 PM, Andrey Ryabinin wrote: > > > On 04/12/2018 08:29 PM, Andrey Konovalov wrote: >> KASAN uses the __no_sanitize_address macro to disable instrumentation >> of particular functions. Right now it's defined only for GCC build, >> which causes false positives when clang i

Re: [PATCH] kasan: add no_sanitize attribute for clang builds

2018-04-13 Thread Andrey Ryabinin
On 04/12/2018 08:29 PM, Andrey Konovalov wrote: > KASAN uses the __no_sanitize_address macro to disable instrumentation > of particular functions. Right now it's defined only for GCC build, > which causes false positives when clang is used. > > This patch adds a definition for clang. > > Note,

[PATCH] kasan: add no_sanitize attribute for clang builds

2018-04-12 Thread Andrey Konovalov
KASAN uses the __no_sanitize_address macro to disable instrumentation of particular functions. Right now it's defined only for GCC build, which causes false positives when clang is used. This patch adds a definition for clang. Note, that clang's revision 329612 or higher is required. Signed-off-