On 28.08.2025 12:06, Penny Zheng wrote:
> @@ -154,6 +156,17 @@ static int get_cpufreq_para(struct xen_sysctl_pm_op *op)
> else
> strlcpy(op->u.get_para.scaling_driver, "Unknown", CPUFREQ_NAME_LEN);
>
> + /*
> + * In CPPC active mode, we are borrowing governor field to indicate
> + * policy info.
> + */
> + if ( policy->governor->name[0] )
> + strlcpy(op->u.get_para.u.s.scaling_governor,
> + policy->governor->name, CPUFREQ_NAME_LEN);
> + else
> + strlcpy(op->u.get_para.u.s.scaling_governor, "Unknown",
> + CPUFREQ_NAME_LEN);
Isn't pulling this ...
> if ( !cpufreq_is_governorless(op->cpuid) )
> {
> if ( !(scaling_available_governors =
... out of this if()'s body going to affect HWP? It's not clear to me whether
that would be entirely benign.
Jan
