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] Re: [PATCH] add long interrupt measurement capability

To: Dan Magenheimer <dan.magenheimer@xxxxxxxxxx>, Dulloor <dulloor@xxxxxxxxx>, Keir Fraser <Keir.Fraser@xxxxxxxxxxxxx>
Subject: RE: [Xen-devel] Re: [PATCH] add long interrupt measurement capability
From: Ian Pratt <Ian.Pratt@xxxxxxxxxxxxx>
Date: Fri, 8 May 2009 22:53:34 +0100
Accept-language: en-US
Acceptlanguage: en-US
Cc: Ian Pratt <Ian.Pratt@xxxxxxxxxxxxx>, "Xen-Devel \(E-mail\)" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Fri, 08 May 2009 14:54:02 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <36c8db51-1796-4f70-9453-babf1f285762@default>
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: <940bcfd20905071714j6eb23e7g3a89854d473df40a@xxxxxxxxxxxxxx> <36c8db51-1796-4f70-9453-babf1f285762@default>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: AcnQJTWUPFXimfVuTqSgp1PQjogQzAAAE0TA
Thread-topic: [Xen-devel] Re: [PATCH] add long interrupt measurement capability
> Perhaps.  However measuring cycles is important and, more
> specifically, measuring MAX cycles spent across a set of
> interrupts.  As a result, I suspect any code that measures
> this (regardless of whether the result is reported by
> xentrace or debug-key) would likely encounter the
> same objection from Keir.

I can't imagine there'd be any objection to adding trace macros to record this. 
The xentrace log processing tool can then be updated to generate max or 
histogram values.

Ian 

> -----Original Message-----
> From: Dulloor [mailto:dulloor@xxxxxxxxx]
> Sent: Thursday, May 07, 2009 6:14 PM
> To: Keir Fraser
> Cc: Dan Magenheimer; Xen-Devel (E-mail)
> Subject: Re: [Xen-devel] Re: [PATCH] add long interrupt measurement
> capability
> 
> 
> Dan, Couldn't you add this to xentrace ?
> 
> -dulloor
> 
> 
> On Wed, Apr 29, 2009 at 2:03 PM, Keir Fraser
> <keir.fraser@xxxxxxxxxxxxx> wrote:
> 
> On 29/04/2009 17:50, "Dan Magenheimer" <dan.magenheimer@xxxxxxxxxx>
> wrote:
> 
> > Since I see you are still taking boot-option patches
> > for 3.4, I thought I'd retry this one which optionally
> > measures cycles spent in each irq.  I thought
> > you had just decided it was too late for 3.4... if
> > you rejected it for another reason, please let me
> > know.
> 
> 
> I don't think it has sufficient general utility to be applied. Anyone
> who
> would actually have the want and wit to run this option can apply the
> patch
> themselves.
> 
>  -- Keir
> 
> 
> 
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@xxxxxxxxxxxxxxxxxxx
> http://lists.xensource.com/xen-devel
> 
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@xxxxxxxxxxxxxxxxxxx
> http://lists.xensource.com/xen-devel

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