bluetooth: bluecard_cs: Replace GFP_ATOMIC with GFP_KERNEL in bluecard_hci_set_baud_r...
authorJia-Ju Bai <baijiaju1990@gmail.com>
Mon, 23 Jul 2018 03:17:35 +0000 (11:17 +0800)
committerMarcel Holtmann <marcel@holtmann.org>
Mon, 23 Jul 2018 16:05:00 +0000 (18:05 +0200)
commit5f9c6580f6324e279417fc66e80ade0c42bfa9d9
tree5188e346ee90d39e7d7dfae6e6b836f18c7a7494
parent6709514f266e9163f9f579006496fbc8bc986d80
bluetooth: bluecard_cs: Replace GFP_ATOMIC with GFP_KERNEL in bluecard_hci_set_baud_rate()

bluecard_hci_set_baud_rate() is never called in atomic context.

bluecard_hci_set_baud_rate() is only by bluecard_hci_open(), which is
set to hdev->open, and hdev->open() is never called in atomic context.

bluecard_hci_set_baud_rate() calls bt_skb_alloc() with GFP_ATOMIC,
which is not necessary. GFP_ATOMIC can be replaced with GFP_KERNEL.

This is found by a static analysis tool named DCNS written by myself.
I also manually check the kernel code before reporting it.

Signed-off-by: Jia-Ju Bai <baijiaju1990@gmail.com>
Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
drivers/bluetooth/bluecard_cs.c