| From: | Álvaro Herrera <alvherre(at)kurilemu(dot)de> |
|---|---|
| To: | Amit Langote <amitlangote09(at)gmail(dot)com> |
| Cc: | Rahila Syed <rahilasyed90(at)gmail(dot)com>, Andres Freund <andres(at)anarazel(dot)de>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: Segmentation fault on proc exit after dshash_find_or_insert |
| Date: | 2026-01-14 09:36:24 |
| Message-ID: | 202601140928.sbxjiuhq5ae6@alvherre.pgsql |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On 2025-Dec-18, Amit Langote wrote:
> Thanks. Updated the commit message too to be more accurate in the
> attached updated patch.
Looks good to me. I would add an Assert(num_held_lwlocks == 0) at the
end of LWLockReleaseAll(), to make it clear that it's idempotent (which
is important for the case where ProcKill will call it again shortly
after).
Are you going to push this soon?
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.
--
Álvaro Herrera 48°01'N 7°57'E — https://www.EnterpriseDB.com/
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Kirill Reshke | 2026-01-14 09:42:54 | Re: GIN pageinspect support for entry tree and posting tree |
| Previous Message | John Naylor | 2026-01-14 09:31:20 | Update some comments for fasthash |