Re: slru bank

On Thu, Aug 22, 2024 at 7:27 PM Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:

> "David G. Johnston" <david(dot)g(dot)johnston(at)gmail(dot)com> writes:
> > On Thu, Aug 22, 2024 at 7:07 PM 席冲(宜穆) <xichong(dot)xc(at)alibaba-inc(dot)com>
> wrote:
> >> In SlruSelectLRUPage(), Why do we need to traverse all slots to find
> that
> >> a page already has a buffer assigned? Why not find it
> >> from the [bankstart,bankend]?
>
> > Only the bank is searched, both of the logic loops are bounded by:
> > for (int slotno = bankstart; slotno < bankend; slotno++)
>
> I think the OP has rediscovered the bug already fixed in 7b063ff26.
> That's post-17beta2, though.
>
>
Indeed. I was looking at HEAD.

David J.