diff options
author | Kent Overstreet <koverstreet@google.com> | 2013-02-06 15:23:11 -0500 |
---|---|---|
committer | Kent Overstreet <koverstreet@google.com> | 2013-03-23 17:26:28 -0400 |
commit | d74c6d514fe314b8bdab58b487b25992291577ec (patch) | |
tree | 2e04cd492235dc7982bae1ba7ef83f276800c319 /fs/exofs | |
parent | 6bc454d150047fcfbf53346412e64cdf3bf61a79 (diff) |
block: Add bio_for_each_segment_all()
__bio_for_each_segment() iterates bvecs from the specified index
instead of bio->bv_idx. Currently, the only usage is to walk all the
bvecs after the bio has been advanced by specifying 0 index.
For immutable bvecs, we need to split these apart;
bio_for_each_segment() is going to have a different implementation.
This will also help document the intent of code that's using it -
bio_for_each_segment_all() is only legal to use for code that owns the
bio.
Signed-off-by: Kent Overstreet <koverstreet@google.com>
CC: Jens Axboe <axboe@kernel.dk>
CC: Neil Brown <neilb@suse.de>
CC: Boaz Harrosh <bharrosh@panasas.com>
Diffstat (limited to 'fs/exofs')
-rw-r--r-- | fs/exofs/ore.c | 2 | ||||
-rw-r--r-- | fs/exofs/ore_raid.c | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/fs/exofs/ore.c b/fs/exofs/ore.c index f936cb50dc0d..b74422888604 100644 --- a/fs/exofs/ore.c +++ b/fs/exofs/ore.c | |||
@@ -401,7 +401,7 @@ static void _clear_bio(struct bio *bio) | |||
401 | struct bio_vec *bv; | 401 | struct bio_vec *bv; |
402 | unsigned i; | 402 | unsigned i; |
403 | 403 | ||
404 | __bio_for_each_segment(bv, bio, i, 0) { | 404 | bio_for_each_segment_all(bv, bio, i) { |
405 | unsigned this_count = bv->bv_len; | 405 | unsigned this_count = bv->bv_len; |
406 | 406 | ||
407 | if (likely(PAGE_SIZE == this_count)) | 407 | if (likely(PAGE_SIZE == this_count)) |
diff --git a/fs/exofs/ore_raid.c b/fs/exofs/ore_raid.c index b963f38ac298..7682b970d0f1 100644 --- a/fs/exofs/ore_raid.c +++ b/fs/exofs/ore_raid.c | |||
@@ -432,7 +432,7 @@ static void _mark_read4write_pages_uptodate(struct ore_io_state *ios, int ret) | |||
432 | if (!bio) | 432 | if (!bio) |
433 | continue; | 433 | continue; |
434 | 434 | ||
435 | __bio_for_each_segment(bv, bio, i, 0) { | 435 | bio_for_each_segment_all(bv, bio, i) { |
436 | struct page *page = bv->bv_page; | 436 | struct page *page = bv->bv_page; |
437 | 437 | ||
438 | SetPageUptodate(page); | 438 | SetPageUptodate(page); |