Re: [Xen-devel] [PATCH 0/9] Porting the intel_pstate driver to Xen

["Wang, Wei W" <wei.w.wang@xxxxxxxxx>]

On 29/04/2015 08:11, Jan Beulich wrote
> >>> "Wang, Wei W" <wei.w.wang@xxxxxxxxx> 04/28/15 3:41 PM >>>
> >Please have a check my conclusion of the changes.
> 
> These look all correct, with ...
> 
> >NO.2
> >The xenpm "get-cpufreq-para" will display the following things:
> >cpu id                            : 0
> >affected_cpus           : 0
> >cpuinfo frequency   : max [3700000] min [1200000] cur [1200000]
> >scaling_driver            : intel_pstate
> >scaling_avail_gov     : Performance Powersave Ondemand Userspace
> >current_governor    : Ondemand
> 
> ... the last two lines being kind of pointless, and hence perhaps omitting
> them in the intel_pstate case would be better. Bus as said while talking to
> you, I'm not insisting on that (and also couldn't really, not being the
> maintainer of that code).

Now, we still keep the "set-scaling-governor" option to let the user tune 
performance via selecting a governor, so I think we should let them know which 
governor mode the CPU is on. In our talk, we assumed there is a 
"get-scaling-governor" option to achieve so, but that option doesn't exist. So 
I think it's better to keep this piece of info in the "get-cpufreq-para" here.

Best,
Wei

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel