2010/7/29 Miao Xie <mi...@cn.fujitsu.com>: > From: Liu Bo <liubo2...@cn.fujitsu.com> > > Only when a page is not found by page_index, we'll go to the error check. > > Signed-off-by: Liu Bo <liubo2...@cn.fujitsu.com> > Signed-off-by: Miao Xie <mi...@cn.fujitsu.com> > --- > fs/btrfs/compression.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/fs/btrfs/compression.c b/fs/btrfs/compression.c > index cb3877c..8458840 100644 > --- a/fs/btrfs/compression.c > +++ b/fs/btrfs/compression.c > @@ -467,7 +467,7 @@ static noinline int add_ra_bio_pages(struct inode *inode, > rcu_read_lock(); > page = radix_tree_lookup(&mapping->page_tree, page_index); > rcu_read_unlock(); > - if (page) { > + if (!page) { > check_misses: > misses++; > if (misses > 4)
This patch is wrong. The word "miss" here means "miss for read-ahead because the page is already in the page cache" Yan, Zheng -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html