Hi Jia-Ju, > After checking all possible call chains to btmrvl_send_sync_cmd(), > my tool finds that this function is never called in atomic context, > namely never in an interrupt handler or holding a spinlock. > And it calls wait_event_interruptible_timeout() after bt_skb_alloc(), > so it indicates that btmrvl_send_sync_cmd() > can call function which can sleep. > Thus GFP_ATOMIC is not necessary, and it can be replaced with GFP_KERNEL. > > This is found by a static analysis tool named DCNS written by myself. > > Signed-off-by: Jia-Ju Bai <baijiaju1990@xxxxxxxxx> > --- > drivers/bluetooth/btmrvl_main.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) patch has been applied to bluetooth-next tree. Regards Marcel -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
