Re: [PATCH 1/1] backing-dev: refactor wb_congested_put()

From: Matthew Wilcox
Date: Wed Mar 11 2020 - 22:29:52 EST


On Wed, Mar 11, 2020 at 05:59:19PM -0700, Andrew Morton wrote:
> hm, it's hard to get excited over this. Open-coding the
> refcount_dec_and_lock_irqsave() internals at a callsite in order to
> make sparse happy.
>
> Is there some other way, using __acquires (for example)?

sparse is really bad at conditional lock acquisition. we have similar
problems over the vfs. but we shouldn't be obfuscating our code to make
the tool happy.