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: 2.1b_release~2518 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=f2525702ab7cd93689c7c6aa0a87319116de9c5d;p=platform%2Fkernel%2Fkernel-mfld-blackbay.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 0e272e0..80874ca 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: