Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758781AbcDENET (ORCPT ); Tue, 5 Apr 2016 09:04:19 -0400 Received: from bombadil.infradead.org ([198.137.202.9]:57887 "EHLO bombadil.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1758092AbcDENED (ORCPT ); Tue, 5 Apr 2016 09:04:03 -0400 Date: Tue, 5 Apr 2016 06:04:02 -0700 From: Christoph Hellwig To: Ming Lei Cc: Jens Axboe , linux-kernel@vger.kernel.org, linux-block@vger.kernel.org, Christoph Hellwig , Boaz Harrosh , Alasdair Kergon , Mike Snitzer , "maintainer:DEVICE-MAPPER (LVM)" , Shaohua Li , "open list:SOFTWARE RAID (Multiple Disks) SUPPORT" Subject: Re: [PATCH 20/27] dm: dm-bufio.c: use bio_set_vec_table() Message-ID: <20160405130402.GD32576@infradead.org> References: <1459858062-21075-1-git-send-email-tom.leiming@gmail.com> <1459858062-21075-6-git-send-email-tom.leiming@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1459858062-21075-6-git-send-email-tom.leiming@gmail.com> User-Agent: Mutt/1.5.24 (2015-08-30) X-SRS-Rewrite: SMTP reverse-path rewritten from by bombadil.infradead.org. See http://www.infradead.org/rpr.html Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 821 Lines: 22 On Tue, Apr 05, 2016 at 08:07:35PM +0800, Ming Lei wrote: > Signed-off-by: Ming Lei > --- > drivers/md/dm-bufio.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/drivers/md/dm-bufio.c b/drivers/md/dm-bufio.c > index cd77216..0e48ad7 100644 > --- a/drivers/md/dm-bufio.c > +++ b/drivers/md/dm-bufio.c > @@ -624,8 +624,7 @@ static void use_inline_bio(struct dm_buffer *b, int rw, sector_t block, > int len; > > bio_init(&b->bio); > - b->bio.bi_io_vec = b->bio_vec; > - b->bio.bi_max_vecs = DM_BUFIO_INLINE_VECS; > + bio_set_vec_table(&b->bio, b->bio_vec, DM_BUFIO_INLINE_VECS); > b->bio.bi_iter.bi_sector = block << b->c->sectors_per_block_bits; > b->bio.bi_bdev = b->c->bdev; > b->bio.bi_end_io = inline_endio; Should be switched to use bio_alloc instead.