diff options
author | Sagi Grimberg <sagig@mellanox.com> | 2015-09-08 09:33:35 -0600 |
---|---|---|
committer | Jens Axboe <axboe@fb.com> | 2015-09-11 09:03:45 -0600 |
commit | 87a816df537e096d404add543ef47b796906c130 (patch) | |
tree | ae4d2c8c2606303f9ff4407a748fb48a111b7520 | |
parent | 7f39add3b08cbbdb99abe50e6d7c342e6800d684 (diff) | |
download | linux-87a816df537e096d404add543ef47b796906c130.tar.gz linux-87a816df537e096d404add543ef47b796906c130.tar.bz2 linux-87a816df537e096d404add543ef47b796906c130.zip |
block: Refuse adding appending a gapped integrity page to a bio
This is only theoretical at the moment given that the only
subsystems that generate integrity payloads are the block layer
itself and the scsi target (which generate well aligned integrity
payloads). But when we will expose integrity meta-data to user-space,
we'll need to refuse appending a page with a gap (if the queue
virtual boundary is set).
Signed-off-by: Sagi Grimberg <sagig@mellanox.com>
Signed-off-by: Jens Axboe <axboe@fb.com>
-rw-r--r-- | block/bio-integrity.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/block/bio-integrity.c b/block/bio-integrity.c index 4aecca79374a..14b8faf8b09d 100644 --- a/block/bio-integrity.c +++ b/block/bio-integrity.c @@ -140,6 +140,11 @@ int bio_integrity_add_page(struct bio *bio, struct page *page, iv = bip->bip_vec + bip->bip_vcnt; + if (bip->bip_vcnt && + bvec_gap_to_prev(bdev_get_queue(bio->bi_bdev), + &bip->bip_vec[bip->bip_vcnt - 1], offset)) + return 0; + iv->bv_page = page; iv->bv_len = len; iv->bv_offset = offset; |