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] Recent trace patch not arch-neutral

To: "Keir Fraser" <Keir.Fraser@xxxxxxxxxxxx>, "Gardner, Robert D" <rob.gardner@xxxxxx>
Subject: RE: [Xen-devel] Recent trace patch not arch-neutral
From: "Magenheimer, Dan (HP Labs Fort Collins)" <dan.magenheimer@xxxxxx>
Date: Tue, 1 Nov 2005 05:25:28 -0800
Cc: Ian Pratt <m+Ian.Pratt@xxxxxxxxxxxx>, xen-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Tue, 01 Nov 2005 13:23:17 +0000
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
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/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: AcXexgiT+9pUS19qSAm296WDyo/oQwAHxeog
Thread-topic: [Xen-devel] Recent trace patch not arch-neutral
How about just adding "#include <asm/timex.h>" and
stealing linux/include/asm-{i386,x86_64}/timex.h?
Xen/ia64 is already including asm-ia64/timex.h
in other code.

> -----Original Message-----
> From: Keir Fraser [mailto:Keir.Fraser@xxxxxxxxxxxx] 
> Sent: Tuesday, November 01, 2005 2:28 AM
> To: Gardner, Robert D
> Cc: xen-devel@xxxxxxxxxxxxxxxxxxx; Magenheimer, Dan (HP Labs 
> Fort Collins); Ian Pratt
> Subject: Re: [Xen-devel] Recent trace patch not arch-neutral
> 
> 
> >  #ifdef x86
> >         rdtscll(rec->cycles);
> >  #endif
> >  #ifdef IA64
> >         __asm__ __volatile ("mov %0=ar.itc;;" : 
> "=r"(rec->cycles) :: 
> > "memory");
> >  #endif
> >
> >  Dan, perhaps you know the nice clean way of doing this 
> sort of thing?
> 
> Okay, let's define the Linux function get_cycles() in 
> include/asm/time.h. So common code can then get at that generic 
> function by including <xen/time.h>. I don't much care for the 
> cycles_t 
> definition but I guess we may as well pull that in as well. Then 
> trace.c can cast to u64 when writing the stamp into the trace record.
> 
>   -- Keir
> 
> 

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

<Prev in Thread] Current Thread [Next in Thread>