Re: [PATCH 11/27] bcache: io.c: use bio_set_vec_table

2016-04-05 Thread Kent Overstreet
On Tue, Apr 05, 2016 at 05:49:02AM -0700, Christoph Hellwig wrote: > On Tue, Apr 05, 2016 at 07:56:56PM +0800, Ming Lei wrote: > > diff --git a/drivers/md/bcache/io.c b/drivers/md/bcache/io.c > > index 86a0bb8..1c48462 100644 > > --- a/drivers/md/bcache/io.c > > +++ b/drivers/md/bcache/io.c > > @@

Re: [PATCH 11/27] bcache: io.c: use bio_set_vec_table

2016-04-05 Thread Christoph Hellwig
On Tue, Apr 05, 2016 at 11:24:30PM +0800, Ming Lei wrote: > - bio can be embedded into one biger instance, which is often allocated > dynamically, so one extra allocation for bio can be avoided. We can also do this the other way around with the bios front_pad, which avoid the caller poking into bi

Re: [PATCH 11/27] bcache: io.c: use bio_set_vec_table

2016-04-05 Thread Ming Lei
On Tue, Apr 5, 2016 at 8:49 PM, Christoph Hellwig wrote: > On Tue, Apr 05, 2016 at 07:56:56PM +0800, Ming Lei wrote: >> diff --git a/drivers/md/bcache/io.c b/drivers/md/bcache/io.c >> index 86a0bb8..1c48462 100644 >> --- a/drivers/md/bcache/io.c >> +++ b/drivers/md/bcache/io.c >> @@ -26,8 +26,7 @@

Re: [PATCH 11/27] bcache: io.c: use bio_set_vec_table

2016-04-05 Thread Christoph Hellwig
On Tue, Apr 05, 2016 at 07:56:56PM +0800, Ming Lei wrote: > diff --git a/drivers/md/bcache/io.c b/drivers/md/bcache/io.c > index 86a0bb8..1c48462 100644 > --- a/drivers/md/bcache/io.c > +++ b/drivers/md/bcache/io.c > @@ -26,8 +26,7 @@ struct bio *bch_bbio_alloc(struct cache_set *c) > > bio_

[PATCH 11/27] bcache: io.c: use bio_set_vec_table

2016-04-05 Thread Ming Lei
Signed-off-by: Ming Lei --- drivers/md/bcache/io.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/md/bcache/io.c b/drivers/md/bcache/io.c index 86a0bb8..1c48462 100644 --- a/drivers/md/bcache/io.c +++ b/drivers/md/bcache/io.c @@ -26,8 +26,7 @@ struct bio *bch_bbio_a