]> granicus.if.org Git - postgresql/commit
Fix race condition that lead to WALInsertLock deadlock with commit_delay.
authorHeikki Linnakangas <heikki.linnakangas@iki.fi>
Sun, 2 Aug 2015 17:08:10 +0000 (20:08 +0300)
committerHeikki Linnakangas <heikki.linnakangas@iki.fi>
Sun, 2 Aug 2015 17:08:10 +0000 (20:08 +0300)
commit358cde320b17535011b0a912e5c9ce1453666ed1
tree313a6d93540a6bd1aab531fbb9446f25ca7a998c
parenta4b09af3e93707351f1d4b8efafabfa135c0f70b
Fix race condition that lead to WALInsertLock deadlock with commit_delay.

If a call to WaitForXLogInsertionsToFinish() returned a value in the middle
of a page, and another backend then started to insert a record to the same
page, and then you called WaitXLogInsertionsToFinish() again, the second
call might return a smaller value than the first call. The problem was in
GetXLogBuffer(), which always updated the insertingAt value to the
beginning of the requested page, not the actual requested location. Because
of that, the second call might return a xlog pointer to the beginning of
the page, while the first one returned a later position on the same page.
XLogFlush() performs two calls to WaitXLogInsertionsToFinish() in
succession, and holds WALWriteLock on the second call, which can deadlock
if the second call to WaitXLogInsertionsToFinish() blocks.

Reported by Spiros Ioannou. Backpatch to 9.4, where the more scalable
WALInsertLock mechanism, and this bug, was introduced.
src/backend/access/transam/xlog.c