On Mon, Apr 16, 2012 at 10:35:34AM +0200, Alexander Holler wrote: > Hello, > > Please consider commit e72acc13c770a82b4ce4a07e9716f29320eae0f8 > "Bluetooth: Remove unneeded locking" for inclusion into the 3.3.x > stable series. > > This patch removes a lock which leads to a deadlock whenever > bluetoothd is stopped while a (battery powered) Bluetooth-HID-device > like mouse or keyboard is connected. > > I'm not sure if that only happens with battery powered devices, but > the deadlock appears here every time and can be seen by reading the > source only too: > > bluetoothd calls ioctl HCIDEVDOWN > hci_sock_ioctl() > hci_dev_close() > hci_dev_do_close() > hci_dev_lock(hdev); > inquiry_cache_flush(); > hci_conn_hash_flush(); > hci_conn_del() > cancel_delayed_work_sync() > hci_conn_timeout() > hci_dev_lock(hdev); /* DEADLOCK */ > > According to the author no prerequisites are necessary > (https://lkml.org/lkml/2012/4/2/218). > > I'm using the patch with 3.3.x now since about 2 weeks without any > problems, so I can offer a Tested-by: too.
Now applied, thanks for letting us know. greg k-h -- To unsubscribe from this list: send the line "unsubscribe stable" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
