Re: [PATCH v3 20/49] block: introduce bio_for_each_segment_mp()

2017-10-19 Thread Ming Lei
On Thu, Aug 10, 2017 at 05:11:10AM -0700, Christoph Hellwig wrote: > First: as mentioned in the previous patches I really hate the name > scheme with the _sp and _mp postfixes. > > To be clear and understandable we should always name the versions > that iterate over segments *segment* and the

Re: [PATCH v3 20/49] block: introduce bio_for_each_segment_mp()

2017-10-19 Thread Ming Lei
On Thu, Aug 10, 2017 at 05:11:10AM -0700, Christoph Hellwig wrote: > First: as mentioned in the previous patches I really hate the name > scheme with the _sp and _mp postfixes. > > To be clear and understandable we should always name the versions > that iterate over segments *segment* and the

Re: [PATCH v3 20/49] block: introduce bio_for_each_segment_mp()

2017-08-10 Thread Christoph Hellwig
First: as mentioned in the previous patches I really hate the name scheme with the _sp and _mp postfixes. To be clear and understandable we should always name the versions that iterate over segments *segment* and the ones that iterate over pages *page*. To make sure we have a clean compile break

Re: [PATCH v3 20/49] block: introduce bio_for_each_segment_mp()

2017-08-10 Thread Christoph Hellwig
First: as mentioned in the previous patches I really hate the name scheme with the _sp and _mp postfixes. To be clear and understandable we should always name the versions that iterate over segments *segment* and the ones that iterate over pages *page*. To make sure we have a clean compile break

[PATCH v3 20/49] block: introduce bio_for_each_segment_mp()

2017-08-08 Thread Ming Lei
This helper is used to iterate multipage bvec and it is required in bio_clone(). Signed-off-by: Ming Lei --- include/linux/bio.h | 34 +++--- include/linux/bvec.h | 36 2 files changed, 63 insertions(+), 7

[PATCH v3 20/49] block: introduce bio_for_each_segment_mp()

2017-08-08 Thread Ming Lei
This helper is used to iterate multipage bvec and it is required in bio_clone(). Signed-off-by: Ming Lei --- include/linux/bio.h | 34 +++--- include/linux/bvec.h | 36 2 files changed, 63 insertions(+), 7 deletions(-) diff