pgsql: Fix invalid checksum state transition in checkpoints - Mailing list pgsql-committers

From Daniel Gustafsson
Subject pgsql: Fix invalid checksum state transition in checkpoints
Date
Msg-id E1wIPqk-003SOa-0q@gemulon.postgresql.org
Whole thread
List pgsql-committers
Fix invalid checksum state transition in checkpoints

Commit 78e950cb8 added checksum state handling to all XLOG_CHECKPOINT
records which caused unnecessary state transitions and emission of
procsignal barriers.  Remove as only the _REDO record need to handle
checksum state.  Barrier emission is also consistently made after
controlfile updates to avoid race conditions.

Additionally, interrupts are held between calling ProcSignalInit and
InitLocalDataChecksumState to remove a window where otherwise invalid
state transitions can happen.

Also remove a pointless assertion on Controlfile which will never hit.

Author: Tomas Vondra <tomas@vondra.me>
Author: Daniel Gustafsson <daniel@yesql.se>
Reviewed-by: Ayush Tiwari <ayushtiwari.slg01@gmail.com>
Reviewed-by: SATYANARAYANA NARLAPURAM <satyanarlapuram@gmail.com>
Discussion: https://postgr.es/m/9197F930-DDEB-4CAC-82A2-16FEC715CCE8@yesql.se

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/25b922ec5825e6938c28cdae718aa70b8f11fd50

Modified Files
--------------
src/backend/access/transam/xlog.c           | 107 +++++++++-------------------
src/backend/postmaster/auxprocess.c         |  10 +++
src/backend/postmaster/datachecksum_state.c |   8 +--
src/backend/utils/init/postinit.c           |  10 +++
4 files changed, 56 insertions(+), 79 deletions(-)


pgsql-committers by date:

Previous
From: Amit Kapila
Date:
Subject: pgsql: Fix double table_close of sequence_rel in copy_sequences().
Next
From: Daniel Gustafsson
Date:
Subject: pgsql: Fix data_checksum GUC show_hook