Allow DSM allocation to be interrupted.
authorThomas Munro <tmunro@postgresql.org>
Tue, 18 Sep 2018 10:56:36 +0000 (22:56 +1200)
committerThomas Munro <tmunro@postgresql.org>
Tue, 18 Sep 2018 11:49:21 +0000 (23:49 +1200)
commitc0c5668c6a073f22a1d39175693b1367ae7f09fa
treeb97549fae023f07a5d3e79eb79b25350e25ad1d8
parent8494755109e97ad22c9817b9dbe550b535961ed4
Allow DSM allocation to be interrupted.

Chris Travers reported that the startup process can repeatedly try to
cancel a backend that is in a posix_fallocate()/EINTR loop and cause it
to loop forever.  Teach the retry loop to give up if an interrupt is
pending.  Don't actually check for interrupts in that loop though,
because a non-local exit would skip some clean-up code in the caller.

Back-patch to 9.4 where DSM was added (and posix_fallocate() was later
back-patched).

Author: Chris Travers
Reviewed-by: Ildar Musin, Murat Kabilov, Oleksii Kliukin
Tested-by: Oleksii Kliukin
Discussion: https://postgr.es/m/CAN-RpxB-oeZve_J3SM_6%3DHXPmvEG%3DHX%2B9V9pi8g2YR7YW0rBBg%40mail.gmail.com
src/backend/storage/ipc/dsm_impl.c