On Thu, Sep 11, 2014 at 05:15:25PM +1000, NeilBrown wrote:

On Wed, 10 Sep 2014 20:40:09 +0800 shli@kernel.org wrote:

quoted

This is the 2nd attempt to make adjacent full stripe write together. The main
change against v1 is how to detect if stripes can be batched. Also some bugs
are fixed.

Thanks a lot for this - it looks quite good.  I've read through most of it
and don't see anything worth mentioning.... except maybe to wonder why
to_addr_page() has an unused 'sh' argument.  Maybe I'll just remove that(?).

Can be removed. I just want the parameters of to_addr_conv/page are the same.

I've pushed it you the 'devel' branch of my md git tree.  I'll try to do some
testing of my own next week.

Thanks!

Thanks,
Shaohua