Re: Elimination of the repetitive code at the SLRU bootstrap functions.

[Andrey Borodin]

> On 14 Feb 2025, at 11:54, Evgeny Voropaev <evgeny.vorop...@tantorlabs.com> 
> wrote:
> 
> <v2-0001-Elimination-of-the-repetitive-code-at-the-SLRU-bo.patch>

Hi! Nice patch!

BootStrapSlruPage() always calls zerofunc(pageno, false) with second argument 
false.
In case of every possible argument (ZeroCLOGPage, ZeroCommitTsPage, 
ZeroMultiXactOffsetPage, ZeroMultiXactMemberPage, ZeroSUBTRANSPage) it means 
just a call to SimpleLruZeroPage().
I think we can safely replace

+ slotno = (*zerofunc)(pageno, false);

with

+ slotno = SimpleLruZeroPage(pageno);

Thus we will not need zerofunc argument at all.

Thanks!


Best regards, Andrey Borodin.