On Sat, Jan 15, 2022 at 12:16:59PM +0500, Andrey Borodin wrote: > > 15 янв. 2022 г., в 03:20, Shawn Debnath <s...@amazon.com> написал(а): > > On Fri, Jan 14, 2022 at 05:28:38PM +0800, Julien Rouhaud wrote: > >>> PFA rebase of the patchset. Also I've added a patch to combine > >>> page_number, page_status, and page_dirty together to touch less > >>> cachelines. > >> > >> The cfbot reports some errors on the latest version of the patch: > >> > >> https://cirrus-ci.com/task/6121317215764480 > >> [...] > >> Could you send a new version? In the meantime I will switch the patch > >> status to Waiting on Author. > > > > I was planning on running a set of stress tests on these patches. Could > > we confirm which ones we plan to include in the commitfest? > > Many thanks for your interest. Here's the latest version.
This is failing to compile under linux and windows due to bitfield syntax. http://cfbot.cputube.org/andrey-borodin.html And compile warnings: slru.c: In function ‘SlruAdjustNSlots’: slru.c:161:2: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement] 161 | int nbanks = 1; | ^~~ slru.c: In function ‘SimpleLruReadPage_ReadOnly’: slru.c:530:2: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement] 530 | int bankstart = (pageno & shared->bank_mask) * shared->bank_size; | ^~~ Note that you can test the CI result using any github account without waiting for the cfbot. See ./src/tools/ci/README. -- Justin