RE: [Xen-devel][PATCH] Revert Jan's patch (c/s 18879) since now it canbe achieved by xenpm tool now

["Liu, Jinsong" <jinsong.liu@xxxxxxxxx>]

Jan Beulich wrote:
>>>> "Liu, Jinsong" <jinsong.liu@xxxxxxxxx> 20.12.08 14:37 >>>
>> Revert Jan's patch (c/s 18879) since now it can be achieved by
>> xenpm tool now.
> 
> Please don't, and rather (as Keir suggested) add an option to also
> select the initial governor on the command line. While the xenpm tool
> is nice, older distro-s won't run it by default, and having to fiddle
> with the system startup scripts or running it manually (which
> wouldn't necessarily work if you don't do a full install of the Xen
> tool - e.g. to keep the distro's tools intact) isn't really desirable
> in certain cases. 
> 
> Jan

Jan,

It's good for old distro-s user to use cmdline to set parameter, but I have 
some concern, since in fact cpufreq have 5 governors, with ~20 status 
parameters and ~10 control parameters. If we add option at grub cmdline to 
select initial governor, and add further options to set control parameters, it 
may make confuse for user, especially considered that some of the control 
parameters are governor-dependent (i.e. in c/s 18879, the control para 'sample 
rate' and 'upthreshold' can only be used for ondemand or conservative 
governor). User may confuse what the default governor is, and what control 
parameters can be used at grub cmdline for that governor.

Thanks,
Jinsong
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel