From mboxrd@z Thu Jan 1 00:00:00 1970 From: George Dunlap Subject: Re: [PATCH v2 0/2] xentrace: Add TRC_HVM_VCHIP (was xentrace: Add TRC_HW_VCHIP). Date: Tue, 1 Apr 2014 14:06:16 +0100 Message-ID: <533AB9C8.309@eu.citrix.com> References: <1396025767-3430-1-git-send-email-dslutz@verizon.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii"; Format="flowed" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1396025767-3430-1-git-send-email-dslutz@verizon.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: Don Slutz , xen-devel@lists.xen.org, Jan Beulich , Tim Deegan Cc: Keir Fraser , Ian Jackson , Ian Campbell , Stefano Stabellini List-Id: xen-devel@lists.xenproject.org On 03/28/2014 04:56 PM, Don Slutz wrote: > Changes v1 to v2: > Tim Deegan: > Switch from TRC_HW_VCHIP to TRC_HVM_VCHIP. > Drop get_cycles() > Fix coding style issues. > Drop code that should not have been included. > Use names not numbers. > > Split in 2 patches for simpler review. > > Here is a subset of sample output: Are you aware of xenalyze? It's got a lot of advantages over xentrace_format; the most basic of which is that it traces vcpus as they run across pcpus. hg clone http://xenbits.xenproject.org/ext/xenalyze -George > > dcs-xen-54:~/xen>grep hpet /home/don/zz-xentrace-3.out > CPU5 841028423232 (+ 3900) hpet create [ tn = 0, irq = 0, delta = 848080, period = 1000000 ] > CPU1 843516134148 (+ 20796) hpet destroy [ tn = 0 ] > CPU7 846913412784 (+ 26184) hpet destroy [ tn = 0 ] > CPU7 846913416102 (+ 1776) hpet create [ tn = 0, irq = 0, delta = 4170764048, period = 0 ] > CPU4 851697560919 (+ 11499) hpet destroy [ tn = 1 ] > CPU4 851697562041 (+ 1122) hpet create [ tn = 1, irq = 8, delta = 15605920, period = 0 ] > CPU4 851765523993 (+ 12567) hpet destroy [ tn = 1 ] > CPU5 1139751965382 (+ 18624) hpet create [ tn = 1, irq = 8, delta = 15598912, period = 0 ] > CPU5 1139789579136 (+ 23088) hpet destroy [ tn = 1 ] > CPU5 1139789581410 (+ 2274) hpet create [ tn = 1, irq = 8, delta = 15550992, period = 0 ] > CPU5 1139827080720 (+ 24174) hpet destroy [ tn = 1 ] > CPU5 1139827082718 (+ 1998) hpet create [ tn = 1, irq = 8, delta = 15550912, period = 0 ] > CPU5 1139864572410 (+ 21198) hpet destroy [ tn = 1 ] > CPU5 1139864574390 (+ 1980) hpet create [ tn = 1, irq = 8, delta = 15554912, period = 0 ] > CPU5 1139902073244 (+ 21180) hpet destroy [ tn = 1 ] > CPU5 1139902075296 (+ 2052) hpet create [ tn = 1, irq = 8, delta = 15555056, period = 0 ] > CPU5 1139939593674 (+ 29112) hpet destroy [ tn = 1 ] > CPU5 1139939595870 (+ 2196) hpet create [ tn = 1, irq = 8, delta = 15546992, period = 0 ] > CPU5 1139977075716 (+ 21048) hpet destroy [ tn = 1 ] > CPU5 1139977077642 (+ 1926) hpet create [ tn = 1, irq = 8, delta = 15555104, period = 0 ] > CPU5 1140014582832 (+ 23088) hpet destroy [ tn = 1 ] > CPU5 1140014584950 (+ 2118) hpet create [ tn = 1, irq = 8, delta = 15552576, period = 0 ] > CPU5 1140038873628 (+ 21150) hpet destroy [ tn = 1 ] > CPU5 1140038875890 (+ 2262) hpet create [ tn = 1, irq = 8, delta = 21056672, period = 0 ] > CPU5 1140039140484 (+ 18348) hpet destroy [ tn = 1 ] > CPU6 1142224601952 (+ 12486) hpet destroy [ tn = 0 ] > > > > Don Slutz (2): > xentrace: Add emacs local variables > xentrace: Add TRC_HVM_VCHIP > > tools/xentrace/formats | 18 ++++++++++++++++++ > xen/arch/x86/hvm/hpet.c | 14 ++++++++++++++ > xen/arch/x86/hvm/i8254.c | 18 ++++++++++++++++++ > xen/arch/x86/hvm/rtc.c | 18 ++++++++++++++++++ > xen/arch/x86/hvm/vlapic.c | 25 +++++++++++++++++++++++++ > xen/arch/x86/hvm/vpic.c | 20 ++++++++++++++++++++ > xen/include/public/trace.h | 20 ++++++++++++++++++++ > 7 files changed, 133 insertions(+) >