Re: [PATCH v2 06/17] cpufreq: Fix the efficient idle check for Intel extended Families
From: Sohil Mehta
Date: Thu Feb 13 2025 - 13:50:41 EST
On 2/11/2025 9:35 PM, Zhang, Rui wrote:
>> static int should_io_be_busy(void)
>> {
>> #if defined(CONFIG_X86)
>> /*
>> - * For Intel, Core 2 (model 15) and later have an efficient
>> idle.
>> + * Starting with Family 6 consider all Intel CPUs to have an
>> + * efficient idle.
>> */
>> if (boot_cpu_data.x86_vendor == X86_VENDOR_INTEL &&
>> - boot_cpu_data.x86 == 6 &&
>> - boot_cpu_data.x86_model >= 15)
>> + boot_cpu_data.x86_vfm >= INTEL_PENTIUM_PRO)
>
> This is "Starting from P4" rather than "Starting from Family 6", right?
>
As described in the commit message, we are extending this to all
relevant Intel processors. That would include Family 6, Family 15 and
the upcoming Family > 15 processors as well.
A VFM check starting at INTEL_PENTIUM_PRO (Family 6, Model 1) is just a
way to simplify that.