On Tue, Feb 14, 2017 at 08:01:09AM -0800, Christoph Hellwig wrote: > On Tue, Feb 14, 2017 at 11:29:03PM +0800, Ming Lei wrote: > > Firstly bio_clone_mddev() is used in raid normal I/O and isn't > > in resync I/O path. > > > > Secondly all the direct access to bvec table in raid happens on > > resync I/O except for write behind of raid1, in which we still > > use bio_clone() for allocating new bvec table. > > > > So this patch replaces bio_clone() with bio_clone_fast() > > in bio_clone_mddev(). > > > > Also kill bio_clone_mddev() and call bio_clone_fast() directly, as > > suggested by Christoph Hellwig. > > > > Signed-off-by: Ming Lei <tom.leiming@xxxxxxxxx> > > Looks fine, > > Reviewed-by: Christoph Hellwig <hch@xxxxxx> > > Btw, can you also tack on another patch to kill bio_alloc_mddev > to be consistent? I'll take care of this
