diff options
author | Wu Fengguang <fengguang.wu@intel.com> | 2009-06-16 18:31:21 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2009-06-16 22:47:28 -0400 |
commit | caca7cb748571a5b39943a9b3e7081feef055e5e (patch) | |
tree | 8f115031584d8ebc14280079fed8183ef1f032ac /mm | |
parent | fc31d16add13773265cc53d59f2e7594cb3c0a14 (diff) |
readahead: remove one unnecessary radix tree lookup
(hit_readahead_marker != 0) means the page at @offset is present, so we
can search for non-present page starting from @offset+1.
Reported-by: Xu Chenfeng <xcf@ustc.edu.cn>
Signed-off-by: Wu Fengguang <fengguang.wu@intel.com>
Cc: Ying Han <yinghan@google.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'mm')
-rw-r--r-- | mm/readahead.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/mm/readahead.c b/mm/readahead.c index bd0f0e88b0a1..56731158309b 100644 --- a/mm/readahead.c +++ b/mm/readahead.c | |||
@@ -395,7 +395,7 @@ ondemand_readahead(struct address_space *mapping, | |||
395 | pgoff_t start; | 395 | pgoff_t start; |
396 | 396 | ||
397 | rcu_read_lock(); | 397 | rcu_read_lock(); |
398 | start = radix_tree_next_hole(&mapping->page_tree, offset,max+1); | 398 | start = radix_tree_next_hole(&mapping->page_tree, offset+1,max); |
399 | rcu_read_unlock(); | 399 | rcu_read_unlock(); |
400 | 400 | ||
401 | if (!start || start - offset > max) | 401 | if (!start || start - offset > max) |