Re: [BUG] [Resend] Possible sleep-in-atomic-context bugs involving regmap_lock_mutex()

From: Mark Brown
Date: Mon Sep 10 2018 - 13:41:15 EST


On Thu, Aug 30, 2018 at 10:34:20AM +0800, Jia-Ju Bai wrote:

> My static tool DSAC reports many sleep-in-atomic-context bugs involving
> regmap_lock_mutex(), so I wonder whether this function is possible to be
> executed in atomic context.

Have you actually analyzed the code paths that are really taken here?
Static tools really don't cope very well with the pluggable locking in
regmap. If there are problems I'd suggest reporting them to the authors
of the drivers using them, it's not really anything to do with regmap if
they've messed up their locking.

> Here are some example bugs and their call paths in Linux-4.16 (from bottom
> to top, and [FUNC_PTR] means that there is a function pointer call):

That's quite old now... not that this has been rapidly developed
recently but still.

Attachment: signature.asc
Description: PGP signature