On Tue, Oct 02, 2012 at 06:05:53PM -0400, Vivek Goyal wrote: > On Tue, Oct 02, 2012 at 01:52:50PM -0700, Kent Overstreet wrote: > > On Tue, Oct 02, 2012 at 11:12:02AM -0400, Vivek Goyal wrote: > > > 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. > > > > I was just trying to make the bugfix patch small, since people > > complained about too much stuff being done in one patch. > > Can't we introduce the pointer while we retain bip_slabs as it is. Then > this patch will be small. I think this patch is big only because you > are trying to allocate integrity vecs from bio_set out of line. Ok - yeah, that makes sense to break out. I think I'll just make the integrity stuff a separate patch series, it's going to be like 4 patches now. -- dm-devel mailing list dm-devel@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/dm-devel
