pgsql: Make BufferIsExclusiveLocked and BufferIsDirty work for local bu

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: Make BufferIsExclusiveLocked and BufferIsDirty work for local bu
Date: 2025-01-29 18:23:44
Message-ID: E1tdCj1-004G5R-VB@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Make BufferIsExclusiveLocked and BufferIsDirty work for local buffers.

These functions tried to check the state of the buffer's content lock
even for local buffers. Since we don't use the content lock for a
local buffer, that would lead to a "false" result from
LWLockHeldByMeInMode, which would mean a misleading "false" answer
from BufferIsExclusiveLocked (we'd rather that case always return
"true") or an assertion failure in BufferIsDirty.

The core code never applies these two functions to local buffers,
and apparently no extensions do either, since we've not heard
complaints. Still, in the name of future-proofing, let's fix
them to act as though a pinned local buffer is content-locked.

Author: Srinath Reddy <srinath2133(at)gmail(dot)com>
Discussion: https://postgr.es/m/19396ef77f8.1098c4a1810508.2255483659262451647@zohocorp.com

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/f6ff75f79689d2d0c5bb241435d7fc6a63273223

Modified Files
--------------
src/backend/storage/buffer/bufmgr.c | 22 +++++++++++-----------
1 file changed, 11 insertions(+), 11 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Tom Lane 2025-01-29 19:24:48 pgsql: Avoid breaking SJIS encoding while de-backslashing Windows paths
Previous Message John Naylor 2025-01-29 07:43:21 pgsql: Fix grammatical typos around possessive "its"