Re: Fix bank selection logic in SLRU - Mailing list pgsql-hackers

From Robert Haas
Subject Re: Fix bank selection logic in SLRU
Date
Msg-id CA+TgmobsdBxApp7EK61Y_81G3dYHN9-M+6PmKz+pU51y58TMpg@mail.gmail.com
Whole thread Raw
In response to Re: Fix bank selection logic in SLRU  (Dilip Kumar <dilipbalaut@gmail.com>)
Responses Re: Fix bank selection logic in SLRU
List pgsql-hackers
On Tue, Dec 10, 2024 at 8:58 AM Dilip Kumar <dilipbalaut@gmail.com> wrote:
>> Bank selection code assumes that number of buffers is power of 2.
>> If the number of buffers is not power of 2 - only subset of buffers will be used. In worst case, e.g. 65 buffers,
everythingwill be buffered only in bank 64. 
>
> But why that would be the case? the acceptable values for GUC to configure the slru buffers are in multiple of
16(banksize) we have that check to check the GUC values. 

"Must be a multiple of 16" and "must be a power of 2" are different
criteria. For example, 48 is a multiple of 16 but it is not a power of
2. If the code assumes that we have an actual power of 2, the check
you quoted in your previous email is insufficient.

--
Robert Haas
EDB: http://www.enterprisedb.com



pgsql-hackers by date:

Previous
From: Andrew Dunstan
Date:
Subject: Re: XMLDocument (SQL/XML X030)
Next
From: Fabrice Chapuis
Date:
Subject: timeline problem when recovery