linux-kernel - Re: [PATCH 2/2] x86/bugs: spectre user default must depend on MITIGATION_SPECTRE

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20241031-romantic-active-spider-5bccd8@leitao>
Date: Thu, 31 Oct 2024 02:42:12 -0700
From: Breno Leitao <leitao@...ian.org>
To: Pawan Gupta <pawan.kumar.gupta@...ux.intel.com>
Cc: bp@...en8.de, x86@...nel.org, Thomas Gleixner <tglx@...utronix.de>,
	Peter Zijlstra <peterz@...radead.org>,
	Josh Poimboeuf <jpoimboe@...nel.org>,
	Ingo Molnar <mingo@...hat.com>,
	Dave Hansen <dave.hansen@...ux.intel.com>,
	"H. Peter Anvin" <hpa@...or.com>, kernel-team@...a.com,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH 2/2] x86/bugs: spectre user default must depend on
 MITIGATION_SPECTRE_V2

Hello Pawan,

On Wed, Oct 30, 2024 at 11:40:53AM -0700, Pawan Gupta wrote:
> On Tue, Oct 29, 2024 at 02:19:12AM -0700, Breno Leitao wrote:
> > > If this is the intention it should be
> > > clearly documented that enabling kernel mitigation does not enable user
> > > mitigation. And an explicit spectre_v2_user= is required to enable user
> > > mitigation.
> > 
> > That is fair. I didn't find a place where to document about diferent
> > behavior when CONFIG_MITIGATION_X is disabled. What would you suggest?
> 
> You could describe the behavior in the commit message and update kernel
> parameter documentation.
> 
> With that:
> 
> Reviewed-by: Pawan Gupta <pawan.kumar.gupta@...ux.intel.com>
> 
> ---
> diff --git a/Documentation/admin-guide/kernel-parameters.txt b/Documentation/admin-guide/kernel-parameters.txt
> index 1518343bbe22..f8bc02cd10ec 100644
> --- a/Documentation/admin-guide/kernel-parameters.txt
> +++ b/Documentation/admin-guide/kernel-parameters.txt
> @@ -6241,6 +6241,8 @@
>  
>  			Selecting 'on' will also enable the mitigation
>  			against user space to user space task attacks.
> +			Selecting specific mitigation does not force enable
> +			user mitigations.
>  
>  			Selecting 'off' will disable both the kernel and
>  			the user space protections.

Sure, I will update it.

Thanks for the review!
--breno