Commit 40f9c513 authored by Jan Kara's avatar Jan Kara Committed by Linus Torvalds

nilfs2: use pagevec_lookup_range_tag()

We want only pages from given range in nilfs_lookup_dirty_data_buffers().
Use pagevec_lookup_range_tag() instead of pagevec_lookup_tag() and
remove unnecessary code.

Link: http://lkml.kernel.org/r/20171009151359.31984-10-jack@suse.czSigned-off-by: default avatarJan Kara <jack@suse.cz>
Reviewed-by: default avatarDaniel Jordan <daniel.m.jordan@oracle.com>
Acked-by: default avatarRyusuke Konishi <konishi.ryusuke@lab.ntt.co.jp>
Cc: Ryusuke Konishi <konishi.ryusuke@lab.ntt.co.jp>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent d2bc5b3c
...@@ -711,18 +711,14 @@ static size_t nilfs_lookup_dirty_data_buffers(struct inode *inode, ...@@ -711,18 +711,14 @@ static size_t nilfs_lookup_dirty_data_buffers(struct inode *inode,
pagevec_init(&pvec, 0); pagevec_init(&pvec, 0);
repeat: repeat:
if (unlikely(index > last) || if (unlikely(index > last) ||
!pagevec_lookup_tag(&pvec, mapping, &index, PAGECACHE_TAG_DIRTY, !pagevec_lookup_range_tag(&pvec, mapping, &index, last,
min_t(pgoff_t, last - index, PAGECACHE_TAG_DIRTY, PAGEVEC_SIZE))
PAGEVEC_SIZE - 1) + 1))
return ndirties; return ndirties;
for (i = 0; i < pagevec_count(&pvec); i++) { for (i = 0; i < pagevec_count(&pvec); i++) {
struct buffer_head *bh, *head; struct buffer_head *bh, *head;
struct page *page = pvec.pages[i]; struct page *page = pvec.pages[i];
if (unlikely(page->index > last))
break;
lock_page(page); lock_page(page);
if (!page_has_buffers(page)) if (!page_has_buffers(page))
create_empty_buffers(page, i_blocksize(inode), 0); create_empty_buffers(page, i_blocksize(inode), 0);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment