Re: [PATCH v1] cpufreq: intel_pstate: Set non-turbo capacity to HWP_GUARANTEED_PERF()

[Ricardo Neri]

On Tue, Jul 07, 2026 at 07:19:55PM +0200, Rafael J. Wysocki wrote:
> From: Rafael J. Wysocki <rafael.j.wysocki@xxxxxxxxx>
> 
> Setting cpu->capacity_perf to cpu->pstate.max_pstate_physical in the
> "no turbo" case is inconsistent with what happens elsewhere in the
> driver and causes arch_scale_cpu_capacity() to be incorrect.  It also
> skews arch_scale_freq_capacity() which ends up differing from 1024 for
> the guaranteed P-state.
> 
> Address that by setting capacity_perf to HWP_GUARANTEED_PERF() in the
> "no turbo" case.

Indeed, using cpu->pstate.max_pstate_physical does not make sense. It only
reflects what PLATFORM_INFO[15:8] says for _all_ CPUs, regardless of type,
resized by cpu->pstate.scaling/cpu->pstate.perf_ctl_scaling. No relation
to any property of the CPU.

Now arch_scale_freq_capacity() looks correct.

> 
> Fixes: 929ebc93ccaa ("cpufreq: intel_pstate: Set asymmetric CPU capacity on hybrid systems")
> Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@xxxxxxxxx>

Tested-by Ricardo Neri <ricardo.neri-calderon@xxxxxxxxxxxxxxx>