Re: [RESEND PATCH V5 00/33] block: support multipage bvec

From: Ming Lei
Date: Fri Jun 01 2018 - 19:56:31 EST


On Fri, Jun 01, 2018 at 01:43:02AM -0700, Christoph Hellwig wrote:
> On Mon, May 28, 2018 at 10:30:43AM +0800, Ming Lei wrote:
> > I hate the conversion too, but looks it can't be avoided since
> > bio_for_each_segment_all() has to be changed.
>
> I guess you mean what is currently bio_for_each_page_all in your
> tree which now takes a bvec_iter_all agument? We could just
> try to hide the bia variable initially under an unlike to be used
> name, and then slowly move to the proper bio_for_each_page_all
> API unhiding it with the rename.

I tried that way at the beginning, it will cause gcc warning, since
the variable will be defined in the middle of one function, and even
worse it might break nested iterator case.

Thanks,
Ming