fix comment for MAX_SIMUL_LWLOCKS

Started by Nathan Bossart5 months ago3 messages
#1Nathan Bossart
nathandbossart@gmail.com
1 attachment(s)

This comment mentions that pg_buffercache locks all buffer partitions
simultaneously, but it hasn't done so since v10 (see commit 6e654546fb).

--
nathan

Attachments:

v1-0001-Fix-comment-for-MAX_SIMUL_LWLOCKS.patchtext/plain; charset=us-asciiDownload
From 68b81e3bf70d5da0a0e2d0a0087218df7fde1101 Mon Sep 17 00:00:00 2001
From: Nathan Bossart <nathan@postgresql.org>
Date: Tue, 19 Aug 2025 16:27:33 -0500
Subject: [PATCH v1 1/1] Fix comment for MAX_SIMUL_LWLOCKS.

This comment mentions that pg_buffercache locks all buffer
partitions simultaneously, but it hasn't done so since v10.

Oversight in commit 6e654546fb.
---
 src/backend/storage/lmgr/lwlock.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/src/backend/storage/lmgr/lwlock.c b/src/backend/storage/lmgr/lwlock.c
index ec9c345ffdf..c80b43f1f55 100644
--- a/src/backend/storage/lmgr/lwlock.c
+++ b/src/backend/storage/lmgr/lwlock.c
@@ -162,8 +162,7 @@ LWLockPadded *MainLWLockArray = NULL;
 /*
  * We use this structure to keep track of locked LWLocks for release
  * during error recovery.  Normally, only a few will be held at once, but
- * occasionally the number can be much higher; for example, the pg_buffercache
- * extension locks all buffer partitions simultaneously.
+ * occasionally the number can be much higher.
  */
 #define MAX_SIMUL_LWLOCKS	200
 
-- 
2.39.5 (Apple Git-154)

#2Andres Freund
andres@anarazel.de
In reply to: Nathan Bossart (#1)
Re: fix comment for MAX_SIMUL_LWLOCKS

Hi,

On 2025-08-19 16:34:56 -0500, Nathan Bossart wrote:

From 68b81e3bf70d5da0a0e2d0a0087218df7fde1101 Mon Sep 17 00:00:00 2001
From: Nathan Bossart <nathan@postgresql.org>
Date: Tue, 19 Aug 2025 16:27:33 -0500
Subject: [PATCH v1 1/1] Fix comment for MAX_SIMUL_LWLOCKS.

This comment mentions that pg_buffercache locks all buffer
partitions simultaneously, but it hasn't done so since v10.

Oversight in commit 6e654546fb.

LGTM

#3Nathan Bossart
nathandbossart@gmail.com
In reply to: Andres Freund (#2)
Re: fix comment for MAX_SIMUL_LWLOCKS

On Tue, Aug 19, 2025 at 05:40:23PM -0400, Andres Freund wrote:

LGTM

Thanks, committed.

--
nathan