>> > On 21 Jun 2022, at 16:59, Jakub Wartak <jakub.war...@tomtom.com> wrote: >> Oh, wow, your benchmarks show really impressive improvement. >> >> > I think that 1 additional syscall is not going to be cheap just for >> > non-standard OS configurations >> Also we can reduce number of syscalls by something like >> >> #if defined(USE_POSIX_FADVISE) && defined(POSIX_FADV_WILLNEED) >> if ((readOff % (8 * XLOG_BLCKSZ)) == 0) >> posix_fadvise(readFile, readOff + XLOG_BLCKSZ, XLOG_BLCKSZ * 8, >> POSIX_FADV_WILLNEED); #endif >> >> and maybe define\reuse the some GUC to control number of prefetched pages >> at once.
Hi, I was thinking the same, so I got the patch (attached) to the point it gets the identical performance with and without readahead enabled: baseline, master, default Linux readahead (128kb): 33.979, 0.478 35.137, 0.504 34.649, 0.518 master+patched, readahead disabled: 34.338, 0.528 34.568, 0.575 34.007, 1.136 master+patched, readahead enabled (as default): 33.935, 0.523 34.109, 0.501 33.408, 0.557 Thoughts? Notes: - no GUC, as the default/identical value seems to be the best - POSIX_FADV_SEQUENTIAL is apparently much slower and doesn't seem to have effect from xlogreader.c at all while _WILLNEED does (testing again contradicts "common wisdom"?) -J.
0001-Use-fadvise-to-prefetch-WAL-in-xlogrecovery.patch
Description: 0001-Use-fadvise-to-prefetch-WAL-in-xlogrecovery.patch