Return-Path: Date: Thu, 2 Feb 2012 10:34:23 +0200 From: Emeltchenko Andrei To: Marcel Holtmann Cc: Ulisses Furquim , Andre Guedes , linux-bluetooth@vger.kernel.org Subject: Re: [RESEND 1/3] Bluetooth: Use cancel_work instead of cancel_work_sync Message-ID: <20120202083417.GA13814@aemeltch-MOBL1> References: <1327999384-16094-1-git-send-email-Andrei.Emeltchenko.news@gmail.com> <1327999384-16094-2-git-send-email-Andrei.Emeltchenko.news@gmail.com> <20120131130242.GB7660@aemeltch-MOBL1> <1328022697.1955.217.camel@aeonflux> <1328025524.1955.219.camel@aeonflux> <20120201133254.GA3388@aemeltch-MOBL1> <1328111610.1955.238.camel@aeonflux> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <1328111610.1955.238.camel@aeonflux> Sender: linux-bluetooth-owner@vger.kernel.org List-ID: Hi Marcel, On Wed, Feb 01, 2012 at 07:53:30AM -0800, Marcel Holtmann wrote: > > > >> >> > > - cancel_delayed_work_sync(&conn->disc_work); > > > >> >> > > + cancel_delayed_work(&conn->disc_work); > > > >> >> > > > > >> >> > I'm afraid we must use _sync variant here. disc_work is not supposed to > > > >> >> > be running after hci_conn is deleted. > > > >> >> > > > > >> >> > BTW, I believe we already addressed this issue in patches [PATCH 1/2] > > > >> >> > Bluetooth: Fix potential deadlock and [PATCH 2/2] Bluetooth: Remove > > > >> >> > unneeded locking. These patches are now pushed upstream. Could you > > > >> >> > > > >> >> I will check those patches from upstream and let you know. > > > >> > > > > >> > crap, I just acked these. > > > >> > > > > >> > Johan, forget about my acks and just ignore them. Lets wait until we get > > > >> > a clean new series. > > > >> > > > > >> This change is really wrong because we're on the delete path and Andre > > > >> sent other patches which I'm almost sure will address this problem. > > > > > > > > lets do it this way, I only look at final patches that you and Andrei > > > > signed off / acked. > > > > > > Sure, we can do that this way. > > > > I've checked recent upstream code and it works fine so far, please forget > > about this particular patch. > > > > The other 2 patches are still valid. > > create a new patch series for it. Done. Johan, please check the patches in new series. Best regards Andrei Emeltchenko