diff options
author | Kent Overstreet <kmo@daterainc.com> | 2013-09-24 19:26:05 -0400 |
---|---|---|
committer | Kent Overstreet <kmo@daterainc.com> | 2013-11-24 01:33:51 -0500 |
commit | 458b76ed2f9517becb74dcc8eedd70d3068ea6e4 (patch) | |
tree | b01b2150b197e0759b3ba59c0f9367d4477ebb8c /drivers/md/raid0.c | |
parent | d57a5f7c6605f15f3b5134837e68b448a7cea88e (diff) |
block: Kill bio_segments()/bi_vcnt usage
When we start sharing biovecs, keeping bi_vcnt accurate for splits is
going to be error prone - and unnecessary, if we refactor some code.
So bio_segments() has to go - but most of the existing users just needed
to know if the bio had multiple segments, which is easier - add a
bio_multiple_segments() for them.
(Two of the current uses of bio_segments() are going to go away in a
couple patches, but the current implementation of bio_segments() is
unsafe as soon as we start doing driver conversions for immutable
biovecs - so implement a dumb version for bisectability, it'll go away
in a couple patches)
Signed-off-by: Kent Overstreet <kmo@daterainc.com>
Cc: Jens Axboe <axboe@kernel.dk>
Cc: Neil Brown <neilb@suse.de>
Cc: Nagalakshmi Nandigama <Nagalakshmi.Nandigama@lsi.com>
Cc: Sreekanth Reddy <Sreekanth.Reddy@lsi.com>
Cc: "James E.J. Bottomley" <JBottomley@parallels.com>
Diffstat (limited to 'drivers/md/raid0.c')
-rw-r--r-- | drivers/md/raid0.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/md/raid0.c b/drivers/md/raid0.c index e38d1d3226f3..8ee1a6c658b4 100644 --- a/drivers/md/raid0.c +++ b/drivers/md/raid0.c | |||
@@ -528,7 +528,7 @@ static void raid0_make_request(struct mddev *mddev, struct bio *bio) | |||
528 | sector_t sector = bio->bi_iter.bi_sector; | 528 | sector_t sector = bio->bi_iter.bi_sector; |
529 | struct bio_pair *bp; | 529 | struct bio_pair *bp; |
530 | /* Sanity check -- queue functions should prevent this happening */ | 530 | /* Sanity check -- queue functions should prevent this happening */ |
531 | if (bio_segments(bio) > 1) | 531 | if (bio_multiple_segments(bio)) |
532 | goto bad_map; | 532 | goto bad_map; |
533 | /* This is a one page bio that upper layers | 533 | /* This is a one page bio that upper layers |
534 | * refuse to split for us, so we need to split it. | 534 | * refuse to split for us, so we need to split it. |