block: remove the i argument to bio_for_each_segment_all

We only have two callers that need the integer loop iterator, and they
can easily maintain it themselves.

Suggested-by: Matthew Wilcox <willy@infradead.org>
Reviewed-by: Johannes Thumshirn <jthumshirn@suse.de>
Acked-by: David Sterba <dsterba@suse.com>
Reviewed-by: Hannes Reinecke <hare@suse.com>
Acked-by: Coly Li <colyli@suse.de>
Reviewed-by: Matthew Wilcox <willy@infradead.org>
Signed-off-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Jens Axboe <axboe@kernel.dk>

16 files changed, 27 insertions, 46 deletions

diff --git a/fs/block_dev.c b/fs/block_dev.c
index 24615c76c1d0..8abc6570d29f 100644
--- a/fs/block_dev.c
+++ b/fs/block_dev.c
@@ -210,7 +210,6 @@ __blkdev_direct_IO_simple(struct kiocb *iocb, struct iov_iter *iter,
         struct bio bio;
         ssize_t ret;
         blk_qc_t qc;
-        int i;
         struct bvec_iter_all iter_all;
 
         if ((pos | iov_iter_alignment(iter)) &
@@ -261,7 +260,7 @@ __blkdev_direct_IO_simple(struct kiocb *iocb, struct iov_iter *iter,
         }
         __set_current_state(TASK_RUNNING);
 
-        bio_for_each_segment_all(bvec, &bio, i, iter_all) {
+        bio_for_each_segment_all(bvec, &bio, iter_all) {
                 if (should_dirty && !PageCompound(bvec->bv_page))
                         set_page_dirty_lock(bvec->bv_page);
                 put_page(bvec->bv_page);
@@ -339,9 +338,8 @@ static void blkdev_bio_end_io(struct bio *bio)
                 if (!bio_flagged(bio, BIO_NO_PAGE_REF)) {
                         struct bvec_iter_all iter_all;
                         struct bio_vec *bvec;
-                        int i;
 
-                        bio_for_each_segment_all(bvec, bio, i, iter_all)
+                        bio_for_each_segment_all(bvec, bio, iter_all)
                                 put_page(bvec->bv_page);
                 }
                 bio_put(bio);
diff --git a/fs/btrfs/compression.c b/fs/btrfs/compression.c
index 4f2a8ae0aa42..6313dc65209e 100644
--- a/fs/btrfs/compression.c
+++ b/fs/btrfs/compression.c
@@ -160,7 +160,6 @@ csum_failed:
         if (cb->errors) {
                 bio_io_error(cb->orig_bio);
         } else {
-                int i;
                 struct bio_vec *bvec;
                 struct bvec_iter_all iter_all;
 
@@ -169,7 +168,7 @@ csum_failed:
                  * checked so the end_io handlers know about it
                  */
                 ASSERT(!bio_flagged(bio, BIO_CLONED));
-                bio_for_each_segment_all(bvec, cb->orig_bio, i, iter_all)
+                bio_for_each_segment_all(bvec, cb->orig_bio, iter_all)
                         SetPageChecked(bvec->bv_page);
 
                 bio_endio(cb->orig_bio);
diff --git a/fs/btrfs/disk-io.c b/fs/btrfs/disk-io.c
index 6fe9197f6ee4..c333e79408ff 100644
--- a/fs/btrfs/disk-io.c
+++ b/fs/btrfs/disk-io.c
@@ -832,11 +832,11 @@ static blk_status_t btree_csum_one_bio(struct bio *bio)
 {
         struct bio_vec *bvec;
         struct btrfs_root *root;
-        int i, ret = 0;
+        int ret = 0;
         struct bvec_iter_all iter_all;
 
         ASSERT(!bio_flagged(bio, BIO_CLONED));
-        bio_for_each_segment_all(bvec, bio, i, iter_all) {
+        bio_for_each_segment_all(bvec, bio, iter_all) {
                 root = BTRFS_I(bvec->bv_page->mapping->host)->root;
                 ret = csum_dirty_buffer(root->fs_info, bvec->bv_page);
                 if (ret)
diff --git a/fs/btrfs/extent_io.c b/fs/btrfs/extent_io.c
index ca8b8e785cf3..c85505c36fa6 100644
--- a/fs/btrfs/extent_io.c
+++ b/fs/btrfs/extent_io.c
@@ -2451,11 +2451,10 @@ static void end_bio_extent_writepage(struct bio *bio)
         struct bio_vec *bvec;
         u64 start;
         u64 end;
-        int i;
         struct bvec_iter_all iter_all;
 
         ASSERT(!bio_flagged(bio, BIO_CLONED));
-        bio_for_each_segment_all(bvec, bio, i, iter_all) {
+        bio_for_each_segment_all(bvec, bio, iter_all) {
                 struct page *page = bvec->bv_page;
                 struct inode *inode = page->mapping->host;
                 struct btrfs_fs_info *fs_info = btrfs_sb(inode->i_sb);
@@ -2523,11 +2522,10 @@ static void end_bio_extent_readpage(struct bio *bio)
         u64 extent_len = 0;
         int mirror;
         int ret;
-        int i;
         struct bvec_iter_all iter_all;
 
         ASSERT(!bio_flagged(bio, BIO_CLONED));
-        bio_for_each_segment_all(bvec, bio, i, iter_all) {
+        bio_for_each_segment_all(bvec, bio, iter_all) {
                 struct page *page = bvec->bv_page;
                 struct inode *inode = page->mapping->host;
                 struct btrfs_fs_info *fs_info = btrfs_sb(inode->i_sb);
@@ -3643,11 +3641,11 @@ static void end_bio_extent_buffer_writepage(struct bio *bio)
 {
         struct bio_vec *bvec;
         struct extent_buffer *eb;
-        int i, done;
+        int done;
         struct bvec_iter_all iter_all;
 
         ASSERT(!bio_flagged(bio, BIO_CLONED));
-        bio_for_each_segment_all(bvec, bio, i, iter_all) {
+        bio_for_each_segment_all(bvec, bio, iter_all) {
                 struct page *page = bvec->bv_page;
 
                 eb = (struct extent_buffer *)page->private;
diff --git a/fs/btrfs/inode.c b/fs/btrfs/inode.c
index 82fdda8ff5ab..10a8d08d3d29 100644
--- a/fs/btrfs/inode.c
+++ b/fs/btrfs/inode.c
@@ -7828,7 +7828,6 @@ static void btrfs_retry_endio_nocsum(struct bio *bio)
         struct inode *inode = done->inode;
         struct bio_vec *bvec;
         struct extent_io_tree *io_tree, *failure_tree;
-        int i;
         struct bvec_iter_all iter_all;
 
         if (bio->bi_status)
@@ -7841,7 +7840,7 @@ static void btrfs_retry_endio_nocsum(struct bio *bio)
 
         done->uptodate = 1;
         ASSERT(!bio_flagged(bio, BIO_CLONED));
-        bio_for_each_segment_all(bvec, bio, i, iter_all)
+        bio_for_each_segment_all(bvec, bio, iter_all)
                 clean_io_failure(BTRFS_I(inode)->root->fs_info, failure_tree,
                                  io_tree, done->start, bvec->bv_page,
                                  btrfs_ino(BTRFS_I(inode)), 0);
@@ -7919,7 +7918,7 @@ static void btrfs_retry_endio(struct bio *bio)
         struct bio_vec *bvec;
         int uptodate;
         int ret;
-        int i;
+        int i = 0;
         struct bvec_iter_all iter_all;
 
         if (bio->bi_status)
@@ -7934,7 +7933,7 @@ static void btrfs_retry_endio(struct bio *bio)
         failure_tree = &BTRFS_I(inode)->io_failure_tree;
 
         ASSERT(!bio_flagged(bio, BIO_CLONED));
-        bio_for_each_segment_all(bvec, bio, i, iter_all) {
+        bio_for_each_segment_all(bvec, bio, iter_all) {
                 ret = __readpage_endio_check(inode, io_bio, i, bvec->bv_page,
                                              bvec->bv_offset, done->start,
                                              bvec->bv_len);
@@ -7946,6 +7945,7 @@ static void btrfs_retry_endio(struct bio *bio)
                                          bvec->bv_offset);
                 else
                         uptodate = 0;
+                i++;
         }
 
         done->uptodate = uptodate;
diff --git a/fs/btrfs/raid56.c b/fs/btrfs/raid56.c
index 67a6f7d47402..f3d0576dd327 100644
--- a/fs/btrfs/raid56.c
+++ b/fs/btrfs/raid56.c
@@ -1442,12 +1442,11 @@ static int fail_bio_stripe(struct btrfs_raid_bio *rbio,
 static void set_bio_pages_uptodate(struct bio *bio)
 {
         struct bio_vec *bvec;
-        int i;
         struct bvec_iter_all iter_all;
 
         ASSERT(!bio_flagged(bio, BIO_CLONED));
 
-        bio_for_each_segment_all(bvec, bio, i, iter_all)
+        bio_for_each_segment_all(bvec, bio, iter_all)
                 SetPageUptodate(bvec->bv_page);
 }
 
diff --git a/fs/crypto/bio.c b/fs/crypto/bio.c
index 5759bcd018cd..8f3a8bc15d98 100644
--- a/fs/crypto/bio.c
+++ b/fs/crypto/bio.c
@@ -29,10 +29,9 @@
 static void __fscrypt_decrypt_bio(struct bio *bio, bool done)
 {
         struct bio_vec *bv;
-        int i;
         struct bvec_iter_all iter_all;
 
-        bio_for_each_segment_all(bv, bio, i, iter_all) {
+        bio_for_each_segment_all(bv, bio, iter_all) {
                 struct page *page = bv->bv_page;
                 int ret = fscrypt_decrypt_page(page->mapping->host, page,
                                 PAGE_SIZE, 0, page->index);
diff --git a/fs/direct-io.c b/fs/direct-io.c
index 9bb015bc4a83..fbe885d68035 100644
--- a/fs/direct-io.c
+++ b/fs/direct-io.c
@@ -538,7 +538,6 @@ static struct bio *dio_await_one(struct dio *dio)
 static blk_status_t dio_bio_complete(struct dio *dio, struct bio *bio)
 {
         struct bio_vec *bvec;
-        unsigned i;
         blk_status_t err = bio->bi_status;
 
         if (err) {
@@ -553,7 +552,7 @@ static blk_status_t dio_bio_complete(struct dio *dio, struct bio *bio)
         } else {
                 struct bvec_iter_all iter_all;
 
-                bio_for_each_segment_all(bvec, bio, i, iter_all) {
+                bio_for_each_segment_all(bvec, bio, iter_all) {
                         struct page *page = bvec->bv_page;
 
                         if (dio->op == REQ_OP_READ && !PageCompound(page) &&
diff --git a/fs/ext4/page-io.c b/fs/ext4/page-io.c
index 3e9298e6a705..4690618a92e9 100644
--- a/fs/ext4/page-io.c
+++ b/fs/ext4/page-io.c
@@ -61,11 +61,10 @@ static void buffer_io_error(struct buffer_head *bh)
 
 static void ext4_finish_bio(struct bio *bio)
 {
-        int i;
         struct bio_vec *bvec;
         struct bvec_iter_all iter_all;
 
-        bio_for_each_segment_all(bvec, bio, i, iter_all) {
+        bio_for_each_segment_all(bvec, bio, iter_all) {
                 struct page *page = bvec->bv_page;
 #ifdef CONFIG_FS_ENCRYPTION
                 struct page *data_page = NULL;
diff --git a/fs/ext4/readpage.c b/fs/ext4/readpage.c
index 3adadf461825..3629a74b7f94 100644
--- a/fs/ext4/readpage.c
+++ b/fs/ext4/readpage.c
@@ -71,7 +71,6 @@ static inline bool ext4_bio_encrypted(struct bio *bio)
 static void mpage_end_io(struct bio *bio)
 {
         struct bio_vec *bv;
-        int i;
         struct bvec_iter_all iter_all;
 
         if (ext4_bio_encrypted(bio)) {
@@ -82,7 +81,7 @@ static void mpage_end_io(struct bio *bio)
                         return;
                 }
         }
-        bio_for_each_segment_all(bv, bio, i, iter_all) {
+        bio_for_each_segment_all(bv, bio, iter_all) {
                 struct page *page = bv->bv_page;
 
                 if (!bio->bi_status) {
diff --git a/fs/f2fs/data.c b/fs/f2fs/data.c
index 9727944139f2..64040e998439 100644
--- a/fs/f2fs/data.c
+++ b/fs/f2fs/data.c
@@ -86,10 +86,9 @@ static void __read_end_io(struct bio *bio)
 {
         struct page *page;
         struct bio_vec *bv;
-        int i;
         struct bvec_iter_all iter_all;
 
-        bio_for_each_segment_all(bv, bio, i, iter_all) {
+        bio_for_each_segment_all(bv, bio, iter_all) {
                 page = bv->bv_page;
 
                 /* PG_error was set if any post_read step failed */
@@ -164,7 +163,6 @@ static void f2fs_write_end_io(struct bio *bio)
 {
         struct f2fs_sb_info *sbi = bio->bi_private;
         struct bio_vec *bvec;
-        int i;
         struct bvec_iter_all iter_all;
 
         if (time_to_inject(sbi, FAULT_WRITE_IO)) {
@@ -172,7 +170,7 @@ static void f2fs_write_end_io(struct bio *bio)
                 bio->bi_status = BLK_STS_IOERR;
         }
 
-        bio_for_each_segment_all(bvec, bio, i, iter_all) {
+        bio_for_each_segment_all(bvec, bio, iter_all) {
                 struct page *page = bvec->bv_page;
                 enum count_type type = WB_DATA_TYPE(page);
 
@@ -349,7 +347,6 @@ static bool __has_merged_page(struct f2fs_bio_info *io, struct inode *inode,
 {
         struct bio_vec *bvec;
         struct page *target;
-        int i;
         struct bvec_iter_all iter_all;
 
         if (!io->bio)
@@ -358,7 +355,7 @@ static bool __has_merged_page(struct f2fs_bio_info *io, struct inode *inode,
         if (!inode && !page && !ino)
                 return true;
 
-        bio_for_each_segment_all(bvec, io->bio, i, iter_all) {
+        bio_for_each_segment_all(bvec, io->bio, iter_all) {
 
                 if (bvec->bv_page->mapping)
                         target = bvec->bv_page;
diff --git a/fs/gfs2/lops.c b/fs/gfs2/lops.c
index 8722c60b11fe..6f09b5e3dd6e 100644
--- a/fs/gfs2/lops.c
+++ b/fs/gfs2/lops.c
@@ -207,7 +207,6 @@ static void gfs2_end_log_write(struct bio *bio)
         struct gfs2_sbd *sdp = bio->bi_private;
         struct bio_vec *bvec;
         struct page *page;
-        int i;
         struct bvec_iter_all iter_all;
 
         if (bio->bi_status) {
@@ -216,7 +215,7 @@ static void gfs2_end_log_write(struct bio *bio)
                 wake_up(&sdp->sd_logd_waitq);
         }
 
-        bio_for_each_segment_all(bvec, bio, i, iter_all) {
+        bio_for_each_segment_all(bvec, bio, iter_all) {
                 page = bvec->bv_page;
                 if (page_has_buffers(page))
                         gfs2_end_log_write_bh(sdp, bvec, bio->bi_status);
diff --git a/fs/gfs2/meta_io.c b/fs/gfs2/meta_io.c
index 3201342404a7..ff86e1d4f8ff 100644
--- a/fs/gfs2/meta_io.c
+++ b/fs/gfs2/meta_io.c
@@ -189,10 +189,9 @@ struct buffer_head *gfs2_meta_new(struct gfs2_glock *gl, u64 blkno)
 static void gfs2_meta_read_endio(struct bio *bio)
 {
         struct bio_vec *bvec;
-        int i;
         struct bvec_iter_all iter_all;
 
-        bio_for_each_segment_all(bvec, bio, i, iter_all) {
+        bio_for_each_segment_all(bvec, bio, iter_all) {
                 struct page *page = bvec->bv_page;
                 struct buffer_head *bh = page_buffers(page);
                 unsigned int len = bvec->bv_len;
diff --git a/fs/iomap.c b/fs/iomap.c
index abdd18e404f8..12a656271076 100644
--- a/fs/iomap.c
+++ b/fs/iomap.c
@@ -273,10 +273,9 @@ iomap_read_end_io(struct bio *bio)
 {
         int error = blk_status_to_errno(bio->bi_status);
         struct bio_vec *bvec;
-        int i;
         struct bvec_iter_all iter_all;
 
-        bio_for_each_segment_all(bvec, bio, i, iter_all)
+        bio_for_each_segment_all(bvec, bio, iter_all)
                 iomap_read_page_end_io(bvec, error);
         bio_put(bio);
 }
@@ -1592,9 +1591,8 @@ static void iomap_dio_bio_end_io(struct bio *bio)
                 if (!bio_flagged(bio, BIO_NO_PAGE_REF)) {
                         struct bvec_iter_all iter_all;
                         struct bio_vec *bvec;
-                        int i;
 
-                        bio_for_each_segment_all(bvec, bio, i, iter_all)
+                        bio_for_each_segment_all(bvec, bio, iter_all)
                                 put_page(bvec->bv_page);
                 }
                 bio_put(bio);
diff --git a/fs/mpage.c b/fs/mpage.c
index 3f19da75178b..436a85260394 100644
--- a/fs/mpage.c
+++ b/fs/mpage.c
@@ -47,10 +47,9 @@
 static void mpage_end_io(struct bio *bio)
 {
         struct bio_vec *bv;
-        int i;
         struct bvec_iter_all iter_all;
 
-        bio_for_each_segment_all(bv, bio, i, iter_all) {
+        bio_for_each_segment_all(bv, bio, iter_all) {
                 struct page *page = bv->bv_page;
                 page_endio(page, bio_op(bio),
                            blk_status_to_errno(bio->bi_status));
diff --git a/fs/xfs/xfs_aops.c b/fs/xfs/xfs_aops.c
index 3619e9e8d359..47941bbaac02 100644
--- a/fs/xfs/xfs_aops.c
+++ b/fs/xfs/xfs_aops.c
@@ -98,7 +98,6 @@ xfs_destroy_ioend(
 
         for (bio = &ioend->io_inline_bio; bio; bio = next) {
                 struct bio_vec  *bvec;
-                int             i;
                 struct bvec_iter_all iter_all;
 
                 /*
@@ -111,7 +110,7 @@ xfs_destroy_ioend(
                         next = bio->bi_private;
 
                 /* walk each page on bio, ending page IO on them */
-                bio_for_each_segment_all(bvec, bio, i, iter_all)
+                bio_for_each_segment_all(bvec, bio, iter_all)
                         xfs_finish_page_writeback(inode, bvec, error);
                 bio_put(bio);
         }