On 03.05.2021 21:28, Jason Andryuk wrote: > --- a/tools/libs/ctrl/xc_pm.c > +++ b/tools/libs/ctrl/xc_pm.c > @@ -330,6 +330,24 @@ int xc_set_cpufreq_para(xc_interface *xch, int cpuid, > return xc_sysctl(xch, &sysctl); > } > > +int xc_set_cpufreq_hwp(xc_interface *xch, int cpuid, > + xc_set_hwp_para_t *set_hwp)
Besides for general considerations, for xenpm to legitimately pass the same struct instance into this function multiple times, the last parameter wants to be pointer-to-const, declaring the intent of the function to leave the struct unaltered. Jan
