pgsql: Don't use _mdfd_getseg() in mdsyncfiletag().

From: Thomas Munro <tmunro(at)postgresql(dot)org>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: Don't use _mdfd_getseg() in mdsyncfiletag().
Date: 2019-12-14 03:48:25
Message-ID: E1ifyPp-0007d7-68@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

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

Branch
------
REL_12_STABLE

Details
-------
https://git.postgresql.org/pg/commitdiff/c3dc0cdd6b58f6d4a632364554c9aec5b4f83f69

Modified Files
--------------
src/backend/storage/smgr/md.c | 57 ++++++++++++++++++++++++++++++-------------
1 file changed, 40 insertions(+), 17 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Thomas Munro 2019-12-14 06:09:24 pgsql: Fix mdsyncfiletag(), take II.
Previous Message Thomas Munro 2019-12-14 03:48:12 pgsql: Don't use _mdfd_getseg() in mdsyncfiletag().