Re: [dm-devel] [PATCH v10 4/8] block: Add bio_reset()

2012-09-07 Thread Kent Overstreet
On Sat, Sep 08, 2012 at 12:14:33AM +0100, Alasdair G Kergon wrote: > As I indicated already in this discussion, dm started to use > merge_bvec_fn as a cheap way of avoiding splitting and this improved > overall efficiency. Often it's better to pay the small price of calling > that function to

Re: [dm-devel] [PATCH v10 4/8] block: Add bio_reset()

2012-09-07 Thread Alasdair G Kergon
As I indicated already in this discussion, dm started to use merge_bvec_fn as a cheap way of avoiding splitting and this improved overall efficiency. Often it's better to pay the small price of calling that function to ensure the bio is created the right size in the first place so it won't have

Re: [dm-devel] [PATCH v10 4/8] block: Add bio_reset()

2012-09-07 Thread Alasdair G Kergon
As I indicated already in this discussion, dm started to use merge_bvec_fn as a cheap way of avoiding splitting and this improved overall efficiency. Often it's better to pay the small price of calling that function to ensure the bio is created the right size in the first place so it won't have

Re: [dm-devel] [PATCH v10 4/8] block: Add bio_reset()

2012-09-07 Thread Kent Overstreet
On Sat, Sep 08, 2012 at 12:14:33AM +0100, Alasdair G Kergon wrote: As I indicated already in this discussion, dm started to use merge_bvec_fn as a cheap way of avoiding splitting and this improved overall efficiency. Often it's better to pay the small price of calling that function to ensure