block: bio: pass bvec table to bio_init()
authorMing Lei <tom.leiming@gmail.com>
Tue, 22 Nov 2016 15:57:21 +0000 (08:57 -0700)
committerJens Axboe <axboe@fb.com>
Tue, 22 Nov 2016 15:57:21 +0000 (08:57 -0700)
Some drivers often use external bvec table, so introduce
this helper for this case. It is always safe to access the
bio->bi_io_vec in this way for this case.

After converting to this usage, it will becomes a bit easier
to evaluate the remaining direct access to bio->bi_io_vec,
so it can help to prepare for the following multipage bvec
support.

Signed-off-by: Ming Lei <tom.leiming@gmail.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Fixed up the new O_DIRECT cases.

Signed-off-by: Jens Axboe <axboe@fb.com>
17 files changed:
block/bio.c
drivers/block/floppy.c
drivers/md/bcache/io.c
drivers/md/bcache/journal.c
drivers/md/bcache/movinggc.c
drivers/md/bcache/request.c
drivers/md/bcache/super.c
drivers/md/bcache/writeback.c
drivers/md/dm-bufio.c
drivers/md/dm.c
drivers/md/multipath.c
drivers/md/raid5-cache.c
drivers/md/raid5.c
drivers/nvme/target/io-cmd.c
fs/block_dev.c
fs/logfs/dev_bdev.c
include/linux/bio.h

index 2cf6ebabc68c5cb10541592eee10cb8b5db71a1b..de257ced69b1df41f98169aaccd59e113070245a 100644 (file)
@@ -270,11 +270,15 @@ static void bio_free(struct bio *bio)
        }
 }
 
-void bio_init(struct bio *bio)
+void bio_init(struct bio *bio, struct bio_vec *table,
+             unsigned short max_vecs)
 {
        memset(bio, 0, sizeof(*bio));
        atomic_set(&bio->__bi_remaining, 1);
        atomic_set(&bio->__bi_cnt, 1);
+
+       bio->bi_io_vec = table;
+       bio->bi_max_vecs = max_vecs;
 }
 EXPORT_SYMBOL(bio_init);
 
@@ -480,7 +484,7 @@ struct bio *bio_alloc_bioset(gfp_t gfp_mask, int nr_iovecs, struct bio_set *bs)
                return NULL;
 
        bio = p + front_pad;
-       bio_init(bio);
+       bio_init(bio, NULL, 0);
 
        if (nr_iovecs > inline_vecs) {
                unsigned long idx = 0;
index e3d8e4ced4a23c57d1f36da593e80a64d8b1d142..6a3ff2b2e3ae7b1ff17dd11b99756521818b3da7 100644 (file)
@@ -3806,8 +3806,7 @@ static int __floppy_read_block_0(struct block_device *bdev, int drive)
 
        cbdata.drive = drive;
 
-       bio_init(&bio);
-       bio.bi_io_vec = &bio_vec;
+       bio_init(&bio, &bio_vec, 1);
        bio_vec.bv_page = page;
        bio_vec.bv_len = size;
        bio_vec.bv_offset = 0;
index e97b0acf7b8dca4a04e2018d23d2c79c051e248b..db45a88c0ce9d76a11fd6a755c0f436aac0fb024 100644 (file)
@@ -24,9 +24,7 @@ struct bio *bch_bbio_alloc(struct cache_set *c)
        struct bbio *b = mempool_alloc(c->bio_meta, GFP_NOIO);
        struct bio *bio = &b->bio;
 
-       bio_init(bio);
-       bio->bi_max_vecs         = bucket_pages(c);
-       bio->bi_io_vec           = bio->bi_inline_vecs;
+       bio_init(bio, bio->bi_inline_vecs, bucket_pages(c));
 
        return bio;
 }
