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] [PATCH] add support for XCHG instruction accessingAPIC

To: "Jiang, Yunhong" <yunhong.jiang@xxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] add support for XCHG instruction accessingAPIC
From: Keir Fraser <Keir.Fraser@xxxxxxxxxxxx>
Date: Wed, 5 Apr 2006 19:06:53 +0100
Cc: xen-devel@xxxxxxxxxxxxxxxxxxx, Boris Ostrovsky <bostrovsky@xxxxxxxxxxxxxxx>
Delivery-date: Wed, 05 Apr 2006 11:06:39 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <FFEFE1749526634699CD3AC2EDB7627A017A7F0B@pdsmsx406>
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>
References: <FFEFE1749526634699CD3AC2EDB7627A017A7F0B@pdsmsx406>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx

On 5 Apr 2006, at 17:50, Jiang, Yunhong wrote:

I think the reason we don't take any lock here is; we are sure this is just used for local APIC range. However, if in future someone adds another MMIO range or if IOAPIC is accessed with XCHG (will this happen?), this may have potential issue.

So I'd suggest adding comments that we didn't take any lock here, or add a check to make sure the range is on local APIC range.

There is a comment in the patch I applied, and that section of code is specific to LAPIC handling.

 -- Keir


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

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