Re: 048_vacuum_horizon_floor.pl hangs due to wakeup lost inside LockBufferForCleanup

From: Fujii Masao <masao(dot)fujii(at)gmail(dot)com>
To: Xuneng Zhou <xunengzhou(at)gmail(dot)com>
Cc: Andres Freund <andres(at)anarazel(dot)de>, Alexander Lakhin <exclusion(at)gmail(dot)com>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: 048_vacuum_horizon_floor.pl hangs due to wakeup lost inside LockBufferForCleanup
Date: 2026-06-26 13:40:57
Message-ID: CAHGQGwFy6hMPgPbwaKA9gumbDwnS7sR8f8orzzhyVgkajDV1rw@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Wed, Jun 24, 2026 at 1:12 AM Xuneng Zhou <xunengzhou(at)gmail(dot)com> wrote:
> Here is the updated patch. Please take a look.

While working on the deadlock detection issue on standby discussed in [1],
you pointed out that both the proposed patch there and this patch modify
LockBufferForCleanup(), and suggested applying this fix first.
So I took a look at it.

As far as I read the patch, it looks good to me, so I'm inclined to
commit it unless another committer on this thread would like to do so.
But any objections?

BTW, since the issue was introduced by commits 5310fac6e0f and
c75ebc657ffc, this fix only needs to be applied to master.
Is that correct?

Regards,

[1] https://postgr.es/m/44c24dcf-5710-410f-b1b6-d10b315f3d51@postgrespro.ru

--
Fujii Masao

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Pavel Borisov 2026-06-26 13:47:06 Re: Add SPLIT PARTITION/MERGE PARTITIONS commands
Previous Message Justin Pryzby 2026-06-26 13:40:14 Re: Add SPLIT PARTITION/MERGE PARTITIONS commands