Re: [PATCH v5 03/10] platform/x86/intel/ifs: Add stub driver for In-Field Scan
From: Thomas Gleixner
Date: Wed May 04 2022 - 06:35:39 EST
On Thu, Apr 28 2022 at 08:38, Tony Luck wrote:
> +#define X86_MATCH(model) \
> + X86_MATCH_VENDOR_FAM_MODEL_FEATURE(INTEL, 6, \
> + INTEL_FAM6_##model, X86_FEATURE_CORE_CAPABILITIES, NULL)
> +
> +static const struct x86_cpu_id ifs_cpu_ids[] __initconst = {
> + X86_MATCH(SAPPHIRERAPIDS_X),
Why do we need a model match here? The core capabilities MSR is only
available when X86_FEATURE_CORE_CAPABILITIES is set:
"If CPUID.(EAX=07H, ECX=0):EDX[30] = 1.
This MSR provides an architectural enumeration
function for model-specific behavior."
So checking for Intel Fam6 ANYMODEL and X86_FEATURE_CORE_CAPABILITIES is
sufficient, no?
We really don't need more match id tables with gazillions of CPU models.
Thanks,
tglx