rtc: r7301: Fix a possible sleep-in-atomic bug in rtc7301_read_time
authorJia-Ju Bai <baijiaju1990@gmail.com>
Wed, 13 Dec 2017 13:55:50 +0000 (21:55 +0800)
committerAlexandre Belloni <alexandre.belloni@free-electrons.com>
Thu, 11 Jan 2018 23:20:28 +0000 (00:20 +0100)
The driver may sleep under a spinlock.
The function call path is:
rtc7301_read_time (acquire the spinlock)
  rtc7301_wait_while_busy
    usleep_range --> may sleep

To fix it, usleep_range is replaced with udelay.

This bug is found by my static analysis tool(DSAC) and checked by my code review.

Signed-off-by: Jia-Ju Bai <baijiaju1990@gmail.com>
Signed-off-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
drivers/rtc/rtc-r7301.c

index 28d5408..d846e97 100644 (file)
@@ -95,7 +95,7 @@ static int rtc7301_wait_while_busy(struct rtc7301_priv *priv)
                if (!(val & RTC7301_CONTROL_BUSY))
                        return 0;
 
-               usleep_range(200, 300);
+               udelay(300);
        }
 
        return -ETIMEDOUT;