Re: [PATCH v1 00/54] block: support multipage bvec
From: Ming Lei
Date: Mon Jan 16 2017 - 21:40:47 EST
On Mon, Jan 16, 2017 at 11:18 PM, Christoph Hellwig <hch@xxxxxxxxxxxxx> wrote:
> On Mon, Jan 16, 2017 at 11:19:19AM +0800, Ming Lei wrote:
>> Any comments on this version?
>
> We'll need to make sure all drivers can handle multi-page bvecs
> before continuing any other work. Without that the series is a no-go.
> Note that in general making a drivers capable of handling multipage
> bvecs will clean it up by using new helpers and be worthwhile on it's
> own.
IMO, the only one left is raid(1/5/10) which can be dealt with by the
"NO_MP" flag. This point can be observed by result from runnig 'git grep':
$git grep -n -E "bi_vcnt|bi_io_vec" ./
Also this patchset addes comment in cases of direct access to bvec table,
and the cases have been minimized too, and most of them are single
bvec based.
Thanks,
Ming Lei