Re: [PATCH] x86: Prefer MWAIT over HALT on AMD processors

[Dave Hansen]

On 4/5/22 08:34, Limonciello, Mario wrote:
>>>         if (!static_cpu_has(X86_FEATURE_ZEN)) {
>>>                 msr |= ssbd_tif_to_amd_ls_cfg(tifn);
>>>                 wrmsrl(MSR_AMD64_LS_CFG, msr);
>>>                 return;
>>>         }
>> This seem _bit_ at odds with the commit message (and the AMD SSBD
>> whitepaper):
>>
>>>     Add the necessary synchronization logic for AMD family 17H.
>> So, is X86_FEATURE_ZEN for family==0x17, or family>=0x17?
> There are Zen family CPUs and APUs from family 0x19.  Perhaps at the
> time of the whitepaper there weren't yet though.

Is this a gap in the documentation, then?  Is there some documentation
of the availability of SSBD mitigations on family 0x19?

Anyway, back to MWAIT...  It would be really great to include the
assumptions about what X86_FEATURE_ZEN means in the context of this
patch.  Does this patch *really* mean "Zen microarchitecture" only?