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] xen: do not unmask disabled IRQ on eoi.

To: "Ian Campbell" <Ian.Campbell@xxxxxxxxxxxxx>, "Stefano Stabellini" <stefano.stabellini@xxxxxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] xen: do not unmask disabled IRQ on eoi.
From: "Jan Beulich" <JBeulich@xxxxxxxxxx>
Date: Fri, 22 Oct 2010 15:48:33 +0100
Cc: "jeremy@xxxxxxxx" <jeremy@xxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Fri, 22 Oct 2010 08:10:15 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <alpine.DEB.2.00.1010221244370.10348@kaball-desktop>
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: <1287139966-19391-2-git-send-email-ian.campbell@xxxxxxxxxx> <alpine.DEB.2.00.1010151614420.2423@kaball-desktop> <1287160335.2003.7973.camel@xxxxxxxxxxxxxxxxxxxxxx> <1287160787.2003.7979.camel@xxxxxxxxxxxxxxxxxxxxxx> <alpine.DEB.2.00.1010151757240.2423@kaball-desktop> <4CBC1CF1020000780001DA3F@xxxxxxxxxxxxxxxxxx> <alpine.DEB.2.00.1010181530200.2423@kaball-desktop> <4CBC80E6020000780001DC78@xxxxxxxxxxxxxxxxxx> <alpine.DEB.2.00.1010181754180.2423@kaball-desktop> <4CBD5883020000780001DEC4@xxxxxxxxxxxxxxxxxx> <alpine.DEB.2.00.1010211411100.10348@kaball-desktop> <4CC15C5B020000780001E8B2@xxxxxxxxxxxxxxxxxx> <1287734836.29224.648.camel@xxxxxxxxxxxxxxxxxxxxx> <4CC16797020000780001E8F7@xxxxxxxxxxxxxxxxxx> <1287736466.11851.5609.camel@xxxxxxxxxxxxxxxxxxxxxx> <alpine.DEB.2.00.1010221244370.10348@kaball-desktop>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
>>> On 22.10.10 at 15:57, Stefano Stabellini <stefano.stabellini@xxxxxxxxxxxxx> 
>>> wrote:
> In none of these cases the evtchn would be masked for sure, so I'll have
> to test if the evtchn is masked and call move_masked_irq or
> move_native_irq accordingly.  I preferred to test the evtchn_mask
> directly as opposed to the irq flags because the test is easier to
> understand (we could arrive in eoi_pirq from both handle_edge_irq or
> handle_fasteoi_irq, the latter doesn't set IRQ_MASKED when masks
> an irq).

But you still didn't add and IRQ_DISABLED check around the call
to move_masked_irq() - do you have any particular reason not to?

Jan


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