On Sun, Apr 07, 2019 at 03:37:00PM +0800, Ming Lei wrote: > > > @@ -48,7 +48,7 @@ struct bvec_iter { > > > struct bvec_iter_all { > > > struct bio_vec bv; > > > int idx; > > > - unsigned done; > > > + unsigned bv_done; > > > > Why the rename here? > > 'done' may be a bit misleading given we know this field is for recording how > many bytes we have done on the current bvec. Or .bvec_done? I doubt add a comment..