From mboxrd@z Thu Jan 1 00:00:00 1970 From: George Dunlap Subject: Re: [PATCH] xentrace: add a tool to break down the result of vmexit Date: Fri, 9 Aug 2013 10:10:19 +0100 Message-ID: <5204B1FB.6020609@eu.citrix.com> References: <1376030098-18037-1-git-send-email-yang.z.zhang@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii"; Format="flowed" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: "Zhang, Yang Z" Cc: Ian Jackson , "xen-devel@lists.xensource.com" , Ian Campbell List-Id: xen-devel@lists.xenproject.org On 09/08/13 09:54, Zhang, Yang Z wrote: > Zhang, Yang Z wrote on 2013-08-09: >> George Dunlap wrote on 2013-08-09: >>> On Fri, Aug 9, 2013 at 7:34 AM, Yang Zhang >> wrote: >>>> From: Yang Zhang >>>> >>>> The tool is able to provide a summary of vmexit. Currently, it only >>>> supports to summay one VCPU result at a time. >>> Have you seen xenalyze? >>> >>> hg clone http://xenbits.xen.org/ext/xenalyze >> Thanks. Now I see it. It's a great tool. > Does it integrate to Xen source? If no, it's better to add it to tool/xentrace/ instead use a separate repo to let more people know it. :) Yes, this has been suggested before. The original reason for having it be a separate repo is that most of the xenalyze development happened after a Xen release. But I think it has more or less reached a 1.0 level some time ago, and isn't seeing as many changes. I'm not sure what it would take to get it checked into the main repo. At the moment the code is just one massive file (with a couple of helper files), and no doubt has a number of coding style inconsistencies - though I there is certainly worse code in the tree. :-) Ian / Ian, any thoughts? -George