Amit Langote <[email protected]> writes:
> On Wed, Jan 14, 2026 at 6:36 PM Álvaro Herrera <[email protected]> wrote
>> Looking at ProcKill, I notice that we do some LWLock ops after its
>> LWLockReleaseAll() call, which seems a bit silly. Why not do that right
>> after the "if (MyProc->lockGroupLeader != NULL)" block instead? Nothing
>> uses LWLocks from there on. This can be a separate commit.
> Just to confirm: you're suggesting moving the LWLockReleaseAll() call
> to after the "if (MyProc->lockGroupLeader != NULL)" block? Makes sense
> -- odd to release all locks right before then going ahead and
> acquiring one. Agreed it should be a separate commit.
I think the idea there might be to make sure that we have released
any pre-existing hold of that lock. Otherwise this could be
a self-deadlock.
regards, tom lane
