Re: [PATCH] futex/requeue: Revert "Prevent NULL pointer dereference in remove_waiter() on self-deadlock""

[Sebastian Andrzej Siewior]

On 2026-07-01 17:25:05 [+0200], Thomas Gleixner wrote:
> > That has been already handled by Davidlohr in commit 40a25d59e85b3
> > ("locking/rtmutex: Skip remove_waiter() when waiter is not enqueued")
> > and requires no further acting.
> 
> No. If the self deadlock is obvious, then this should not even go into
> the proxy lock code. I clearly failed to notice the leak problem, but
> reverting it and relying on some magic down the road to handle it
> correctly is not really solid.

Davidlohr fixed the issue that has been fixed again. We don't need two
changes for the same thing.
We have deadlock detection down the road, why special case it here?

> The below should fix it properly.
>
Sebastian