Bluetooth: Use GFP_KERNEL when cloning SKB in a workqueue
authorMarcel Holtmann <marcel@holtmann.org>
Wed, 4 Sep 2013 01:11:07 +0000 (18:11 -0700)
committerGustavo Padovan <gustavo.padovan@collabora.co.uk>
Mon, 16 Sep 2013 17:35:56 +0000 (14:35 -0300)
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>
net/bluetooth/hci_core.c

index 0ee0f01d33df9a24a7211d8171f0d143598e2e36..26673d332997392586edcc8d55d72af89066ffba 100644 (file)
@@ -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);