]> granicus.if.org Git - postgresql/commit
Avoid tricky race condition recording XLOG_HINT
authorSimon Riggs <simon@2ndQuadrant.com>
Mon, 8 Apr 2013 07:52:39 +0000 (08:52 +0100)
committerSimon Riggs <simon@2ndQuadrant.com>
Mon, 8 Apr 2013 07:52:39 +0000 (08:52 +0100)
commit47c4333189c9eb5ac1f95adc90213e61996f53c4
treecb66f513c3c2932cae1e97034585a4448184383f
parenta4b94b85156192b839a3c840f8aaf0cf8699a8c8
Avoid tricky race condition recording XLOG_HINT
We copy the buffer before inserting an XLOG_HINT to avoid WAL CRC errors
caused by concurrent hint writes to buffer while share locked. To make this work
we refactor RestoreBackupBlock() to allow an XLOG_HINT to avoid the normal
path for backup blocks, which assumes the underlying buffer is exclusive locked.
Resulting code completely changes layout of XLOG_HINT WAL records, but
this isn't even beta code, so this is a low impact change.
In passing, avoid taking WALInsertLock for full page writes on checksummed
hints, remove related cruft from XLogInsert() and improve xlog_desc record for
XLOG_HINT.

Andres Freund

Bug report by Fujii Masao, testing by Jeff Janes and Jaime Casanova,
review by Jeff Davis and Simon Riggs. Applied with changes from review
and some comment editing.
src/backend/access/rmgrdesc/xlogdesc.c
src/backend/access/transam/xlog.c
src/backend/storage/buffer/bufmgr.c
src/include/catalog/catversion.h