[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <aBu3BNS60PEw5Uwu@gmail.com>
Date: Wed, 7 May 2025 21:39:48 +0200
From: Ingo Molnar <mingo@...nel.org>
To: Kees Cook <kees@...nel.org>
Cc: Arnd Bergmann <arnd@...db.de>, x86@...nel.org,
"Gustavo A. R. Silva" <gustavoars@...nel.org>,
linux-doc@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
kvmarm@...ts.linux.dev, linux-riscv@...ts.infradead.org,
linux-s390@...r.kernel.org, linux-efi@...r.kernel.org,
linux-hardening@...r.kernel.org, linux-kbuild@...r.kernel.org,
linux-security-module@...r.kernel.org,
linux-kselftest@...r.kernel.org, Christoph Hellwig <hch@....de>,
Marco Elver <elver@...gle.com>,
Andrey Konovalov <andreyknvl@...il.com>,
Andrey Ryabinin <ryabinin.a.a@...il.com>,
Ard Biesheuvel <ardb@...nel.org>,
Masahiro Yamada <masahiroy@...nel.org>,
Nathan Chancellor <nathan@...nel.org>,
Nicolas Schier <nicolas.schier@...ux.dev>,
Nick Desaulniers <nick.desaulniers+lkml@...il.com>,
Bill Wendling <morbo@...gle.com>,
Justin Stitt <justinstitt@...gle.com>, linux-kernel@...r.kernel.org,
kasan-dev@...glegroups.com, sparclinux@...r.kernel.org,
llvm@...ts.linux.dev
Subject: Re: [PATCH 3/8] stackleak: Rename CONFIG_GCC_PLUGIN_STACKLEAK to
CONFIG_STACKLEAK
* Kees Cook <kees@...nel.org> wrote:
> On Wed, May 07, 2025 at 08:45:15PM +0200, Ingo Molnar wrote:
> >
> > * Kees Cook <kees@...nel.org> wrote:
> >
> > > - The STACKLEAK gcc plugin instruments the kernel code for tracking
> > > + The STACKLEAK options instruments the kernel code for tracking
> >
> > speling.
>
> Thanks!
>
> > Also, any chance to fix this terrible name? Should be something like
> > KSTACKZERO or KSTACKCLEAR, to tell people that it doesn't leak the
> > stack but prevents leaks on the stack by clearing it, and that it's
> > about the kernel stack, not any other stack.
>
> Yeah, better to name it for what it does rather than want to protects
> against. The internal naming for what it does is "stack erase", so
> perhaps KSTACK_ERASE ?
That's even better, and I like the word separation as well. Thanks!
Ingo
Powered by blists - more mailing lists