On Sat, Nov 30, 2024 at 04:31:25PM +0100, Borislav Petkov wrote:
> On Thu, Nov 21, 2024 at 12:07:18PM -0800, Josh Poimboeuf wrote:
> > --- a/arch/x86/kernel/cpu/bugs.c
> > +++ b/arch/x86/kernel/cpu/bugs.c
> > @@ -1605,20 +1605,20 @@ static void __init spectre_v2_determine_rsb_fill_type_at_vmexit(enum spectre_v2_
> > case SPECTRE_V2_NONE:
> > return;
> >
> > - case SPECTRE_V2_EIBRS_LFENCE:
> > case SPECTRE_V2_EIBRS:
> > + case SPECTRE_V2_EIBRS_LFENCE:
> > + case SPECTRE_V2_EIBRS_RETPOLINE:
> > if (boot_cpu_has_bug(X86_BUG_EIBRS_PBRSB)) {
> > - setup_force_cpu_cap(X86_FEATURE_RSB_VMEXIT_LITE);
> > pr_info("Spectre v2 / PBRSB-eIBRS: Retire a single CALL on VMEXIT\n");
> > + setup_force_cpu_cap(X86_FEATURE_RSB_VMEXIT_LITE);
>
> Why are you swapping those?
>
> > }
> > return;
> >
> > - case SPECTRE_V2_EIBRS_RETPOLINE:
> > case SPECTRE_V2_RETPOLINE:
> > case SPECTRE_V2_LFENCE:
> > case SPECTRE_V2_IBRS:
> > - setup_force_cpu_cap(X86_FEATURE_RSB_VMEXIT);
> > pr_info("Spectre v2 / SpectreRSB : Filling RSB on VMEXIT\n");
> > + setup_force_cpu_cap(X86_FEATURE_RSB_VMEXIT);
>
> Ditto?
It's more readable that way, similar to how a comment goes before code.
--
Josh
