commit d1dbf12e3be0befcd3fd1f978202c5f72d2cc67b
author Johan Hedberg <johan.hedberg@intel.com> Fri Jul 04 16:17:23 2014 +0300
committer Marcel Holtmann <marcel@holtmann.org> Fri Jul 04 15:54:29 2014 +0200
tree 93321567f8e229a45b495086871e54107f4f48a8
parent 99a6768e0e55d19a47934ccd653ff0f9b3236401

Bluetooth: Use list_del when freeing the list entry

It's wasteful to use list_del_init (which re-initializes the list_head)
if we're just about to free the element and never use it again.

Signed-off-by: Johan Hedberg <johan.hedberg@intel.com>
Signed-off-by: Marcel Holtmann <marcel@holtmann.org>

net/bluetooth/mgmt.c

diff --git a/net/bluetooth/mgmt.c b/net/bluetooth/mgmt.c
index f1672b1..90eabca 100644
--- a/net/bluetooth/mgmt.c
+++ b/net/bluetooth/mgmt.c
@@ -5225,7 +5225,7 @@
 			goto unlock;
 		}
 
-		list_del_init(&params->action);
+		list_del(&params->action);
 		list_del(&params->list);
 		kfree(params);
 		hci_update_background_scan(hdev);