Re: [PATCH v11 4/9] x86/alternatives: Disable LASS when patching kernel code
From: Dave Hansen
Date: Fri Oct 31 2025 - 13:10:54 EST
On 10/29/25 14:03, Sohil Mehta wrote:
> Introduce LASS-specific STAC/CLAC helpers to set the AC bit only on
> platforms that need it. Clarify the usage of the new helpers versus the
> existing stac()/clac() helpers for SMAP.
Reviewed-by: Dave Hansen <dave.hansen@xxxxxxxxxxxxxxx>
One review nit: The
- /* Note: a barrier is implicit in alternative() */
looks a bit funky in the diffstat. It took me a minute to realize that
you'd moved it. I _probably_ would have specifically called out that you
*added* comments for stac()/clac() and moved and existing duplicated
comment there. Adding a whole new comment block deserves calling out
explicitly. It is far beyond the "clarify" that's in the changelog.
But it's just a nit in the end.