From 954c1813ac042f7d6a2cd32d8dbf18153da91fca Mon Sep 17 00:00:00 2001 From: Tom Lane Date: Sun, 22 Oct 2006 20:34:54 +0000 Subject: [PATCH] Remove an unnecessary HOLD_INTERRUPTS/RESUME_INTERRUPTS pair. This was required back when RESUME_INTERRUPTS could actually execute ProcessInterrupts, but that hasn't been true since 2001... --- src/backend/storage/buffer/bufmgr.c | 11 ++--------- 1 file changed, 2 insertions(+), 9 deletions(-) diff --git a/src/backend/storage/buffer/bufmgr.c b/src/backend/storage/buffer/bufmgr.c index 5efd9cc7b13..45cdace1626 100644 --- a/src/backend/storage/buffer/bufmgr.c +++ b/src/backend/storage/buffer/bufmgr.c @@ -8,7 +8,7 @@ * * * IDENTIFICATION - * $PostgreSQL: pgsql/src/backend/storage/buffer/bufmgr.c,v 1.212 2006/10/04 00:29:57 momjian Exp $ + * $PostgreSQL: pgsql/src/backend/storage/buffer/bufmgr.c,v 1.213 2006/10/22 20:34:54 tgl Exp $ * *------------------------------------------------------------------------- */ @@ -694,21 +694,14 @@ retry: BufTableDelete(&oldTag, oldHash); /* - * Avoid accepting a cancel interrupt when we release the mapping lock; - * that would leave the buffer free but not on the freelist. (Which would - * not be fatal, since it'd get picked up again by the clock scanning - * code, but we'd rather be sure it gets to the freelist.) + * Done with mapping lock. */ - HOLD_INTERRUPTS(); - LWLockRelease(oldPartitionLock); /* * Insert the buffer at the head of the list of free buffers. */ StrategyFreeBuffer(buf, true); - - RESUME_INTERRUPTS(); } /* -- 2.39.5