From mboxrd@z Thu Jan 1 00:00:00 1970 From: Keir Fraser Subject: Re: [xen-4.1-testing test] 9805: regressions - FAIL Date: Thu, 17 Nov 2011 10:13:10 +0000 Message-ID: References: <4EC4DA9C.60600@canonical.com> Mime-Version: 1.0 Content-Type: text/plain; charset="US-ASCII" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <4EC4DA9C.60600@canonical.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: Stefan Bader Cc: xen-devel@lists.xensource.com, Ian Jackson , Jan Beulich , Stefano Stabellini List-Id: xen-devel@lists.xenproject.org On 17/11/2011 09:57, "Stefan Bader" 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... Perhaps your dom0 kernel doesn't register a pirq_eoi_map. -- Keir