From mboxrd@z Thu Jan  1 00:00:00 1970
From: Ian Campbell <Ian.Campbell@citrix.com>
Subject: Re: [PATCH] docs: add vpmu description to
 xen-command-line.markdown
Date: Fri, 27 Apr 2012 10:29:55 +0100
Message-ID: <1335518995.28015.186.camel@zakaz.uk.xensource.com>
References: <95424045.BJ4a0xs4Vr@amur>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Return-path: <xen-devel-bounces@lists.xen.org>
In-Reply-To: <95424045.BJ4a0xs4Vr@amur>
List-Unsubscribe: <http://lists.xen.org/cgi-bin/mailman/options/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=unsubscribe>
List-Post: <mailto:xen-devel@lists.xen.org>
List-Help: <mailto:xen-devel-request@lists.xen.org?subject=help>
List-Subscribe: <http://lists.xen.org/cgi-bin/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=subscribe>
Sender: xen-devel-bounces@lists.xen.org
Errors-To: xen-devel-bounces@lists.xen.org
To: Dietmar Hahn <dietmar.hahn@ts.fujitsu.com>
Cc: xen-devel <xen-devel@lists.xen.org>
List-Id: xen-devel@lists.xenproject.org

On Fri, 2012-04-27 at 10:20 +0100, Dietmar Hahn wrote:
> # HG changeset patch
> # User Dietmar Hahn <dietmar.hahn@ts.fujitsu.com>
> # Date 1335517766 -7200
> # Node ID 9ba4e4a5f3b1b7dad77e2801afc0253d6d30d565
> # Parent  107285938c50f82667bd4d014820b439a077c22c
> 
> Add a short description to the vpmu boot option in the
> xen-command-line.markdown
> 
> 
> Signed-off-by:  Dietmar Hahn <dietmar.hahn@ts.fujitsu.com>

Acked-by: Ian Campbell <ian.campbell@citrix.com>

Can't we just blacklist systems with the nehalem quirk and otherwise
enable it?

Ian.

> 
> 
> diff -r 107285938c50 docs/misc/xen-command-line.markdown
> --- a/docs/misc/xen-command-line.markdown	Thu Apr 26 10:03:08 2012 +0100
> +++ b/docs/misc/xen-command-line.markdown	Fri Apr 27 11:09:26 2012 +0200
> @@ -543,7 +543,28 @@ console even after dom0 has been started
>  relinquish control to dom0.
>  
>  ### vpid
> +
>  ### vpmu
> +> `= ( bts )`
> +
> +> Default: `off`
> +
> +Switch on the virtualized performance monitoring unit for HVM guests.
> +
> +If the current cpu isn't supported a message like  
> +'VPMU: Initialization failed. ...'  
> +is printed on the hypervisor serial log.
> +
> +For some Intel Nehalem processors a quirk handling exist for an unknown
> +wrong behaviour (see handle_pmc_quirk()).
> +
> +If 'vpmu=bts' is specified the virtualisation of the Branch Trace Store (BTS)
> +feature is switched on on Intel processors supporting this feature.
> +
> +*Warning:*
> +As the BTS virtualisation is not 100% safe and because of the nehalem quirk
> +don't use the vpmu flag on production systems with Intel cpus!
> +
>  ### vti\_vhpt\_size
>  ### vti\_vtlb\_size
>  ### watchdog
> 
>