]> granicus.if.org Git - postgresql/commit
Fix WaitLatch() to return promptly when the requested timeout expires.
authorTom Lane <tgl@sss.pgh.pa.us>
Fri, 9 Nov 2012 01:04:48 +0000 (20:04 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Fri, 9 Nov 2012 01:04:48 +0000 (20:04 -0500)
commit3e7fdcffd6f77187b72b0496d61d782932973af6
treec519f69bb53eff6a49213de71f40091549ac265b
parentdcc55dd21aa2ba18b28ba9fa441885ef9c99d3c6
Fix WaitLatch() to return promptly when the requested timeout expires.

If the sleep is interrupted by a signal, we must recompute the remaining
time to wait; otherwise, a steady stream of non-wait-terminating interrupts
could delay return from WaitLatch indefinitely.  This has been shown to be
a problem for the autovacuum launcher, and there may well be other places
now or in the future with similar issues.  So we'd better make the function
robust, even though this'll add at least one gettimeofday call per wait.

Back-patch to 9.2.  We might eventually need to fix 9.1 as well, but the
code is quite different there, and the usage of WaitLatch in 9.1 is so
limited that it's not clearly important to do so.

Reported and diagnosed by Jeff Janes, though I rewrote his patch rather
heavily.
src/backend/port/unix_latch.c
src/backend/port/win32_latch.c
src/include/storage/latch.h