From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chris Wright Subject: Re: [PATCH 1/1] KVM: Fix potentially recursively get kvm lock Date: Fri, 22 May 2009 08:06:23 -0700 Message-ID: <20090522150623.GD20823@sequoia.sous-sol.org> References: <200905121705.53176.sheng.yang@intel.com> <1242120729-2280-1-git-send-email-sheng@linux.intel.com> <20090512115524.GB10901@amt.cnet> <200905122213.36833.sheng.yang@intel.com> <20090512143021.GB12888@amt.cnet> <20090512194432.GA19969@amt.cnet> <1242164187.4788.4.camel@2710p.home> <20090512220908.GA22626@amt.cnet> <1242166650.4788.16.camel@2710p.home> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Marcelo Tosatti , "Yang, Sheng" , Avi Kivity , kvm@vger.kernel.org To: Alex Williamson Return-path: Received: from sous-sol.org ([216.99.217.87]:57393 "EHLO sequoia.sous-sol.org" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1752386AbZEVPGn (ORCPT ); Fri, 22 May 2009 11:06:43 -0400 Content-Disposition: inline In-Reply-To: <1242166650.4788.16.camel@2710p.home> Sender: kvm-owner@vger.kernel.org List-ID: * Alex Williamson (alex.williamson@hp.com) wrote: > On Tue, 2009-05-12 at 19:09 -0300, Marcelo Tosatti wrote: > > KVM: workaround workqueue / deassign_host_irq deadlock > > > > I think I'm running into the following deadlock in the kvm kernel module > > when trying to use device assignment: > > > > CPU A CPU B > > kvm_vm_ioctl_deassign_dev_irq() > > mutex_lock(&kvm->lock); worker_thread() > > -> kvm_deassign_irq() -> > > kvm_assigned_dev_interrupt_work_handler() > > -> deassign_host_irq() mutex_lock(&kvm->lock); > > -> cancel_work_sync() [blocked] > > > > Workaround the issue by dropping kvm->lock for cancel_work_sync(). > > > > Reported-by: Alex Williamson > > From: Sheng Yang > > Signed-off-by: Marcelo Tosatti > > Perfect, thanks. > > Acked-by: Alex Williamson Is this still pending? thanks, -chris