From mboxrd@z Thu Jan 1 00:00:00 1970 From: Boris Ostrovsky Subject: Re: [PATCHv1] xen/events/fifo: Handle linked events when closing a PIRQ port Date: Mon, 10 Aug 2015 12:09:50 -0400 Message-ID: <55C8CCCE.9010503@oracle.com> References: <1439216678-12407-1-git-send-email-david.vrabel@citrix.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii"; Format="flowed" Content-Transfer-Encoding: 7bit Return-path: Received: from mail6.bemta3.messagelabs.com ([195.245.230.39]) by lists.xen.org with esmtp (Exim 4.72) (envelope-from ) id 1ZOpf8-0001Wa-0A for xen-devel@lists.xenproject.org; Mon, 10 Aug 2015 16:10:58 +0000 In-Reply-To: <1439216678-12407-1-git-send-email-david.vrabel@citrix.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: David Vrabel , xen-devel@lists.xenproject.org Cc: Ross Lagerwall , Sander Eikelenboom List-Id: xen-devel@lists.xenproject.org On 08/10/2015 10:24 AM, David Vrabel wrote: > Commit fcdf31a7c162de0c93a2bee51df4688ab0a348f8 (xen/events/fifo: > Handle linked events when closing a port) did not handle closing a > port bound to a PIRQ because these are closed from shutdown_pirq() > which is called with interrupts disabled. > > Defer the close to a work queue where we can safely spin waiting for > the LINKED bit to clear. For simplicity, the close is always deferred > even if it is not required (i.e., we're already in process context). > > Signed-off-by: David Vrabel > Cc: Ross Lagerwall Reviewed-by: Boris Ostrovsky