diff options
| author | Andres Freund <andres@anarazel.de> | 2015-02-03 23:25:00 +0100 |
|---|---|---|
| committer | Andres Freund <andres@anarazel.de> | 2015-02-03 23:25:00 +0100 |
| commit | d06995710bc7e347d39866c1793ae282498d65e0 (patch) | |
| tree | 5d4de1a97f5518baba36e0b68864ac84e67ceef0 /src/backend/storage/lmgr/lwlock.c | |
| parent | 6753333f55e1d9bcb9da4323556b456583624a07 (diff) | |
| download | postgresql-d06995710bc7e347d39866c1793ae282498d65e0.tar.gz | |
Remove the option to service interrupts during PGSemaphoreLock().
The remaining caller (lwlocks) doesn't need that facility, and we plan
to remove ImmedidateInterruptOK entirely. That means that interrupts
can't be serviced race-free and portably anyway, so there's little
reason for keeping the feature.
Reviewed-By: Heikki Linnakangas
Diffstat (limited to 'src/backend/storage/lmgr/lwlock.c')
| -rw-r--r-- | src/backend/storage/lmgr/lwlock.c | 12 |
1 files changed, 4 insertions, 8 deletions
diff --git a/src/backend/storage/lmgr/lwlock.c b/src/backend/storage/lmgr/lwlock.c index 7cb002357a..7ca8dc0007 100644 --- a/src/backend/storage/lmgr/lwlock.c +++ b/src/backend/storage/lmgr/lwlock.c @@ -863,8 +863,7 @@ LWLockDequeueSelf(LWLock *lock) */ for (;;) { - /* "false" means cannot accept cancel/die interrupt here. */ - PGSemaphoreLock(&MyProc->sem, false); + PGSemaphoreLock(&MyProc->sem); if (!MyProc->lwWaiting) break; extraWaits++; @@ -1034,8 +1033,7 @@ LWLockAcquireCommon(LWLock *lock, LWLockMode mode, uint64 *valptr, uint64 val) for (;;) { - /* "false" means cannot accept cancel/die interrupt here. */ - PGSemaphoreLock(&proc->sem, false); + PGSemaphoreLock(&proc->sem); if (!proc->lwWaiting) break; extraWaits++; @@ -1195,8 +1193,7 @@ LWLockAcquireOrWait(LWLock *lock, LWLockMode mode) for (;;) { - /* "false" means cannot accept cancel/die interrupt here. */ - PGSemaphoreLock(&proc->sem, false); + PGSemaphoreLock(&proc->sem); if (!proc->lwWaiting) break; extraWaits++; @@ -1397,8 +1394,7 @@ LWLockWaitForVar(LWLock *lock, uint64 *valptr, uint64 oldval, uint64 *newval) for (;;) { - /* "false" means cannot accept cancel/die interrupt here. */ - PGSemaphoreLock(&proc->sem, false); + PGSemaphoreLock(&proc->sem); if (!proc->lwWaiting) break; extraWaits++; |