From: Wei Liu Subject: Re: [PATCH v6 5/6] x86/intel_pstate: support the use of intel_pstate in pmstat.c Date: Wed, 28 Oct 2015 11:18:59 +0000 Message-ID: <20151028111859.GC18674@zion.uk.xensource.com> References: <1446002478-16105-1-git-send-email-wei.w.wang@intel.com> <1446002478-16105-6-git-send-email-wei.w.wang@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <1446002478-16105-6-git-send-email-wei.w.wang@intel.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Wei Wang Cc: andrew.cooper3@citrix.com, wei.liu2@citrix.com, jbeulich@suse.com, xen-devel@lists.xen.org List-Id: xen-devel@lists.xenproject.org On Wed, Oct 28, 2015 at 11:21:17AM +0800, Wei Wang wrote: > Add support in the pmstat.c so that the xenpm tool can request to > access the intel_pstate driver. > > Signed-off-by: Wei Wang > --- > changes in v6: > 1) add the NON_INTERNAL_GOV macro to replace literal 0; > 2) code consolidation (e.g. merging some code into if/else, as required in v5); > 3) somewhere, change to use clamp, instead of clamp_t; > 4) xen_perf_alias, instead of perf_alias. > > tools/libxc/include/xenctrl.h | 20 ++-- > tools/libxc/xc_pm.c | 16 ++-- > tools/misc/xenpm.c | 4 +- I think all changes to toolstack code are merely trying to mirror hypervisor side changes and there are no functional changes. With that understanding and subject to an ack from hypervisor side maintainer: Acked-by: Wei Liu