Xen 
 
Home About Xen.org Xen Xen Summit Wiki Mailing List Bug Tracker Xen Downloads
 
   
 

xen-devel

Re: [Xen-devel] [PATCH] fix event channel notification loss on IA64

To: Keir Fraser <Keir.Fraser@xxxxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] fix event channel notification loss on IA64
From: Isaku Yamahata <yamahata@xxxxxxxxxxxxx>
Date: Fri, 30 Jun 2006 10:48:20 +0900
Cc: xen-devel@xxxxxxxxxxxxxxxxxxx, xen-ia64-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Fri, 30 Jun 2006 03:45:53 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <80a283f8488c290a42b3503cd95f095f@xxxxxxxxxxxx>
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: <20060629054305.GC1406%yamahata@xxxxxxxxxxxxx> <80a283f8488c290a42b3503cd95f095f@xxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Mutt/1.4.2.1i
On Thu, Jun 29, 2006 at 02:28:37PM +0100, Keir Fraser wrote:
> 
> On 29 Jun 2006, at 06:43, Isaku Yamahata wrote:
> 
> >evtchn_do_upcall() has a micro optimization which is depends on
> >that xchg is a barrier.
> >However xchg of IA64 has acquire semantics so that event
> >channel notification is lost sometimes. This patch fixes it.
> 
> clear_bit() isn't a barrier either (at least according to Linux bitop 
> semantics). You want 'rmb();'. I suggest just putting it inside ifndef 
> CONFIG_X86, with the comment about XCHG being a barrier on x86 placed 
> just before the ifndef. Leave the zap of pending flag as a plain write.

I see. Here is the updated patch.


-- 
yamahata

Attachment: 10521:cc21edd63141_memory_barrier_evtchn_do_upcall.patch
Description: Text document

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel
<Prev in Thread] Current Thread [Next in Thread>