[Public]

> -----Original Message-----
> From: Jan Beulich <jbeul...@suse.com>
> Sent: Thursday, August 28, 2025 7:07 PM
> To: Penny, Zheng <penny.zh...@amd.com>
> Cc: Huang, Ray <ray.hu...@amd.com>; Anthony PERARD
> <anthony.per...@vates.tech>; Andrew Cooper <andrew.coop...@citrix.com>;
> Roger Pau Monné <roger....@citrix.com>; xen-devel@lists.xenproject.org
> Subject: Re: [PATCH v8 8/8] xen/cpufreq: Adapt SET/GET_CPUFREQ_CPPC
> xen_sysctl_pm_op for amd-cppc driver
>
> 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.
>

HWP has its own unique "hwp" governor. So, imo, it may not affect.

> Jan

Reply via email to