If the file size is 20kb and readahead request is [0, 16kb), it's better to expand the readahead request to [0, 20kb), which will likely save one followup I/O for the ending [16kb, 20kb). If the readahead request already covers EOF, trimm it down to EOF. Also don't set the PG_readahead mark to avoid an unnecessary future invocation of the readahead code. This special handling looks worthwhile because small to medium sized files are pretty common. Acked-by: Jan Kara Signed-off-by: Wu Fengguang --- mm/readahead.c | 21 +++++++++++++++++++++ 1 file changed, 21 insertions(+) --- linux-next.orig/mm/readahead.c 2012-01-25 15:57:58.000000000 +0800 +++ linux-next/mm/readahead.c 2012-01-25 15:57:59.000000000 +0800 @@ -466,6 +466,25 @@ unsigned long max_sane_readahead(unsigne + node_page_state(numa_node_id(), NR_FREE_PAGES)) / 2); } +static void snap_to_eof(struct file_ra_state *ra, struct address_space *mapping) +{ + pgoff_t eof = ((i_size_read(mapping->host)-1) >> PAGE_CACHE_SHIFT) + 1; + pgoff_t start = ra->start; + unsigned int size = ra->size; + + /* + * skip backwards and random reads + */ + if (ra->pattern > RA_PATTERN_MMAP_AROUND) + return; + + size += min(size / 2, ra->ra_pages / 4); + if (start + size > eof) { + ra->size = eof - start; + ra->async_size = 0; + } +} + /* * Submit IO for the read-ahead request in file_ra_state. */ @@ -477,6 +496,8 @@ unsigned long ra_submit(struct file_ra_s { int actual; + snap_to_eof(ra, mapping); + actual = __do_page_cache_readahead(mapping, filp, ra->start, ra->size, ra->async_size); -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/