Re: [PATCH] ubsan: disable unsigned-integer-overflow sanitizer with clang

2021-01-07 Thread Arnd Bergmann
On Thu, Jan 7, 2021 at 7:15 PM Nathan Chancellor wrote: > On Thu, Jan 07, 2021 at 09:22:00AM -0800, Kees Cook wrote: > > I think this is: > > https://github.com/ClangBuiltLinux/linux/issues/256 > > and that bug needs re-opening? Or maybe there's a new bug I can't find? > > The problem is that app

Re: [PATCH] ubsan: disable unsigned-integer-overflow sanitizer with clang

2021-01-07 Thread Nathan Chancellor
On Thu, Jan 07, 2021 at 09:22:00AM -0800, Kees Cook wrote: > On Thu, Jan 07, 2021 at 05:09:59PM +0100, Arnd Bergmann wrote: > > On Wed, Jan 6, 2021 at 11:12 PM Arnd Bergmann wrote: > > > > > > On Wed, Jan 6, 2021 at 10:57 PM Kees Cook wrote: > > > > On Wed, Dec 30, 2020 at 04:47:35PM +0100, Arnd

Re: [PATCH] ubsan: disable unsigned-integer-overflow sanitizer with clang

2021-01-07 Thread Kees Cook
On Thu, Jan 07, 2021 at 05:09:59PM +0100, Arnd Bergmann wrote: > On Wed, Jan 6, 2021 at 11:12 PM Arnd Bergmann wrote: > > > > On Wed, Jan 6, 2021 at 10:57 PM Kees Cook wrote: > > > On Wed, Dec 30, 2020 at 04:47:35PM +0100, Arnd Bergmann wrote: > > > > diff --git a/lib/Kconfig.ubsan b/lib/Kconfig.

Re: [PATCH] ubsan: disable unsigned-integer-overflow sanitizer with clang

2021-01-07 Thread Arnd Bergmann
On Wed, Jan 6, 2021 at 11:12 PM Arnd Bergmann wrote: > > On Wed, Jan 6, 2021 at 10:57 PM Kees Cook wrote: > > On Wed, Dec 30, 2020 at 04:47:35PM +0100, Arnd Bergmann wrote: > > > diff --git a/lib/Kconfig.ubsan b/lib/Kconfig.ubsan > > > index 8b635fd75fe4..e23873282ba7 100644 > > > --- a/lib/Kconf

Re: [PATCH] ubsan: disable unsigned-integer-overflow sanitizer with clang

2021-01-06 Thread Nathan Chancellor
On Wed, Jan 06, 2021 at 11:06:39PM +0100, Arnd Bergmann wrote: > On Wed, Jan 6, 2021 at 10:38 PM Nathan Chancellor > wrote: > > On Wed, Jan 06, 2021 at 10:12:51AM +0100, Arnd Bergmann wrote: > > > On Tue, Jan 5, 2021 at 10:25 AM Arnd Bergmann wrote: > > > > > > > > On Mon, Jan 4, 2021 at 11:33 PM

Re: [PATCH] ubsan: disable unsigned-integer-overflow sanitizer with clang

2021-01-06 Thread Arnd Bergmann
On Thu, Jan 7, 2021 at 12:17 AM Kees Cook wrote: > > On Wed, Jan 06, 2021 at 11:12:18PM +0100, Arnd Bergmann wrote: > > On Wed, Jan 6, 2021 at 10:57 PM Kees Cook wrote: > > > On Wed, Dec 30, 2020 at 04:47:35PM +0100, Arnd Bergmann wrote: > > > > diff --git a/lib/Kconfig.ubsan b/lib/Kconfig.ubsan

Re: [PATCH] ubsan: disable unsigned-integer-overflow sanitizer with clang

2021-01-06 Thread Kees Cook
On Wed, Jan 06, 2021 at 11:12:18PM +0100, Arnd Bergmann wrote: > On Wed, Jan 6, 2021 at 10:57 PM Kees Cook wrote: > > On Wed, Dec 30, 2020 at 04:47:35PM +0100, Arnd Bergmann wrote: > > > diff --git a/lib/Kconfig.ubsan b/lib/Kconfig.ubsan > > > index 8b635fd75fe4..e23873282ba7 100644 > > > --- a/li

Re: [PATCH] ubsan: disable unsigned-integer-overflow sanitizer with clang

2021-01-06 Thread Arnd Bergmann
On Wed, Jan 6, 2021 at 10:57 PM Kees Cook wrote: > On Wed, Dec 30, 2020 at 04:47:35PM +0100, Arnd Bergmann wrote: > > diff --git a/lib/Kconfig.ubsan b/lib/Kconfig.ubsan > > index 8b635fd75fe4..e23873282ba7 100644 > > --- a/lib/Kconfig.ubsan > > +++ b/lib/Kconfig.ubsan > > @@ -122,6 +122,8 @@ confi

Re: [PATCH] ubsan: disable unsigned-integer-overflow sanitizer with clang

