Re: Why we need to check for local buffers in BufferIsExclusiveLocked and BufferIsDirty?

Srinath Reddy Sadipiralla <srinath2133@gmail.com>

From: Srinath Reddy <srinath2133@gmail.com>
To: Tom Lane <tgl@sss.pgh.pa.us>, pgsql-hackers@lists.postgresql.org
Date: 2025-01-26T16:54:50Z
Lists: pgsql-hackers
On Sun, Jan 26, 2025 at 9:49 PM Tom Lane <tgl@sss.pgh.pa.us> wrote:

> Srinath Reddy <srinath2133@gmail.com> writes:
> > as suggested did the changes and attached the patch for the same.
>
> Uh ... what in the world is the point of changing
> BufferIsExclusiveLocked's signature?
>
>                         regards, tom lane
>

as there was repeated code between BufferIsExclusiveLocked and
BufferIsDirty to check if buffer is pinned and its locked exclusively,i
thought it would be nice to move that repeated code into
BufferIsExclusiveLocked and as we need bufHdr in BufferIsDirty which is
assigned in BufferIsExclusiveLocked,so I had to change the signature of
BufferIsExclusiveLocked by adding (BufferDesc **bufHdr).

Regards,
Srinath Reddy Sadipiralla,
EDB: http://www.enterprisedb.com

Commits

  1. Make BufferIsExclusiveLocked and BufferIsDirty work for local buffers.