]> granicus.if.org Git - postgresql/commit
Refactor pid, random seed and start time initialization.
authorThomas Munro <tmunro@postgresql.org>
Fri, 19 Oct 2018 00:59:14 +0000 (13:59 +1300)
committerThomas Munro <tmunro@postgresql.org>
Fri, 19 Oct 2018 00:59:28 +0000 (13:59 +1300)
commit197e4af9d5da180190a0f2be851b095dba57d9cd
tree4121b3b50a5f020f982eb4cbbe754420d200ff31
parente74dd00f53cd6dc1887f76b9672e5f6dcf0fd8a2
Refactor pid, random seed and start time initialization.

Background workers, including parallel workers, were generating
the same sequence of numbers in random().  This showed up as DSM
handle collisions when Parallel Hash created multiple segments,
but any code that calls random() in background workers could be
affected if it cares about different backends generating different
numbers.

Repair by making sure that all new processes initialize the seed
at the same time as they set MyProcPid and MyStartTime in a new
function InitProcessGlobals(), called by the postmaster, its
children and also standalone processes.  Also add a new high
resolution MyStartTimestamp as a potentially useful by-product,
and remove SessionStartTime from struct Port as it is now
redundant.

No back-patch for now, as the known consequences so far are just
a bunch of harmless shm_open(O_EXCL) collisions.

Author: Thomas Munro
Reviewed-by: Tom Lane
Discussion: https://postgr.es/m/CAEepm%3D2eJj_6%3DB%2B2tEpGu2nf1BjthCf9nXXUouYvJJ4C5WSwhg%40mail.gmail.com
src/backend/postmaster/pgstat.c
src/backend/postmaster/postmaster.c
src/backend/tcop/postgres.c
src/backend/utils/init/globals.c
src/backend/utils/init/miscinit.c
src/include/libpq/libpq-be.h
src/include/miscadmin.h
src/include/postmaster/postmaster.h