Re: [PATCH] bluecard: Fix a possible sleep-in-atomic bug in bluecard_write_wakeup

From: Marcel Holtmann
Date: Wed Dec 13 2017 - 08:39:52 EST


Hi Jia-Ju,

> The driver may sleep in the interrupt handler.
> The function call path is:
> bluecard_interrupt (interrupt handler)
> bluecard_write_wakeup
> schedule_timeout --> may sleep
>
> To fix it, schedule_timeout is replaced with mdelay.
>
> This bug is found by my static analysis tool(DSAC) and checked by my code review.
>
> Signed-off-by: Jia-Ju Bai <baijiaju1990@xxxxxxxxx>
> ---
> drivers/bluetooth/bluecard_cs.c | 8 ++------
> 1 file changed, 2 insertions(+), 6 deletions(-)

patch has been applied to bluetooth-next tree.

Regards

Marcel