Bluetooth: Use GFP_KERNEL in mgmt events functions
authorAndre Guedes <aguedespe@gmail.com>
Thu, 7 Jun 2012 22:05:46 +0000 (19:05 -0300)
committerGustavo Padovan <gustavo.padovan@collabora.co.uk>
Tue, 19 Jun 2012 03:53:38 +0000 (00:53 -0300)
cmd_status, cmd_complete and mgmt_event functions are executed in
process context and they are not called inside atomic sections. Thus,
they should use GFP_KERNEL for memory allocation instead of GFP_ATOMIC.

Signed-off-by: Andre Guedes <aguedespe@gmail.com>
Signed-off-by: Gustavo Padovan <gustavo.padovan@collabora.co.uk>
net/bluetooth/mgmt.c

index 41b5192bc04152836a09bde4692df347d505d668..2f9f52f19b48bd2f2df6427fe71706f980b5aa2b 100644 (file)
@@ -210,7 +210,7 @@ static int cmd_status(struct sock *sk, u16 index, u16 cmd, u8 status)
 
        BT_DBG("sock %p, index %u, cmd %u, status %u", sk, index, cmd, status);
 
-       skb = alloc_skb(sizeof(*hdr) + sizeof(*ev), GFP_ATOMIC);
+       skb = alloc_skb(sizeof(*hdr) + sizeof(*ev), GFP_KERNEL);
        if (!skb)
                return -ENOMEM;
 
@@ -241,7 +241,7 @@ static int cmd_complete(struct sock *sk, u16 index, u16 cmd, u8 status,
 
        BT_DBG("sock %p", sk);
 
-       skb = alloc_skb(sizeof(*hdr) + sizeof(*ev) + rp_len, GFP_ATOMIC);
+       skb = alloc_skb(sizeof(*hdr) + sizeof(*ev) + rp_len, GFP_KERNEL);
        if (!skb)
                return -ENOMEM;
 
@@ -812,7 +812,7 @@ static int mgmt_event(u16 event, struct hci_dev *hdev, void *data, u16 data_len,
        struct sk_buff *skb;
        struct mgmt_hdr *hdr;
 
-       skb = alloc_skb(sizeof(*hdr) + data_len, GFP_ATOMIC);
+       skb = alloc_skb(sizeof(*hdr) + data_len, GFP_KERNEL);
        if (!skb)
                return -ENOMEM;