Paolo Bonzini <[email protected]> writes: > mmap is a bad idea for sequentially accessed file because it will cause > a page fault for every read page.
That would not be true if grep used posix_madvise with the POSIX_MADV_SEQUENTIAL option, right? Shouldn't this should be tried out before removing the use of mmap? I'm raising this issue because, if I recall correctly, I'm the one who ripped out the madvise use, way back when madvise actually hurt performance in SunOS 4.1. It's pretty safe to say that SunOS 4.1 performance bugs are not relevant any more.
