Please "reply to all" to keep the Cc list.
Vasyl' Vavrychuk schrieb:
quoted
Did you notice any differences with this? Or is this change just
because-we-can?
Not yet.
This patch makes sense only if there is a noticable speed improvement. I
doubt that the gain is measurable in the warm-cache case, but in the
cold-cache case we can perhaps indeed save a number of disk accesses if
whole pack files need not be slurped in (which the current mmap
replacements do).
Maybe write own getpagesize() based on GetSystemInfo()?
It's probably worth it (but only if mmap() makes sense at all).
getpagesize() is used in the "sliding window" code to access pack files,
and then the offset is non-zero and must be aligned properly. Make this a
separate commit.
-- Hannes