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] [xen-4.1-testing test] 9805: regressions - FAIL

To: Keir Fraser <keir.xen@xxxxxxxxx>
Subject: Re: [Xen-devel] [xen-4.1-testing test] 9805: regressions - FAIL
From: Stefan Bader <stefan.bader@xxxxxxxxxxxxx>
Date: Thu, 17 Nov 2011 10:57:48 +0100
Cc: xen-devel@xxxxxxxxxxxxxxxxxxx, Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>, Jan Beulich <JBeulich@xxxxxxxx>, Stefano Stabellini <stefano.stabellini@xxxxxxxxxxxxx>
Delivery-date: Thu, 17 Nov 2011 02:00:34 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <CAEA7F2B.250F3%keir.xen@xxxxxxxxx>
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>
References: <CAEA7F2B.250F3%keir.xen@xxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:7.0.1) Gecko/20111031 Thunderbird/7.0.1
On 17.11.2011 10:06, Keir Fraser wrote:
> On 17/11/2011 08:32, "Jan Beulich" <JBeulich@xxxxxxxx> wrote:
> 
>> This is due to a bad backport of c/s 24007:0526644ad2a6: In -unstable,
>> evtchn_unmask() must be called with d->event_lock held, while in 4.1
>> the function acquires the lock (and now gets called with the lock already
>> held from do_physdev_op()'s case PHYSDEVOP_eoi). The change dates
>> back to 23573:584c2e5e03d9, which hardly is a candidate for backporting
>> (but maybe the locking change needs to be pulled out of there).
> 
> Interestingly, Ubuntu's 4.1 fix has exactly the same problem.
> 

Hm, yes we should. I am pretty sure I hit that code path often enough, Wonder
why I never saw any dead lock there...

--Stefan

> I'll revert the patch until someone has time to actually implement and test
> a working patch against our vanilla 4.1-testing tree.
> 
>  -- Keir
> 
> 
> 
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@xxxxxxxxxxxxxxxxxxx
> http://lists.xensource.com/xen-devel


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