WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

RE: [Xen-devel][PATCH] Revert Jan's patch (c/s 18879) since now itcanb

To: Jan Beulich <jbeulich@xxxxxxxxxx>
Subject: RE: [Xen-devel][PATCH] Revert Jan's patch (c/s 18879) since now itcanbe achieved by xenpm tool now
From: "Liu, Jinsong" <jinsong.liu@xxxxxxxxx>
Date: Tue, 23 Dec 2008 10:29:27 +0800
Accept-language: en-US
Acceptlanguage: en-US
Cc: "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>, Keir Fraser <keir.fraser@xxxxxxxxxxxxx>
Delivery-date: Mon, 22 Dec 2008 18:30:59 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <494F762A.76E4.0078.0@xxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References: <706158FABBBA044BAD4FE898A02E4BC21C7FD572@xxxxxxxxxxxxxxxxxxxxxxxxxxxx> <494F63F5.76E4.0078.0@xxxxxxxxxx> <706158FABBBA044BAD4FE898A02E4BC21C7FD9B7@xxxxxxxxxxxxxxxxxxxxxxxxxxxx> <494F762A.76E4.0078.0@xxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: AclkHcfZNpERRbhfRTy+IvjSVCpxEgAg83pA
Thread-topic: [Xen-devel][PATCH] Revert Jan's patch (c/s 18879) since now itcanbe achieved by xenpm tool now
Jan Beulich wrote:
>>>> "Liu, Jinsong" <jinsong.liu@xxxxxxxxx> 22.12.08 10:40 >>>
>> 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 
> 
> Which is why they are being parsed in the ondemand governor's source
> file...
> 
>> default governor is, and what control parameters can be used at grub
>> cmdline for that governor.
> 
> Anything not applicable would be ignored. I don't think there's much
> confusion associated with this.
> 
> Jan

Some cpufreq governor may fail at init stage, i.e. performance governor cannot 
start at some platform with hardware flaw.
In this case, if user select performance gov at cmdline, governor init will 
fail and then whole cpufreq init will fail, xen will have no cpufreq then.

Our idea is, first step cpufreq logic select a 'safe' governor as default, say, 
userspace governor. It will never fail, since it keep cpu freq just like it 
didn't work at init stage, so will not have any influence to perfromance, 
power, ..., etc.
Second step, user can change to any other governor as he like, if fail, cpufreq 
logic can gracefully back to the 'old-safe-governor'.
This way at least xen can ensure cpufreq logic successful, and safe change 
between different governors.

Thanks,
Jinsong

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