2021-01-06 Thread Arnd Bergmann
On Wed, Jan 6, 2021 at 10:38 PM Nathan Chancellor wrote: > On Wed, Jan 06, 2021 at 10:12:51AM +0100, Arnd Bergmann wrote: > > On Tue, Jan 5, 2021 at 10:25 AM Arnd Bergmann wrote: > > > > > > On Mon, Jan 4, 2021 at 11:33 PM Nathan Chancellor > > > wrote: > > > > On Wed, Dec 30, 2020 at 05:13:03PM

Re: [PATCH] ubsan: disable unsigned-integer-overflow sanitizer with clang

2021-01-06 Thread Kees Cook
On Wed, Dec 30, 2020 at 04:47:35PM +0100, Arnd Bergmann wrote: > From: Arnd Bergmann > > Building ubsan kernels even for compile-testing introduced these > warnings in my randconfig environment: > > crypto/blake2b_generic.c:98:13: error: stack frame size of 9636 bytes in > function 'blake2b_com

Re: [PATCH] ubsan: disable unsigned-integer-overflow sanitizer with clang

2021-01-06 Thread Nathan Chancellor
On Wed, Jan 06, 2021 at 10:12:51AM +0100, Arnd Bergmann wrote: > On Tue, Jan 5, 2021 at 10:25 AM Arnd Bergmann wrote: > > > > On Mon, Jan 4, 2021 at 11:33 PM Nathan Chancellor > > wrote: > > > On Wed, Dec 30, 2020 at 05:13:03PM +0100, Marco Elver wrote: > > > > On Wed, 30 Dec 2020 at 16:47, Arnd

Re: [PATCH] ubsan: disable unsigned-integer-overflow sanitizer with clang

2021-01-06 Thread Arnd Bergmann
On Tue, Jan 5, 2021 at 10:25 AM Arnd Bergmann wrote: > > On Mon, Jan 4, 2021 at 11:33 PM Nathan Chancellor > wrote: > > On Wed, Dec 30, 2020 at 05:13:03PM +0100, Marco Elver wrote: > > > On Wed, 30 Dec 2020 at 16:47, Arnd Bergmann wrote: > > > > > > > > From: Arnd Bergmann > > > > > > > > Build

Re: [PATCH] ubsan: disable unsigned-integer-overflow sanitizer with clang

2021-01-05 Thread Arnd Bergmann
On Mon, Jan 4, 2021 at 11:33 PM Nathan Chancellor wrote: > On Wed, Dec 30, 2020 at 05:13:03PM +0100, Marco Elver wrote: > > On Wed, 30 Dec 2020 at 16:47, Arnd Bergmann wrote: > > > > > > From: Arnd Bergmann > > > > > > Building ubsan kernels even for compile-testing introduced these > > > warnin

Re: [PATCH] ubsan: disable unsigned-integer-overflow sanitizer with clang

2021-01-04 Thread Nathan Chancellor
On Wed, Dec 30, 2020 at 05:13:03PM +0100, Marco Elver wrote: > On Wed, 30 Dec 2020 at 16:47, Arnd Bergmann wrote: > > > > From: Arnd Bergmann > > > > Building ubsan kernels even for compile-testing introduced these > > warnings in my randconfig environment: > > > > crypto/blake2b_generic.c:98:13:

Re: [PATCH] ubsan: disable unsigned-integer-overflow sanitizer with clang

2021-01-04 Thread Nathan Chancellor
On Mon, Jan 04, 2021 at 03:33:33PM -0800, Andrew Morton wrote: > On Mon, 4 Jan 2021 15:33:36 -0700 Nathan Chancellor > wrote: > > > > > +++ b/lib/Kconfig.ubsan > > > > @@ -122,6 +122,8 @@ config UBSAN_SIGNED_OVERFLOW > > > > > > > > config UBSAN_UNSIGNED_OVERFLOW > > > > bool "Perform c

Re: [PATCH] ubsan: disable unsigned-integer-overflow sanitizer with clang

2021-01-04 Thread Andrew Morton
On Mon, 4 Jan 2021 15:33:36 -0700 Nathan Chancellor wrote: > > > +++ b/lib/Kconfig.ubsan > > > @@ -122,6 +122,8 @@ config UBSAN_SIGNED_OVERFLOW > > > > > > config UBSAN_UNSIGNED_OVERFLOW > > > bool "Perform checking for unsigned arithmetic overflow" > > > + # clang hugely expands

Re: [PATCH] ubsan: disable unsigned-integer-overflow sanitizer with clang

2020-12-30 Thread Marco Elver
On Wed, 30 Dec 2020 at 16:47, Arnd Bergmann wrote: > > From: Arnd Bergmann > > Building ubsan kernels even for compile-testing introduced these > warnings in my randconfig environment: > > crypto/blake2b_generic.c:98:13: error: stack frame size of 9636 bytes in > function 'blake2b_compress' [-We