Re: Add pg_buffercache_mark_dirty[_all] functions to the pg_buffercache

From: Nazir Bilal Yavuz <byavuz81(at)gmail(dot)com>
To: 邱宇航 <iamqyh(at)gmail(dot)com>
Cc: pgsql-hackers(at)lists(dot)postgresql(dot)org
Subject: Re: Add pg_buffercache_mark_dirty[_all] functions to the pg_buffercache
Date: 2025-11-27 08:00:03
Message-ID: CAN55FZ1S_e32YoKjeA9rco7+a9RpaB4vrFQJZfjXTu=JoiVJ3w@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hi,

On Thu, 27 Nov 2025 at 05:51, 邱宇航 <iamqyh(at)gmail(dot)com> wrote:
>
> > I do not think that will be a problem but I can change it if the
> > general consensus is towards this way. Also, if we change this for
> > pg_buffercache_mark_dirty_* functions, I think we need to apply the
> > same for the pg_buffercache_evict_* functions.
>
> After some testing, bgwriter/checkpointer didn' blocks the mark buffer
> dirty SQL. it's ok to use LWLockAcquire.

Thanks for testing this!

> There is an extra line break after elog(ERROR, "bad buffer ID: %d", buf)
> which can be removed.

Done in the v10. I wonder why pgindent did not catch this.

--
Regards,
Nazir Bilal Yavuz
Microsoft

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Mahendra Singh Thalor 2025-11-27 08:15:12 Re: Non-text mode for pg_dumpall
Previous Message Nazir Bilal Yavuz 2025-11-27 07:59:47 Re: Add pg_buffercache_mark_dirty[_all] functions to the pg_buffercache