diff options
author | Christoph Hellwig <hch@lst.de> | 2019-04-25 03:03:00 -0400 |
---|---|---|
committer | Jens Axboe <axboe@kernel.dk> | 2019-04-30 11:26:13 -0400 |
commit | 2b070cfe582b8e99fec6ada57d2e59e194aae202 (patch) | |
tree | 354961c24c2cc77cdaa66128c5f2a0997bc746e8 /fs/xfs | |
parent | f936b06ae53815a7633b30ffd8cf5661ac826b3a (diff) |
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>
Diffstat (limited to 'fs/xfs')
-rw-r--r-- | fs/xfs/xfs_aops.c | 3 |
1 files changed, 1 insertions, 2 deletions
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( | |||
98 | 98 | ||
99 | for (bio = &ioend->io_inline_bio; bio; bio = next) { | 99 | for (bio = &ioend->io_inline_bio; bio; bio = next) { |
100 | struct bio_vec *bvec; | 100 | struct bio_vec *bvec; |
101 | int i; | ||
102 | struct bvec_iter_all iter_all; | 101 | struct bvec_iter_all iter_all; |
103 | 102 | ||
104 | /* | 103 | /* |
@@ -111,7 +110,7 @@ xfs_destroy_ioend( | |||
111 | next = bio->bi_private; | 110 | next = bio->bi_private; |
112 | 111 | ||
113 | /* walk each page on bio, ending page IO on them */ | 112 | /* walk each page on bio, ending page IO on them */ |
114 | bio_for_each_segment_all(bvec, bio, i, iter_all) | 113 | bio_for_each_segment_all(bvec, bio, iter_all) |
115 | xfs_finish_page_writeback(inode, bvec, error); | 114 | xfs_finish_page_writeback(inode, bvec, error); |
116 | bio_put(bio); | 115 | bio_put(bio); |
117 | } | 116 | } |