index 6925023e12d45656ce6802ea5237c0069b9beb29..1198e53d5670317263e86d5dfdd7b4695fd264d2 100644 (file)
@@ -448,13 +448,11 @@ static void do_journal_discard(struct cache *ca)
 
                atomic_set(&ja->discard_in_flight, DISCARD_IN_FLIGHT);
 
-               bio_init(bio);
+               bio_init(bio, bio->bi_inline_vecs, 1);
                bio_set_op_attrs(bio, REQ_OP_DISCARD, 0);
                bio->bi_iter.bi_sector  = bucket_to_sector(ca->set,
                                                ca->sb.d[ja->discard_idx]);
                bio->bi_bdev            = ca->bdev;
-               bio->bi_max_vecs        = 1;
-               bio->bi_io_vec          = bio->bi_inline_vecs;
                bio->bi_iter.bi_size    = bucket_bytes(ca);
                bio->bi_end_io          = journal_discard_endio;
 
index 5c4bddecfaf092c6ec1e3a6a16da605c44dd9574..13b8a907006dd2be35c89c1188c0fa4932459fde 100644 (file)
@@ -77,15 +77,13 @@ static void moving_init(struct moving_io *io)
 {
        struct bio *bio = &io->bio.bio;
 
-       bio_init(bio);
+       bio_init(bio, bio->bi_inline_vecs,
+                DIV_ROUND_UP(KEY_SIZE(&io->w->key), PAGE_SECTORS));
        bio_get(bio);
        bio_set_prio(bio, IOPRIO_PRIO_VALUE(IOPRIO_CLASS_IDLE, 0));
 
        bio->bi_iter.bi_size    = KEY_SIZE(&io->w->key) << 9;
-       bio->bi_max_vecs        = DIV_ROUND_UP(KEY_SIZE(&io->w->key),
-                                              PAGE_SECTORS);
        bio->bi_private         = &io->cl;
-       bio->bi_io_vec          = bio->bi_inline_vecs;
        bch_bio_map(bio, NULL);
 }
 
