diff options
author | Kent Overstreet <koverstreet@google.com> | 2012-05-25 16:03:11 -0400 |
---|---|---|
committer | Kent Overstreet <koverstreet@google.com> | 2013-03-23 17:26:33 -0400 |
commit | a38352e0ac02dbbd4fa464dc22d1352b5fbd06fd (patch) | |
tree | cae9097633ac86ab3b32ee8db9dc2e847929ad06 /fs/bio.c | |
parent | a07876064a0b73ab5ef1ebcf14b1cf0231c07858 (diff) |
block: Add an explicit bio flag for bios that own their bvec
This is for the new bio splitting code. When we split a bio, if the
split occured on a bvec boundry we reuse the bvec for the new bio. But
that means bio_free() can't free it, hence the explicit flag.
Signed-off-by: Kent Overstreet <koverstreet@google.com>
CC: Jens Axboe <axboe@kernel.dk>
Acked-by: Tejun Heo <tj@kernel.org>
Diffstat (limited to 'fs/bio.c')
-rw-r--r-- | fs/bio.c | 4 |
1 files changed, 3 insertions, 1 deletions
@@ -252,7 +252,7 @@ static void bio_free(struct bio *bio) | |||
252 | __bio_free(bio); | 252 | __bio_free(bio); |
253 | 253 | ||
254 | if (bs) { | 254 | if (bs) { |
255 | if (bio_has_allocated_vec(bio)) | 255 | if (bio_flagged(bio, BIO_OWNS_VEC)) |
256 | bvec_free(bs->bvec_pool, bio->bi_io_vec, BIO_POOL_IDX(bio)); | 256 | bvec_free(bs->bvec_pool, bio->bi_io_vec, BIO_POOL_IDX(bio)); |
257 | 257 | ||
258 | /* | 258 | /* |
@@ -451,6 +451,8 @@ struct bio *bio_alloc_bioset(gfp_t gfp_mask, int nr_iovecs, struct bio_set *bs) | |||
451 | 451 | ||
452 | if (unlikely(!bvl)) | 452 | if (unlikely(!bvl)) |
453 | goto err_free; | 453 | goto err_free; |
454 | |||
455 | bio->bi_flags |= 1 << BIO_OWNS_VEC; | ||
454 | } else if (nr_iovecs) { | 456 | } else if (nr_iovecs) { |
455 | bvl = bio->bi_inline_vecs; | 457 | bvl = bio->bi_inline_vecs; |
456 | } | 458 | } |