On Mon, Sep 03, 2018 at 02:45:31PM +0200, Jiri Kosina wrote: > +static bool __init stibp_needed(void) > +{ > + return (cpu_smt_control != CPU_SMT_NOT_SUPPORTED && > + boot_cpu_has(X86_FEATURE_STIBP)); > +}
Should that not be: ((cpu_smt_control != CPU_SMT_NOT_SUPPORTED) || (cpu_smt_control != CPU_SMT_FORCE_DISABLED)) && boot_cpu_has(X86_FEATURE_STIBP); instead?