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-ia64-devel]PATCH] Remove duplicate check is_running_on_xen

To: "Alex Williamson" <alex.williamson@xxxxxx>
Subject: RE: [Xen-ia64-devel]PATCH] Remove duplicate check is_running_on_xen
From: "Xu, Anthony" <anthony.xu@xxxxxxxxx>
Date: Wed, 17 Jan 2007 11:13:02 +0800
Cc: xen-ia64-devel <xen-ia64-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Tue, 16 Jan 2007 19:12:33 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <1169003412.5469.46.camel@bling>
List-help: <mailto:xen-ia64-devel-request@lists.xensource.com?subject=help>
List-id: Discussion of the ia64 port of Xen <xen-ia64-devel.lists.xensource.com>
List-post: <mailto:xen-ia64-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-ia64-devel>, <mailto:xen-ia64-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-ia64-devel>, <mailto:xen-ia64-devel-request@lists.xensource.com?subject=unsubscribe>
Sender: xen-ia64-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: Acc55QhcCtdpNrq+TB6O++qI7oyhxAAAESFQ
Thread-topic: [Xen-ia64-devel]PATCH] Remove duplicate check is_running_on_xen
Alex Williamson write on 2007年1月17日 11:10:
> On Wed, 2007-01-17 at 10:17 +0800, Xu, Anthony wrote:
> 
>> There are two duplicate checks as below, so there are two way to
>> eliminate one check. 
>> 
>> 1. eliminate check in xen_get_tpr, then xen_get_tpr will never be
>>      called on bare metal, but it's a little perfomance impact, due
>> to there is another switch inside __ia64_getreg. 
>> 
>> 2. eliminate check in xen_ia64_getreg, then xen_get_tpr will be
>>    called both on para platform or bare metal, it can produce better
>> performance than option 1. so I prefer this. 
> 
>    Is the performance difference actually measurable?  I suspect it's
> effectively within the noise of any benchmark.  I prefer option 1
> because the leaf function when running on bare metal is the same as
> the native kernel, so we're not duplicating code.  Thanks,

Okey, I will do this.

Anthony
> 
>       Alex

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