From mboxrd@z Thu Jan  1 00:00:00 1970
From: "Jan Beulich" <jbeulich@novell.com>
Subject: Re: [PATCH] Revert Jan's patch (c/s 18879) since
	now it canbe achieved by xenpm tool now
Date: Mon, 22 Dec 2008 08:55:01 +0000
Message-ID: <494F63F5.76E4.0078.0@novell.com>
References: <706158FABBBA044BAD4FE898A02E4BC21C7FD572@pdsmsx503.ccr.corp.intel.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: quoted-printable
Return-path: <xen-devel-bounces@lists.xensource.com>
In-Reply-To: <706158FABBBA044BAD4FE898A02E4BC21C7FD572@pdsmsx503.ccr.corp.intel.com>
Content-Disposition: inline
List-Unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
List-Post: <mailto:xen-devel@lists.xensource.com>
List-Help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-Subscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
Sender: xen-devel-bounces@lists.xensource.com
Errors-To: xen-devel-bounces@lists.xensource.com
To: Jinsong Liu <jinsong.liu@intel.com>
Cc: "xen-devel@lists.xensource.com" <xen-devel@lists.xensource.com>, Keir Fraser <keir.fraser@eu.citrix.com>
List-Id: xen-devel@lists.xenproject.org

>>> "Liu, Jinsong" <jinsong.liu@intel.com> 20.12.08 14:37 >>>
>Revert Jan's patch (c/s 18879) since now it can be achieved by=20
>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