mm: remove nr_pages argument from pagevec_lookup_{,range}_tag()
All users of pagevec_lookup() and pagevec_lookup_range() now pass PAGEVEC_SIZE as a desired number of pages. Just drop the argument. Link: http://lkml.kernel.org/r/20171009151359.31984-15-jack@suse.cz Signed-off-by: Jan Kara <jack@suse.cz> Reviewed-by: Daniel Jordan <daniel.m.jordan@oracle.com> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
This commit is contained in:
parent
feb94dc829
commit
91e7d9d2dd
14 changed files with 27 additions and 31 deletions
|
@ -3933,7 +3933,7 @@ retry:
|
|||
tag_pages_for_writeback(mapping, index, end);
|
||||
while (!done && !nr_to_write_done && (index <= end) &&
|
||||
(nr_pages = pagevec_lookup_range_tag(&pvec, mapping, &index, end,
|
||||
tag, PAGEVEC_SIZE))) {
|
||||
tag))) {
|
||||
unsigned i;
|
||||
|
||||
scanned = 1;
|
||||
|
@ -4071,8 +4071,8 @@ retry:
|
|||
if (wbc->sync_mode == WB_SYNC_ALL)
|
||||
tag_pages_for_writeback(mapping, index, end);
|
||||
while (!done && !nr_to_write_done && (index <= end) &&
|
||||
(nr_pages = pagevec_lookup_range_tag(&pvec, mapping, &index, end,
|
||||
tag, PAGEVEC_SIZE))) {
|
||||
(nr_pages = pagevec_lookup_range_tag(&pvec, mapping,
|
||||
&index, end, tag))) {
|
||||
unsigned i;
|
||||
|
||||
scanned = 1;
|
||||
|
|
|
@ -793,8 +793,7 @@ retry:
|
|||
get_more_pages:
|
||||
first = -1;
|
||||
pvec_pages = pagevec_lookup_range_tag(&pvec, mapping, &index,
|
||||
end, PAGECACHE_TAG_DIRTY,
|
||||
max_pages - locked_pages);
|
||||
end, PAGECACHE_TAG_DIRTY);
|
||||
dout("pagevec_lookup_range_tag got %d\n", pvec_pages);
|
||||
if (!pvec_pages && !locked_pages)
|
||||
break;
|
||||
|
|
|
@ -2368,7 +2368,7 @@ static int mpage_prepare_extent_to_map(struct mpage_da_data *mpd)
|
|||
mpd->next_page = index;
|
||||
while (index <= end) {
|
||||
nr_pages = pagevec_lookup_range_tag(&pvec, mapping, &index, end,
|
||||
tag, PAGEVEC_SIZE);
|
||||
tag);
|
||||
if (nr_pages == 0)
|
||||
goto out;
|
||||
|
||||
|
|
|
@ -326,7 +326,7 @@ long sync_meta_pages(struct f2fs_sb_info *sbi, enum page_type type,
|
|||
blk_start_plug(&plug);
|
||||
|
||||
while ((nr_pages = pagevec_lookup_tag(&pvec, mapping, &index,
|
||||
PAGECACHE_TAG_DIRTY, PAGEVEC_SIZE))) {
|
||||
PAGECACHE_TAG_DIRTY))) {
|
||||
int i;
|
||||
|
||||
for (i = 0; i < nr_pages; i++) {
|
||||
|
|
|
@ -1970,7 +1970,7 @@ retry:
|
|||
int i;
|
||||
|
||||
nr_pages = pagevec_lookup_range_tag(&pvec, mapping, &index, end,
|
||||
tag, PAGEVEC_SIZE);
|
||||
tag);
|
||||
if (nr_pages == 0)
|
||||
break;
|
||||
|
||||
|
|
|
@ -1308,7 +1308,7 @@ static struct page *last_fsync_dnode(struct f2fs_sb_info *sbi, nid_t ino)
|
|||
index = 0;
|
||||
|
||||
while ((nr_pages = pagevec_lookup_tag(&pvec, NODE_MAPPING(sbi), &index,
|
||||
PAGECACHE_TAG_DIRTY, PAGEVEC_SIZE))) {
|
||||
PAGECACHE_TAG_DIRTY))) {
|
||||
int i;
|
||||
|
||||
for (i = 0; i < nr_pages; i++) {
|
||||
|
@ -1497,7 +1497,7 @@ retry:
|
|||
index = 0;
|
||||
|
||||
while ((nr_pages = pagevec_lookup_tag(&pvec, NODE_MAPPING(sbi), &index,
|
||||
PAGECACHE_TAG_DIRTY, PAGEVEC_SIZE))) {
|
||||
PAGECACHE_TAG_DIRTY))) {
|
||||
int i;
|
||||
|
||||
for (i = 0; i < nr_pages; i++) {
|
||||
|
@ -1610,7 +1610,7 @@ next_step:
|
|||
index = 0;
|
||||
|
||||
while ((nr_pages = pagevec_lookup_tag(&pvec, NODE_MAPPING(sbi), &index,
|
||||
PAGECACHE_TAG_DIRTY, PAGEVEC_SIZE))) {
|
||||
PAGECACHE_TAG_DIRTY))) {
|
||||
int i;
|
||||
|
||||
for (i = 0; i < nr_pages; i++) {
|
||||
|
@ -1705,7 +1705,7 @@ int wait_on_node_pages_writeback(struct f2fs_sb_info *sbi, nid_t ino)
|
|||
pagevec_init(&pvec, 0);
|
||||
|
||||
while ((nr_pages = pagevec_lookup_tag(&pvec, NODE_MAPPING(sbi), &index,
|
||||
PAGECACHE_TAG_WRITEBACK, PAGEVEC_SIZE))) {
|
||||
PAGECACHE_TAG_WRITEBACK))) {
|
||||
int i;
|
||||
|
||||
for (i = 0; i < nr_pages; i++) {
|
||||
|
|
|
@ -367,7 +367,7 @@ retry:
|
|||
done_index = index;
|
||||
while (!done && (index <= end)) {
|
||||
nr_pages = pagevec_lookup_range_tag(&pvec, mapping, &index, end,
|
||||
tag, PAGEVEC_SIZE);
|
||||
tag);
|
||||
if (nr_pages == 0)
|
||||
break;
|
||||
|
||||
|
|
|
@ -2147,8 +2147,8 @@ static void nilfs_btree_lookup_dirty_buffers(struct nilfs_bmap *btree,
|
|||
|
||||
pagevec_init(&pvec, 0);
|
||||
|
||||
while (pagevec_lookup_tag(&pvec, btcache, &index, PAGECACHE_TAG_DIRTY,
|
||||
PAGEVEC_SIZE)) {
|
||||
while (pagevec_lookup_tag(&pvec, btcache, &index,
|
||||
PAGECACHE_TAG_DIRTY)) {
|
||||
for (i = 0; i < pagevec_count(&pvec); i++) {
|
||||
bh = head = page_buffers(pvec.pages[i]);
|
||||
do {
|
||||
|
|
|
@ -262,8 +262,7 @@ int nilfs_copy_dirty_pages(struct address_space *dmap,
|
|||
|
||||
pagevec_init(&pvec, 0);
|
||||
repeat:
|
||||
if (!pagevec_lookup_tag(&pvec, smap, &index, PAGECACHE_TAG_DIRTY,
|
||||
PAGEVEC_SIZE))
|
||||
if (!pagevec_lookup_tag(&pvec, smap, &index, PAGECACHE_TAG_DIRTY))
|
||||
return 0;
|
||||
|
||||
for (i = 0; i < pagevec_count(&pvec); i++) {
|
||||
|
@ -382,8 +381,8 @@ void nilfs_clear_dirty_pages(struct address_space *mapping, bool silent)
|
|||
|
||||
pagevec_init(&pvec, 0);
|
||||
|
||||
while (pagevec_lookup_tag(&pvec, mapping, &index, PAGECACHE_TAG_DIRTY,
|
||||
PAGEVEC_SIZE)) {
|
||||
while (pagevec_lookup_tag(&pvec, mapping, &index,
|
||||
PAGECACHE_TAG_DIRTY)) {
|
||||
for (i = 0; i < pagevec_count(&pvec); i++) {
|
||||
struct page *page = pvec.pages[i];
|
||||
|
||||
|
|
|
@ -706,7 +706,7 @@ static size_t nilfs_lookup_dirty_data_buffers(struct inode *inode,
|
|||
repeat:
|
||||
if (unlikely(index > last) ||
|
||||
!pagevec_lookup_range_tag(&pvec, mapping, &index, last,
|
||||
PAGECACHE_TAG_DIRTY, PAGEVEC_SIZE))
|
||||
PAGECACHE_TAG_DIRTY))
|
||||
return ndirties;
|
||||
|
||||
for (i = 0; i < pagevec_count(&pvec); i++) {
|
||||
|
@ -749,8 +749,8 @@ static void nilfs_lookup_dirty_node_buffers(struct inode *inode,
|
|||
|
||||
pagevec_init(&pvec, 0);
|
||||
|
||||
while (pagevec_lookup_tag(&pvec, mapping, &index, PAGECACHE_TAG_DIRTY,
|
||||
PAGEVEC_SIZE)) {
|
||||
while (pagevec_lookup_tag(&pvec, mapping, &index,
|
||||
PAGECACHE_TAG_DIRTY)) {
|
||||
for (i = 0; i < pagevec_count(&pvec); i++) {
|
||||
bh = head = page_buffers(pvec.pages[i]);
|
||||
do {
|
||||
|
|
|
@ -31,16 +31,14 @@ unsigned pagevec_lookup(struct pagevec *pvec, struct address_space *mapping,
|
|||
pgoff_t start, unsigned nr_pages);
|
||||
unsigned pagevec_lookup_range_tag(struct pagevec *pvec,
|
||||
struct address_space *mapping, pgoff_t *index, pgoff_t end,
|
||||
int tag, unsigned nr_pages);
|
||||
int tag);
|
||||
unsigned pagevec_lookup_range_nr_tag(struct pagevec *pvec,
|
||||
struct address_space *mapping, pgoff_t *index, pgoff_t end,
|
||||
int tag, unsigned max_pages);
|
||||
static inline unsigned pagevec_lookup_tag(struct pagevec *pvec,
|
||||
struct address_space *mapping, pgoff_t *index, int tag,
|
||||
unsigned nr_pages)
|
||||
struct address_space *mapping, pgoff_t *index, int tag)
|
||||
{
|
||||
return pagevec_lookup_range_tag(pvec, mapping, index, (pgoff_t)-1, tag,
|
||||
nr_pages);
|
||||
return pagevec_lookup_range_tag(pvec, mapping, index, (pgoff_t)-1, tag);
|
||||
}
|
||||
|
||||
static inline void pagevec_init(struct pagevec *pvec, int cold)
|
||||
|
|
|
@ -348,7 +348,7 @@ static int __filemap_fdatawait_range(struct address_space *mapping,
|
|||
unsigned i;
|
||||
|
||||
nr_pages = pagevec_lookup_range_tag(&pvec, mapping, &index,
|
||||
end, PAGECACHE_TAG_WRITEBACK, PAGEVEC_SIZE);
|
||||
end, PAGECACHE_TAG_WRITEBACK);
|
||||
if (!nr_pages)
|
||||
break;
|
||||
|
||||
|
|
|
@ -2188,7 +2188,7 @@ retry:
|
|||
int i;
|
||||
|
||||
nr_pages = pagevec_lookup_range_tag(&pvec, mapping, &index, end,
|
||||
tag, PAGEVEC_SIZE);
|
||||
tag);
|
||||
if (nr_pages == 0)
|
||||
break;
|
||||
|
||||
|
|
|
@ -1132,10 +1132,10 @@ EXPORT_SYMBOL(pagevec_lookup);
|
|||
|
||||
unsigned pagevec_lookup_range_tag(struct pagevec *pvec,
|
||||
struct address_space *mapping, pgoff_t *index, pgoff_t end,
|
||||
int tag, unsigned nr_pages)
|
||||
int tag)
|
||||
{
|
||||
pvec->nr = find_get_pages_range_tag(mapping, index, end, tag,
|
||||
nr_pages, pvec->pages);
|
||||
PAGEVEC_SIZE, pvec->pages);
|
||||
return pagevec_count(pvec);
|
||||
}
|
||||
EXPORT_SYMBOL(pagevec_lookup_range_tag);
|
||||
|
|
Loading…
Add table
Reference in a new issue