Re: [PATCH 1/2] block: Annotate the queue limits functions

[Christoph Hellwig <hch@lst.de>]

On Thu, Jan 15, 2026 at 08:53:32AM -0800, Bart Van Assche wrote:
> On 1/15/26 1:11 AM, John Garry wrote:
>> On 15/01/2026 06:26, Christoph Hellwig wrote:
>>> This is missing a commit log.  And not really telling what kind
>>> of annotation you're adding.
>>
>> And we removed these previously - see c3042a5403ef2.
>>
>> Does sparse now handle mutexes?
>
> sparse is dead.

It's not.  And even if it was we still need it.

> The most recent commit is from February 2024 (almost two
> years ago).

The repository disagrees:

commit fbdde3127b83e6d09e0ba808d7925dd84407f3c6 (origin/master, origin/HEAD)
Author: Dan Carpenter <dan.carpenter@linaro.org>
Date:   Wed Oct 15 16:08:13 2025 +0300

    builtin: implement __builtin_strlen() for constants


> Additionally, the sparse maintainer doesn't reply anymore to
> emails or bug reports about sparse.

While I'd love to see more activity, that is clearly untrue as well:

[PATCH v2 3/3] module: Add compile-time check for embedded NUL characters


> These annotations aren't for sparse - these are for clang. This patch 
> series has been queued by Peter Zijlstra on the tip master branch and is
> expected to be sent to Linus during the next merge window: "[PATCH v5
> 00/36] Compiler-Based Context- and Locking-Analysis"
> (https://lore.kernel.org/lkml/20251219154418.3592607-1-elver@google.com/).

Until that actually becomes mainstream we can't break the existing
annotations.