From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Sender: "Gustavo F. Padovan" Date: Tue, 26 Apr 2011 12:49:50 -0300 From: "Gustavo F. Padovan" To: Imran Haider Cc: marcel@holtmann.org, linux-bluetooth@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] Bluetooth: Don't sleep after deleting command timer Message-ID: <20110426154950.GA2242@joana> References: <1303513362-9357-1-git-send-email-imran1008@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <1303513362-9357-1-git-send-email-imran1008@gmail.com> List-ID: Hi Imran, * Imran Haider [2011-04-22 19:02:42 -0400]: > This fix adds on to commit b77dcf84. It seems that scheduling a timeout > of 250ms after deleting the command timer causes a kernel panic when > system is resuming from Suspend-to-RAM. I tried adding another timeout > before deleting the command timer but that didn't help so I believe that > some code in hci_dev_do_close must execute after the command timer is > deleted before control is returned to another thread. > > The only bit the command timer handler hci_cmd_timer() touches is HCI_RESET. > It should be fine to delete the timer after testing the HCI_UP bit. > --- > net/bluetooth/hci_core.c | 9 ++++++--- > 1 files changed, 6 insertions(+), 3 deletions(-) I already have for that. But thanks anyway. ;) -- Gustavo F. Padovan http://profusion.mobi