From: Johan Hedberg Date: Fri, 20 May 2011 18:50:41 +0000 (-0700) Subject: Bluetooth: Remove unnecessary use of hci_dev_list_lock X-Git-Tag: firefly_0821_release~7613^2~539 X-Git-Url: http://demsky.eecs.uci.edu/git/?a=commitdiff_plain;h=ceb723b8a337b6544594b39427bd3c2e5eaf0102;p=firefly-linux-kernel-4.4.55.git Bluetooth: Remove unnecessary use of hci_dev_list_lock The get_connections function has no need to use hci_dev_list_lock. The code was there probably because of a copy-paste mistake. Signed-off-by: Johan Hedberg Signed-off-by: Gustavo F. Padovan --- diff --git a/net/bluetooth/mgmt.c b/net/bluetooth/mgmt.c index 0e272e0c3c73..80874ca06a94 100644 --- a/net/bluetooth/mgmt.c +++ b/net/bluetooth/mgmt.c @@ -1092,8 +1092,6 @@ static int get_connections(struct sock *sk, u16 index) put_unaligned_le16(count, &rp->conn_count); - read_lock(&hci_dev_list_lock); - i = 0; list_for_each(p, &hdev->conn_hash.list) { struct hci_conn *c = list_entry(p, struct hci_conn, list); @@ -1101,8 +1099,6 @@ static int get_connections(struct sock *sk, u16 index) bacpy(&rp->conn[i++], &c->dst); } - read_unlock(&hci_dev_list_lock); - err = cmd_complete(sk, index, MGMT_OP_GET_CONNECTIONS, rp, rp_len); unlock: