[N/U][PATCH 03/11] x86/bugs: Change commas to semicolons in 'spectre_v2' sysfs file

[Andrei Gherzan]

On 24/04/11 08:24AM, Andrea Righi wrote:
> From: Josh Poimboeuf <jpoimboe at kernel.org>
> 
> BugLink: https://bugs.launchpad.net/bugs/2060909
> 
> Change the format of the 'spectre_v2' vulnerabilities sysfs file
> slightly by converting the commas to semicolons, so that mitigations for
> future variants can be grouped together and separated by commas.
> 
> Signed-off-by: Josh Poimboeuf <jpoimboe at kernel.org>
> Signed-off-by: Daniel Sneddon <daniel.sneddon at linux.intel.com>
> Signed-off-by: Thomas Gleixner <tglx at linutronix.de>

We missed the cherry-pick/backport line here. It should have been
0cd01ac5dcb1e18eb18df0f0d05b5de76522a437.

> Signed-off-by: Andrea Righi <andrea.righi at canonical.com>
> ---
>  arch/x86/kernel/cpu/bugs.c | 24 ++++++++++++------------
>  1 file changed, 12 insertions(+), 12 deletions(-)
> 
> diff --git a/arch/x86/kernel/cpu/bugs.c b/arch/x86/kernel/cpu/bugs.c
> index 01ac18f56147..119c24b47574 100644
> --- a/arch/x86/kernel/cpu/bugs.c
> +++ b/arch/x86/kernel/cpu/bugs.c
> @@ -2694,15 +2694,15 @@ static char *stibp_state(void)
>  
>  	switch (spectre_v2_user_stibp) {
>  	case SPECTRE_V2_USER_NONE:
> -		return ", STIBP: disabled";
> +		return "; STIBP: disabled";
>  	case SPECTRE_V2_USER_STRICT:
> -		return ", STIBP: forced";
> +		return "; STIBP: forced";
>  	case SPECTRE_V2_USER_STRICT_PREFERRED:
> -		return ", STIBP: always-on";
> +		return "; STIBP: always-on";
>  	case SPECTRE_V2_USER_PRCTL:
>  	case SPECTRE_V2_USER_SECCOMP:
>  		if (static_key_enabled(&switch_to_cond_stibp))
> -			return ", STIBP: conditional";
> +			return "; STIBP: conditional";
>  	}
>  	return "";
>  }
> @@ -2711,10 +2711,10 @@ static char *ibpb_state(void)
>  {
>  	if (boot_cpu_has(X86_FEATURE_IBPB)) {
>  		if (static_key_enabled(&switch_mm_always_ibpb))
> -			return ", IBPB: always-on";
> +			return "; IBPB: always-on";
>  		if (static_key_enabled(&switch_mm_cond_ibpb))
> -			return ", IBPB: conditional";
> -		return ", IBPB: disabled";
> +			return "; IBPB: conditional";
> +		return "; IBPB: disabled";
>  	}
>  	return "";
>  }
> @@ -2724,11 +2724,11 @@ static char *pbrsb_eibrs_state(void)
>  	if (boot_cpu_has_bug(X86_BUG_EIBRS_PBRSB)) {
>  		if (boot_cpu_has(X86_FEATURE_RSB_VMEXIT_LITE) ||
>  		    boot_cpu_has(X86_FEATURE_RSB_VMEXIT))
> -			return ", PBRSB-eIBRS: SW sequence";
> +			return "; PBRSB-eIBRS: SW sequence";
>  		else
> -			return ", PBRSB-eIBRS: Vulnerable";
> +			return "; PBRSB-eIBRS: Vulnerable";
>  	} else {
> -		return ", PBRSB-eIBRS: Not affected";
> +		return "; PBRSB-eIBRS: Not affected";
>  	}
>  }
>  
> @@ -2747,9 +2747,9 @@ static ssize_t spectre_v2_show_state(char *buf)
>  	return sysfs_emit(buf, "%s%s%s%s%s%s%s\n",
>  			  spectre_v2_strings[spectre_v2_enabled],
>  			  ibpb_state(),
> -			  boot_cpu_has(X86_FEATURE_USE_IBRS_FW) ? ", IBRS_FW" : "",
> +			  boot_cpu_has(X86_FEATURE_USE_IBRS_FW) ? "; IBRS_FW" : "",
>  			  stibp_state(),
> -			  boot_cpu_has(X86_FEATURE_RSB_CTXSW) ? ", RSB filling" : "",
> +			  boot_cpu_has(X86_FEATURE_RSB_CTXSW) ? "; RSB filling" : "",
>  			  pbrsb_eibrs_state(),
>  			  spectre_v2_module_string());
>  }

-- 
Andrei Gherzan
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <https://lists.ubuntu.com/archives/kernel-team/attachments/20240412/f54ad783/attachment.sig>