[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <202508250916.6424C531F@keescook>
Date: Mon, 25 Aug 2025 09:16:45 -0700
From: Kees Cook <kees@...nel.org>
To: Peter Zijlstra <peterz@...radead.org>
Cc: Sami Tolvanen <samitolvanen@...gle.com>,
David Woodhouse <dwmw2@...radead.org>,
Linus Walleij <linus.walleij@...aro.org>,
Mark Rutland <mark.rutland@....com>,
Puranjay Mohan <puranjay@...nel.org>,
Jonathan Corbet <corbet@....net>,
Nathan Chancellor <nathan@...nel.org>, x86@...nel.org,
linux-doc@...r.kernel.org, linux-kbuild@...r.kernel.org,
linux-arm-kernel@...ts.infradead.org,
linux-riscv@...ts.infradead.org, llvm@...ts.linux.dev,
linux-hardening@...r.kernel.org
Subject: Re: [PATCH 3/5] x86/cfi: Add option for cfi=debug bootparam
On Mon, Aug 25, 2025 at 05:59:23PM +0200, Peter Zijlstra wrote:
> On Mon, Aug 25, 2025 at 07:25:50AM -0700, Kees Cook wrote:
>
>
> > + if (builtin && cfi_debug)
> > + if (builtin && cfi_debug)
> > + if (builtin && cfi_debug)
> > + if (builtin && cfi_debug)
> > + if (builtin && cfi_debug)
> > + if (builtin && cfi_debug)
> > + if (builtin && cfi_debug)
> > + if (builtin && cfi_debug)
> > + if (builtin && cfi_debug)
>
> You could just make that cfi_debug and add:
>
> if (builtin)
> cfi_debug = false;
>
> at the very end of __apply_fineibt().
Oh sneaky! Yeah. Will do.
--
Kees Cook
Powered by blists - more mailing lists