Re: [PATCH v2] grep: remove --mmap

[Paul Eggert]

Paolo Bonzini <address@hidden> 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.