diff options
author | NeilBrown <neilb@suse.com> | 2017-06-18 00:38:57 -0400 |
---|---|---|
committer | Jens Axboe <axboe@kernel.dk> | 2017-06-18 14:40:59 -0400 |
commit | af67c31fba3b879b241536a48df703a2eee18ebf (patch) | |
tree | 60b9f32d39c955a79e552b00900e1a6ceb66532b /drivers/md/md.c | |
parent | e4cdf1a1cb161a648cc1ed7d6148fc3b99a1b3f5 (diff) |
blk: remove bio_set arg from blk_queue_split()
blk_queue_split() is always called with the last arg being q->bio_split,
where 'q' is the first arg.
Also blk_queue_split() sometimes uses the passed-in 'bs' and sometimes uses
q->bio_split.
This is inconsistent and unnecessary. Remove the last arg and always use
q->bio_split inside blk_queue_split()
Reviewed-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Ming Lei <ming.lei@redhat.com>
Credit-to: Javier González <jg@lightnvm.io> (Noticed that lightnvm was missed)
Reviewed-by: Javier González <javier@cnexlabs.com>
Tested-by: Javier González <javier@cnexlabs.com>
Signed-off-by: NeilBrown <neilb@suse.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Diffstat (limited to 'drivers/md/md.c')
-rw-r--r-- | drivers/md/md.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/md/md.c b/drivers/md/md.c index 6d493b54d56c..d43df1176c23 100644 --- a/drivers/md/md.c +++ b/drivers/md/md.c | |||
@@ -265,7 +265,7 @@ static blk_qc_t md_make_request(struct request_queue *q, struct bio *bio) | |||
265 | unsigned int sectors; | 265 | unsigned int sectors; |
266 | int cpu; | 266 | int cpu; |
267 | 267 | ||
268 | blk_queue_split(q, &bio, q->bio_split); | 268 | blk_queue_split(q, &bio); |
269 | 269 | ||
270 | if (mddev == NULL || mddev->pers == NULL) { | 270 | if (mddev == NULL || mddev->pers == NULL) { |
271 | bio_io_error(bio); | 271 | bio_io_error(bio); |