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-ia64-devel

RE: [Xen-devel] [PATCH 4/5 TAKE 2] xenoprof: make linux xenoprof codearc

To: "Isaku Yamahata" <yamahata@xxxxxxxxxxxxx>, <xen-devel@xxxxxxxxxxxxxxxxxxx>
Subject: RE: [Xen-devel] [PATCH 4/5 TAKE 2] xenoprof: make linux xenoprof codearch-generic
From: "Santos, Jose Renato G" <joserenato.santos@xxxxxx>
Date: Mon, 20 Nov 2006 12:38:51 -0600
Cc: xen-ia64-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Mon, 20 Nov 2006 10:39:38 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <20061118042150.GB2090%yamahata@xxxxxxxxxxxxx>
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: AccKyT1IkRCGKncmSdGrq/ihIkVodwCCDzWA
Thread-topic: [Xen-devel] [PATCH 4/5 TAKE 2] xenoprof: make linux xenoprof codearch-generic
 

> -----Original Message-----
> From: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx 
> [mailto:xen-devel-bounces@xxxxxxxxxxxxxxxxxxx] On Behalf Of 
> Isaku Yamahata
> Sent: Friday, November 17, 2006 8:22 PM
> To: xen-devel@xxxxxxxxxxxxxxxxxxx
> Cc: xen-ia64-devel@xxxxxxxxxxxxxxxxxxx
> Subject: Re: [Xen-devel] [PATCH 4/5 TAKE 2] xenoprof: make 
> linux xenoprof codearch-generic
> 
> 
> make xenoprof of linux side arch generic with some bug fixes.
> 
> Changes:
> - support modular oprofile
> - move xenoprof_is_primary from xenoprofile.c. 
>   It is referenced by perfmon which isn't module even when xenoprof
>   is module.
> 

   Why perfmon needs to access Oprofile? It should be the other way
around ...
   It seems that the only change was to create a new file named
xenoprof_is_primary.c to hold the definition of xenoprof_is_primary
   I don't like this. And I don't understand why you need this. You
should deal with IA64 specifics in arch specific code as much as
possible. Could you please find an IA64 specific solution or explain why
this is needed in the generic part? 

  Also, if sending another patch please send me only the changes from
the initial 5 patches you sent me. I would like to see only the changes
...
  (do not use this last patch as reference, I am ignoring it)
  Thanks

  Renato

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

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