> On 23 Jun 2022, at 12:50, Jakub Wartak <jakub.war...@tomtom.com> wrote:
>
> Thoughts?
I've looked into the patch one more time. And I propose to change this line
+ posix_fadvise(readFile, readOff + RACHUNK, RACHUNK,
POSIX_FADV_WILLNEED);
to
+ posix_fadvise(readFile, readOff + XLOG_BLCKSZ, RACHUNK,
POSIX_FADV_WILLNEED);
Currently first 128Kb of the file are not prefetched. But I expect that this
change will produce similar performance results. I propose this change only for
consistency, so we prefetch all data that we did not prefetch yet and going to
read. What do you think?
Best regards, Andrey Borodin.
