pg.ddx.io  pgsql-hackers@postgresql.org mailing list archive  

help / color / mirror / Atom feed
From: Nathan Bossart <nathandbossart@gmail.com>
To: Noah Misch <noah@leadboat.com>
Cc: pgsql-hackers@postgresql.org
Subject: Re: Fix early elog(FATAL)
Date: Thu, 12 Dec 2024 10:07:00 -0600
Message-ID: <Z1sKJBE6-Lo08PEt@nathan> (raw)
In-Reply-To: <20241212033414.e0.nmisch@google.com>
References: <20241208034614.45.nmisch@google.com>
	<Z1i-K95hTcggVOgn@nathan>
	<20241212033414.e0.nmisch@google.com>

On Wed, Dec 11, 2024 at 07:34:14PM -0800, Noah Misch wrote:
> On Tue, Dec 10, 2024 at 04:18:19PM -0600, Nathan Bossart wrote:
>> FWIW I'd probably vote for option 1.  That keeps the initialization of the
>> globals together, reduces the call sites, and fixes the bug.  I'd worry a
>> little about moving the MyProcPid assignments out of that function without
>> adding a bunch of commentary to explain why.
> 
> Can you say more about that?  A comment about MyProcPid could say "fork() is
> the one thing that changes the getpid() return value".  To me, the things
> InitProcessGlobals() sets are all different.  MyProcPid can be set without
> elog(ERROR) and gets invalidated at fork().  The others reasonably could
> elog(ERROR).  (They currently don't.)  The random state could have a different
> lifecycle.  If we had a builtin pooler that reused processes, we'd
> reinitialize random state at each process reuse, not at each fork().  So I see
> the grouping of (MyProcPid, MyStartTimestamp, random seed) as mostly an
> accident of history.

Fair enough.  I suppose part of my hesitation stems from expecting hackers
to be more likely to remember to call InitProcessGlobals() than to
initialize MyProcPid.  But given your change requires initializing
MyProcPid in exactly 2 places, and there are unlikely to be more in the
near future, I might be overthinking it.

-- 
nathan





view thread (7+ messages)  latest in thread

Message-ID: <Z1sKJBE6-Lo08PEt@nathan>
Permalink:  ../Z1sKJBE6-Lo08PEt@nathan/
Also on:    postgresql.org/message-id/Z1sKJBE6-Lo08PEt@nathan

 · 

reply
Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-hackers@postgresql.org
  Cc: nathandbossart@gmail.com, noah@leadboat.com
  Subject: Re: Fix early elog(FATAL)
  In-Reply-To: <Z1sKJBE6-Lo08PEt@nathan>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox