diff options
author | Kent Overstreet <koverstreet@google.com> | 2012-09-05 18:22:02 -0400 |
---|---|---|
committer | Kent Overstreet <koverstreet@google.com> | 2013-03-23 17:26:30 -0400 |
commit | cb34e057ad22a1c2c6f2cb6cd1cbd05cc2f28f28 (patch) | |
tree | 4077b8e6e88bc7c5eb4cd738b4203401daba28a9 /mm | |
parent | d74c6d514fe314b8bdab58b487b25992291577ec (diff) |
block: Convert some code to bio_for_each_segment_all()
More prep work for immutable bvecs:
A few places in the code were either open coding or using the wrong
version - fix.
After we introduce the bvec iter, it'll no longer be possible to modify
the biovec through bio_for_each_segment_all() - it doesn't increment a
pointer to the current bvec, you pass in a struct bio_vec (not a
pointer) which is updated with what the current biovec would be (taking
into account bi_bvec_done and bi_size).
So because of that it's more worthwhile to be consistent about
bio_for_each_segment()/bio_for_each_segment_all() usage.
Signed-off-by: Kent Overstreet <koverstreet@google.com>
CC: Jens Axboe <axboe@kernel.dk>
CC: NeilBrown <neilb@suse.de>
CC: Alasdair Kergon <agk@redhat.com>
CC: dm-devel@redhat.com
CC: Alexander Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to 'mm')
-rw-r--r-- | mm/bounce.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/mm/bounce.c b/mm/bounce.c index 2ee1b6fef44a..f5326b24d65d 100644 --- a/mm/bounce.c +++ b/mm/bounce.c | |||
@@ -231,7 +231,7 @@ static void __blk_queue_bounce(struct request_queue *q, struct bio **bio_orig, | |||
231 | bounce: | 231 | bounce: |
232 | bio = bio_clone_bioset(*bio_orig, GFP_NOIO, fs_bio_set); | 232 | bio = bio_clone_bioset(*bio_orig, GFP_NOIO, fs_bio_set); |
233 | 233 | ||
234 | bio_for_each_segment(to, bio, i) { | 234 | bio_for_each_segment_all(to, bio, i) { |
235 | struct page *page = to->bv_page; | 235 | struct page *page = to->bv_page; |
236 | 236 | ||
237 | if (page_to_pfn(page) <= queue_bounce_pfn(q) && !force) | 237 | if (page_to_pfn(page) <= queue_bounce_pfn(q) && !force) |