Re: [regression] connecting a bluetooth mouse triggers multiple BUGs and warnings

[Marcel Holtmann <marcel@holtmann.org>]

[-- Attachment #1: Type: text/plain, Size: 3156 bytes --]

Hi Davide,

> the NULL pointer dereference and the warnings are indeed fixed in
> bluetooth-testing, and the mouse works. Thanks!
> However the following issue remains:
> 
> [  110.012125] BUG: sleeping function called from invalid context at
> mm/slub.c:1595
> [  110.012135] in_atomic(): 1, irqs_disabled(): 0, pid: 0, name:
> swapper
> [  110.012141] 2 locks held by swapper/0:
> [  110.012145]  #0:  (hci_task_lock){++.-.+}, at: [<ffffffffa01f822f>]
> hci_rx_task+0x2f/0x2d0 [bluetooth]
> [  110.012173]  #1:  (&hdev->lock){+.-.+.}, at: [<ffffffffa01fb9e2>]
> hci_event_packet+0x72/0x25c0 [bluetooth]
> [  110.012198] Pid: 0, comm: swapper Tainted: G        W
> 2.6.30-rc4-g953cdaa #1
> [  110.012203] Call Trace:
> [  110.012207]  <IRQ>  [<ffffffff8023eabd>] __might_sleep+0x14d/0x170
> [  110.012228]  [<ffffffff802cfbe1>] __kmalloc+0x111/0x170
> [  110.012239]  [<ffffffff803c2094>] kvasprintf+0x64/0xb0
> [  110.012248]  [<ffffffff803b7a5b>] kobject_set_name_vargs+0x3b/0xa0
> [  110.012257]  [<ffffffff80465326>] dev_set_name+0x76/0xa0
> [  110.012273]  [<ffffffffa01fb9e2>] ? hci_event_packet+0x72/0x25c0
> [bluetooth]
> [  110.012289]  [<ffffffffa01ffc1d>] hci_conn_add_sysfs+0x3d/0x70
> [bluetooth]
> [  110.012303]  [<ffffffffa01fba2c>] hci_event_packet+0xbc/0x25c0
> [bluetooth]
> [  110.012312]  [<ffffffff80516eb0>] ? sock_def_readable+0x80/0xa0
> [  110.012328]  [<ffffffffa01fee0c>] ? hci_send_to_sock+0xfc/0x1c0
> [bluetooth]
> [  110.012343]  [<ffffffff80516eb0>] ? sock_def_readable+0x80/0xa0
> [  110.012347]  [<ffffffff805e88c5>] ? _read_unlock+0x75/0x80
> [  110.012354]  [<ffffffffa01fee0c>] ? hci_send_to_sock+0xfc/0x1c0
> [bluetooth]
> [  110.012360]  [<ffffffffa01f8403>] hci_rx_task+0x203/0x2d0
> [bluetooth]
> [  110.012365]  [<ffffffff80250ab5>] tasklet_action+0xb5/0x160
> [  110.012369]  [<ffffffff8025116c>] __do_softirq+0x9c/0x150
> [  110.012372]  [<ffffffff805e850f>] ? _spin_unlock+0x3f/0x80
> [  110.012376]  [<ffffffff8020cbbc>] call_softirq+0x1c/0x30
> [  110.012380]  [<ffffffff8020f01d>] do_softirq+0x8d/0xe0
> [  110.012383]  [<ffffffff80250df5>] irq_exit+0xc5/0xe0
> [  110.012386]  [<ffffffff8020e71d>] do_IRQ+0x9d/0x120
> [  110.012389]  [<ffffffff8020c3d3>] ret_from_intr+0x0/0xf
> [  110.012391]  <EOI>  [<ffffffff80431832>] ? acpi_idle_enter_bm+0x264/0x2a6
> [  110.012399]  [<ffffffff80431828>] ? acpi_idle_enter_bm+0x25a/0x2a6
> [  110.012403]  [<ffffffff804f50d5>] ? cpuidle_idle_call+0xc5/0x130
> [  110.012407]  [<ffffffff8020a4b4>] ? cpu_idle+0xc4/0x130
> [  110.012411]  [<ffffffff805d2268>] ? rest_init+0x88/0xb0
> [  110.012416]  [<ffffffff807e2fbd>] ? start_kernel+0x3b5/0x412
> [  110.012420]  [<ffffffff807e2281>] ? x86_64_start_reservations+0x91/0xb5
> [  110.012424]  [<ffffffff807e2394>] ? x86_64_start_kernel+0xef/0x11b
> [  110.320835] Bluetooth: HIDP (Human Interface Emulation) ver 1.2
> [  110.349150] input: Mighty Mouse as
> /devices/pci0000:00/0000:00:1d.3/usb5/5-1/5-1:1.0/bluetooth/hci0/hci0:46/input10
> [  110.351343] apple 0005:05AC:030C.0004: input: BLUETOOTH HID v2.00
> Mouse [Mighty Mouse] on 00:17:F2:AB:8D:45

try the attached patch. It should fix this.

Regards

Marcel


[-- Attachment #2: patch --]
[-- Type: text/x-patch, Size: 919 bytes --]

diff --git a/net/bluetooth/hci_sysfs.c b/net/bluetooth/hci_sysfs.c
index 582d887..a05d45e 100644
--- a/net/bluetooth/hci_sysfs.c
+++ b/net/bluetooth/hci_sysfs.c
@@ -88,10 +88,13 @@ static struct device_type bt_link = {
 static void add_conn(struct work_struct *work)
 {
 	struct hci_conn *conn = container_of(work, struct hci_conn, work_add);
+	struct hci_dev *hdev = conn->hdev;
 
 	/* ensure previous del is complete */
 	flush_work(&conn->work_del);
 
+	dev_set_name(&conn->dev, "%s:%d", hdev->name, conn->handle);
+
 	if (device_add(&conn->dev) < 0) {
 		BT_ERR("Failed to register connection device");
 		return;
@@ -154,12 +157,8 @@ void hci_conn_init_sysfs(struct hci_conn *conn)
 
 void hci_conn_add_sysfs(struct hci_conn *conn)
 {
-	struct hci_dev *hdev = conn->hdev;
-
 	BT_DBG("conn %p", conn);
 
-	dev_set_name(&conn->dev, "%s:%d", hdev->name, conn->handle);
-
 	queue_work(bt_workq, &conn->work_add);
 }