From: Marcel Holtmann <marcel@holtmann.org>
Date: Wed, 4 Sep 2013 01:11:07 +0000 (-0700)
Subject: Bluetooth: Use GFP_KERNEL when cloning SKB in a workqueue
X-Git-Tag: accepted/tizen/common/20141203.182822~1141^2~225^2~24^2~60
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=a675d7f1a0a0c0d7bde56ae89609dee56f2033e1;p=platform%2Fkernel%2Flinux-arm64.git

Bluetooth: Use GFP_KERNEL when cloning SKB in a workqueue

There is no need to use GFP_ATOMIC with skb_clone() when the code is
executed in a workqueue.

Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
Acked-by: Johan Hedberg <johan.hedberg@intel.com>
Signed-off-by: Gustavo Padovan <gustavo.padovan@collabora.co.uk>
---

diff --git a/net/bluetooth/hci_core.c b/net/bluetooth/hci_core.c
index 0ee0f01..26673d3 100644
--- a/net/bluetooth/hci_core.c
+++ b/net/bluetooth/hci_core.c
@@ -3547,7 +3547,7 @@ static void hci_cmd_work(struct work_struct *work)
 
 		kfree_skb(hdev->sent_cmd);
 
-		hdev->sent_cmd = skb_clone(skb, GFP_ATOMIC);
+		hdev->sent_cmd = skb_clone(skb, GFP_KERNEL);
 		if (hdev->sent_cmd) {
 			atomic_dec(&hdev->cmd_cnt);
 			hci_send_frame(skb);