Re: [PATCH v2 0/2] xentrace: Add TRC_HVM_VCHIP (was xentrace: Add TRC_HW_VCHIP).

[George Dunlap <george.dunlap@eu.citrix.com>]

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(+)
>