index 0d99b5f4b3e61334c745e967fd6441282c7136bf..f49c5417527dcbb8e0a839a32ec5d354323d63a4 100644 (file)
@@ -623,7 +623,7 @@ static void do_bio_hook(struct search *s, struct bio *orig_bio)
 {
        struct bio *bio = &s->bio.bio;
 
-       bio_init(bio);
+       bio_init(bio, NULL, 0);
        __bio_clone_fast(bio, orig_bio);
        bio->bi_end_io          = request_endio;
        bio->bi_private         = &s->cl;
index 988edf92846603875534180091ae7f9d4473366b..2fb5bfeb43e2e1668051667d6582a087b6bb14ad 100644 (file)
@@ -1152,9 +1152,7 @@ static void register_bdev(struct cache_sb *sb, struct page *sb_page,
        dc->bdev = bdev;
        dc->bdev->bd_holder = dc;
 
-       bio_init(&dc->sb_bio);
-       dc->sb_bio.bi_max_vecs  = 1;
-       dc->sb_bio.bi_io_vec    = dc->sb_bio.bi_inline_vecs;
+       bio_init(&dc->sb_bio, dc->sb_bio.bi_inline_vecs, 1);
        dc->sb_bio.bi_io_vec[0].bv_page = sb_page;
        get_page(sb_page);
 
@@ -1814,9 +1812,7 @@ static int cache_alloc(struct cache *ca)
        __module_get(THIS_MODULE);
        kobject_init(&ca->kobj, &bch_cache_ktype);
 
-       bio_init(&ca->journal.bio);
-       ca->journal.bio.bi_max_vecs = 8;
-       ca->journal.bio.bi_io_vec = ca->journal.bio.bi_inline_vecs;
+       bio_init(&ca->journal.bio, ca->journal.bio.bi_inline_vecs, 8);
 
        free = roundup_pow_of_two(ca->sb.nbuckets) >> 10;
 
@@ -1852,9 +1848,7 @@ static int register_cache(struct cache_sb *sb, struct page *sb_page,
        ca->bdev = bdev;
        ca->bdev->bd_holder = ca;
 
-       bio_init(&ca->sb_bio);
-       ca->sb_bio.bi_max_vecs  = 1;
-       ca->sb_bio.bi_io_vec    = ca->sb_bio.bi_inline_vecs;
+       bio_init(&ca->sb_bio, ca->sb_bio.bi_inline_vecs, 1);
        ca->sb_bio.bi_io_vec[0].bv_page = sb_page;
        get_page(sb_page);
 
index e51644e503a53b5643aee19a1c605db0f94c11f4..69e1ae59cab8b9755d33c77e4435cabce634dce6 100644 (file)
@@ -106,14 +106,13 @@ static void dirty_init(struct keybuf_key *w)
        struct dirty_io *io = w->private;
        struct bio *bio = &io->bio;
 
-       bio_init(bio);
+       bio_init(bio, bio->bi_inline_vecs,
+                DIV_ROUND_UP(KEY_SIZE(&w->key), PAGE_SECTORS));
        if (!io->dc->writeback_percent)
                bio_set_prio(bio, IOPRIO_PRIO_VALUE(IOPRIO_CLASS_IDLE, 0));
 
        bio->bi_iter.bi_size    = KEY_SIZE(&w->key) << 9;
-       bio->bi_max_vecs        = DIV_ROUND_UP(KEY_SIZE(&w->key), PAGE_SECTORS);
        bio->bi_private         = w;
-       bio->bi_io_vec          = bio->bi_inline_vecs;
        bch_bio_map(bio, NULL);
 }
 
index b3ba142e59a4cd2c0a8bbbbb70f3095034218092..262e75365cc04b9ce4dec973d8b3edbfa687f98d 100644 (file)
@@ -611,9 +611,7 @@ static void use_inline_bio(struct dm_buffer *b, int rw, sector_t block,
        char *ptr;
        int len;
 
-       bio_init(&b->bio);
-       b->bio.bi_io_vec = b->bio_vec;
-       b->bio.bi_max_vecs = DM_BUFIO_INLINE_VECS;
+       bio_init(&b->bio, b->bio_vec, DM_BUFIO_INLINE_VECS);
        b->bio.bi_iter.bi_sector = block << b->c->sectors_per_block_bits;
        b->bio.bi_bdev = b->c->bdev;
        b->bio.bi_end_io = inline_endio;
index b2abfa41af3ef3da9d5e6f84f6a11a9f6ef1282e..7915467d37268683580dc5f0fc2038736a24b6a9 100644 (file)
@@ -1525,7 +1525,7 @@ static struct mapped_device *alloc_dev(int minor)
        if (!md->bdev)
                goto bad;
 
-       bio_init(&md->flush_bio);
+       bio_init(&md->flush_bio, NULL, 0);
        md->flush_bio.bi_bdev = md->bdev;
        md->flush_bio.bi_opf = REQ_OP_WRITE | REQ_PREFLUSH;
 
index 673efbd6fc4767af25f7be84717fa291e154fcd7..4da06d813b8fca7025dc47df2c3b3648e0b1bb85 100644 (file)
@@ -130,7 +130,7 @@ static void multipath_make_request(struct mddev *mddev, struct bio * bio)
        }
        multipath = conf->multipaths + mp_bh->path;
 
-       bio_init(&mp_bh->bio);
+       bio_init(&mp_bh->bio, NULL, 0);
        __bio_clone_fast(&mp_bh->bio, bio);
 
        mp_bh->bio.bi_iter.bi_sector += multipath->rdev->data_offset;
index 28d015c6fffe1e205385bb7e4adb7c052e2a6a4d..25e9622d7d80be4fabef52b8d7e210cc757d22a2 100644 (file)
@@ -1201,7 +1201,7 @@ int r5l_init_log(struct r5conf *conf, struct md_rdev *rdev)
        INIT_LIST_HEAD(&log->io_end_ios);
        INIT_LIST_HEAD(&log->flushing_ios);
        INIT_LIST_HEAD(&log->finished_ios);
-       bio_init(&log->flush_bio);
+       bio_init(&log->flush_bio, NULL, 0);
 
        log->io_kc = KMEM_CACHE(r5l_io_unit, 0);
        if (!log->io_kc)
index 70acdd379e4490d3107242f8502cf4d704400963..5f9e28443c8acbe5335f01638444c13f60d3c810 100644 (file)
@@ -2004,13 +2004,8 @@ static struct stripe_head *alloc_stripe(struct kmem_cache *sc, gfp_t gfp,
                for (i = 0; i < disks; i++) {
                        struct r5dev *dev = &sh->dev[i];
 
-                       bio_init(&dev->req);
-                       dev->req.bi_io_vec = &dev->vec;
-                       dev->req.bi_max_vecs = 1;
-
-                       bio_init(&dev->rreq);
-                       dev->rreq.bi_io_vec = &dev->rvec;
-                       dev->rreq.bi_max_vecs = 1;
+                       bio_init(&dev->req, &dev->vec, 1);
+                       bio_init(&dev->rreq, &dev->rvec, 1);
                }
        }
        return sh;
index ef52b1e701447ee9544ea664ac8163573de7cfb3..c4dc9ea8ade0b14727850ba45491fb724bf0d1e5 100644 (file)
@@ -37,9 +37,7 @@ static void nvmet_inline_bio_init(struct nvmet_req *req)
 {
        struct bio *bio = &req->inline_bio;
 
-       bio_init(bio);
-       bio->bi_max_vecs = NVMET_MAX_INLINE_BIOVEC;
-       bio->bi_io_vec = req->inline_bvec;
+       bio_init(bio, req->inline_bvec, NVMET_MAX_INLINE_BIOVEC);
 }
 
 static void nvmet_execute_rw(struct nvmet_req *req)
index b0c790a19db9a38db06eb00398b4bca5242c13f8..7022ddc55b124498896301e99486564f1c3b88af 100644 (file)
@@ -222,9 +222,7 @@ __blkdev_direct_IO_simple(struct kiocb *iocb, struct iov_iter *iter,
                        return -ENOMEM;
        }
 
-       bio_init(&bio);
-       bio.bi_max_vecs = nr_pages;
-       bio.bi_io_vec = vecs;
+       bio_init(&bio, vecs, nr_pages);
        bio.bi_bdev = bdev;
        bio.bi_iter.bi_sector = pos >> 9;
        bio.bi_private = current;
index a8329cc47decd627ca26cf608e6f15276697a4f6..dc8cafeee038bc4227bfe2b685ad99981e6126ec 100644 (file)
@@ -19,9 +19,7 @@ static int sync_request(struct page *page, struct block_device *bdev, int op)
        struct bio bio;
        struct bio_vec bio_vec;
 
-       bio_init(&bio);
-       bio.bi_max_vecs = 1;
-       bio.bi_io_vec = &bio_vec;
+       bio_init(&bio, &bio_vec, 1);
        bio_vec.bv_page = page;
        bio_vec.bv_len = PAGE_SIZE;
        bio_vec.bv_offset = 0;
index d367cd37a7f7569066e5176d68c16dcd4e54b5b1..70a7244f08a72e7c01ff00945003c69a55a8a810 100644 (file)
@@ -420,7 +420,8 @@ extern int bio_phys_segments(struct request_queue *, struct bio *);
 extern int submit_bio_wait(struct bio *bio);
 extern void bio_advance(struct bio *, unsigned);
 
-extern void bio_init(struct bio *);
+extern void bio_init(struct bio *bio, struct bio_vec *table,
+                    unsigned short max_vecs);
 extern void bio_reset(struct bio *);
 void bio_chain(struct bio *, struct bio *);