On Mon, Sep 24, 2012 at 03:34:42PM -0700, Kent Overstreet wrote: > This adds a pointer to the bvec array to struct bio_integrity_payload, > instead of the bvecs always being inline; then the bvecs are allocated > with bvec_alloc_bs(). Ok, you are introducing bio_vec pointer in this patch. May be we can do it earlier so that we take care of bio pair related issue. > > Changed bvec_alloc_bs() and bvec_free_bs() to take a pointer to a > mempool instead of the bioset, so that bio integrity can use a different > mempool for its bvecs, and thus avoid a potential deadlock. If you are taking mempool as input, then bvec_alloc_bs() name does not make sense, as I think the very fact "bs" in the name their suggests that you are taking a pointer to bio set (and not mempool). Given the fact that bio_vec pool for integrity inside bio set, why not take additional boolean/enum argument to function bvec_alloc_bs() and that argument can tell whehter to allocate bvec from which bvec pool. > > This is eventually for immutable bio vecs - immutable bvecs aren't > useful if we still have to copy them, hence the need for the pointer. > Less code is always nice too, though. Can you explain a bit more about immutable bio vecs. I know there was some discussion and I missed it. So either point me to right mail thread or just explain a bit more what are immutable bio vecs, how we are planning to use these. Is it about during split we don't want to copy bio vecs and that's why we need a pointer so that newly created bio/bip can point to parent's bio_vec? Thanks Vivek -- dm-devel mailing list dm-devel@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/dm-devel
