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?

Reply via email to