Don't use _mdfd_getseg() in mdsyncfiletag().
authorThomas Munro <tmunro@postgresql.org>
Sat, 14 Dec 2019 02:54:31 +0000 (15:54 +1300)
committerThomas Munro <tmunro@postgresql.org>
Sat, 14 Dec 2019 03:32:03 +0000 (16:32 +1300)
commit7bb3102cea02101efcbb4c4fba3fdd452a52bdab
tree4856ad91c2edd256f90ca01d2eac17b39ff3e06d
parenta7ee7c85132221ff7231b6f910915a1b3ce1ecbc
Don't use _mdfd_getseg() in mdsyncfiletag().

_mdfd_getseg() opens all segments up to the requested one.  That
causes problems for mdsyncfiletag(), if mdunlinkfork() has
already unlinked other segment files.  Open the file we want
directly by name instead, if it's not already open.

The consequence of this bug was a rare panic in the checkpointer,
made more likely if you saturated the sync request queue so that
the SYNC_FORGET_REQUEST messages for a given relation were more
likely to be absorbed in separate cycles by the checkpointer.

Back-patch to 12.  Defect in commit 3eb77eba.

Author: Thomas Munro
Reported-by: Justin Pryzby
Discussion: https://postgr.es/m/20191119115759.GI30362%40telsasoft.com
src/backend/storage/smgr/md.c