block: Remove bip_buf
authorMartin K. Petersen <martin.petersen@oracle.com>
Fri, 26 Sep 2014 23:19:58 +0000 (19:19 -0400)
committerJens Axboe <axboe@fb.com>
Sat, 27 Sep 2014 15:14:50 +0000 (09:14 -0600)
bip_buf is not really needed so we can remove it.

Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Sagi Grimberg <sagig@mellanox.com>
Signed-off-by: Jens Axboe <axboe@fb.com>
block/bio-integrity.c
include/linux/bio.h

index 367bb24bb9f12ebeccc5a450971e0ae7bc4edadf..e84f7fb8694b674f4435cc42fc7f73617c658fd5 100644 (file)
@@ -101,7 +101,8 @@ void bio_integrity_free(struct bio *bio)
        struct bio_set *bs = bio->bi_pool;
 
        if (bip->bip_owns_buf)
-               kfree(bip->bip_buf);
+               kfree(page_address(bip->bip_vec->bv_page) +
+                     bip->bip_vec->bv_offset);
 
        if (bs) {
                if (bip->bip_slab != BIO_POOL_NONE)
@@ -219,14 +220,16 @@ static int bio_integrity_generate_verify(struct bio *bio, int operate)
        struct blk_integrity *bi = bdev_get_integrity(bio->bi_bdev);
        struct blk_integrity_exchg bix;
        struct bio_vec *bv;
+       struct bio_integrity_payload *bip = bio_integrity(bio);
        sector_t sector;
        unsigned int sectors, ret = 0, i;
-       void *prot_buf = bio_integrity(bio)->bip_buf;
+       void *prot_buf = page_address(bip->bip_vec->bv_page) +
+               bip->bip_vec->bv_offset;
 
        if (operate)
                sector = bio->bi_iter.bi_sector;
        else
-               sector = bio_integrity(bio)->bip_iter.bi_sector;
+               sector = bip->bip_iter.bi_sector;
 
        bix.disk_name = bio->bi_bdev->bd_disk->disk_name;
        bix.sector_size = bi->sector_size;
@@ -321,7 +324,6 @@ int bio_integrity_prep(struct bio *bio)
        }
 
        bip->bip_owns_buf = 1;
-       bip->bip_buf = buf;
        bip->bip_iter.bi_size = len;
        bip->bip_iter.bi_sector = bio->bi_iter.bi_sector;
 
index 63a0e53e238c9574e9daa075e734a9916b3385ce..448d8c052cb77ae96279196c9a7b7d12dbd7021f 100644 (file)
@@ -310,9 +310,6 @@ struct bio_integrity_payload {
 
        struct bvec_iter        bip_iter;
 
-       /* kill - should just use bip_vec */
-       void                    *bip_buf;       /* generated integrity data */
-
        bio_end_io_t            *bip_end_io;    /* saved I/O completion fn */
 
        unsigned short          bip_slab;       /* slab the bip came from */