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] evtchn_do_upcall: search a snapshot of level 2

To: Kaushik Kumar Ram <kaushik@xxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] evtchn_do_upcall: search a snapshot of level 2 bits for pending upcalls
From: Keir Fraser <keir.fraser@xxxxxxxxxxxxx>
Date: Wed, 03 Feb 2010 09:32:26 +0000
Cc: "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Wed, 03 Feb 2010 01:33:01 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <9FD43D8F-4A4C-456F-8E84-78F39040E2F7@xxxxxxxx>
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/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: AcqjdCasIZlZvmu0SeqPIChRvz6xYQBP6UzW
Thread-topic: [Xen-devel] [PATCH] evtchn_do_upcall: search a snapshot of level 2 bits for pending upcalls
User-agent: Microsoft-Entourage/12.23.0.091001
On 01/02/2010 19:24, "Kaushik Kumar Ram" <kaushik@xxxxxxxx> wrote:

>> Somehow the patch is mangled and does not apply. But also it is very big.
>> How about the attached patch instead? I think it does all we agreed on, and
>> is much smaller.
> 
> Well, I tried really hard to avoid the extra checks inside the loop.
> Maybe it doesn't matter.
> 
> There is one small issue in your patch, otherwise it is fine. When start_l1i
> is not set, l2i has to be made zero. Otherwise, we would start halfway through
> the next l2 whose l1i is set. I think this issue exists even in the original
> code.

Thanks Kaushik! I fixed that and checked in as linux-2.6.18-xen:990, which
will soon appear in the main public repo.

 -- Keir



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