On Wed, 22 Aug 2012, Tejun Heo wrote: > > This is prep work for killing bi_destructor - previously, pktcdvd had > > its own pkt_bio_alloc which was basically duplication bio_kmalloc(), > > necessitating its own bi_destructor implementation. > > > > v5: Un-reorder some functions, to make the patch easier to review > > > > Signed-off-by: Kent Overstreet <koverstreet@xxxxxxxxxx> > > CC: Peter Osterlund <petero2@xxxxxxxxx> > > Apart from bio_reset() not resetting bi_end_io and bi_private, this > looks fine to me but lack of testing makes me a bit hesitant to ack > it. I have just tested this v6 patchset (and it works for pktcddvd), only to see that Kent has sent out v7 (unfortunately he hasn't CCed me on it). v6 works nicely, bus as far as I understand it, even in v7 we are not resetting bi_end_io and bi_private properly, right? -- Jiri Kosina SUSE Labs -- dm-devel mailing list dm-devel@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/dm-devel