Groups > linux.kernel > #1519748

[PATCH 00/12] block: cleanup direct access to bvec table

Path	csiph.com!news.mixmin.net!aioe.org!bofh.it!news.nic.it!robomod
From	Ming Lei <tom.leiming@gmail.com>
Newsgroups	linux.kernel
Subject	[PATCH 00/12] block: cleanup direct access to bvec table
Date	Fri, 11 Nov 2016 13:10:03 +0100
Message-ID	<sCiOC-48t-3@gated-at.bofh.it> (permalink)
Dkim-Signature	v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=from:to:cc:subject:date:message-id; bh=RTrVoa784uTTMHqz1OQXcxsS4rY1di5TN7TKdAu5hQQ=; b=JlNZBAGAat7ZuZ8MRkiy6+VOEWPgWY5iLthRFnF27X+g/5MOPrjIVmvD9Q3T9l4+7o fpwIU05N3lHFhcc5FWdUmJK00NgM/N2qm3UHMuK9nuvF05XNMRKwN7g1hROQjoUd1roD FyoTWNHQuvnEgvIv64rQKxWa8KkbHTdmCh6KccHw5n5UiS1hSycRZnK3+ENxUYfXiwW6 RcQj0iifDBMznXa3zop+mckzdH/bkPu70qqnVnk49SQAEvL+U/a2PkhKOgkTgmQydaHS BvdZErM4hav/n2ww+C3NZ2K0vosuTV54OIy1ZNTGIiCYBso3DJWFWPPhj4B4obw5G2iZ mxnw==
X-Google-Dkim-Signature	v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:cc:subject:date:message-id; bh=RTrVoa784uTTMHqz1OQXcxsS4rY1di5TN7TKdAu5hQQ=; b=AkT4taFaMmh/lt7U24Qyz4R1LDrI1g/oNvaGq9OCGYIhR5Za4sf5ZawFFBLev0N4CZ yaibAy0q/IGyu6N/F3044HhbflM2N+9EH3RnH5dvORN+iABNNg4OIIRJJdCl53Qc5s5C zUM0g8D9QM6wXM2+ssxDM/NztuF4u5sg9oo8854S1SGrFJS9iEos/SSjfrJQDyPUS0io YuDYbQiQN89ZGT+2BTH1ER5batOOTGTMxMqHdlpFtmobLlozkKpbeAoIn18dNOGkn+3F DeqtZmOyO1ZI3xq5HhjvcCmaxFZn4ZRet6rQJM0O7fykZnXSNhMD8IOb0vGOUymJnzdS LtQA==
X-Gm-Message-State	ABUngvdBwPSzOaS7m3e+Ju+vbXzCObqQlVRKP5pVEauqxIGi0lk8zPX1F/0b+hhmk+dGaA==
X-Received	by 10.99.173.78 with SMTP id y14mr2993527pgo.130.1478865972354; Fri, 11 Nov 2016 04:06:12 -0800 (PST)
X-Mailer	git-send-email 2.7.4
Sender	robomod@news.nic.it
List-ID	<linux-kernel.vger.kernel.org>
X-Mailing-List	linux-kernel@vger.kernel.org
Approved	robomod@news.nic.it
Lines	61
Organization	linux.* mail to news gateway
X-Original-Cc	linux-block@vger.kernel.org, linux-fsdevel@vger.kernel.org, Christoph Hellwig <hch@infradead.org>, Ming Lei <tom.leiming@gmail.com>, Adrian Hunter <adrian.hunter@intel.com>, Coly Li <colyli@suse.de>, dm-devel@redhat.com (open list:DEVICE-MAPPER (LVM)), drbd-dev@lists.linbit.com (open list:DRBD DRIVER), Eric Wheeler <git@linux.ewheeler.net>, Guoqing Jiang <gqjiang@suse.com>, Hannes Reinecke <hare@suse.com>, Jiri Kosina <jkosina@suse.cz>, Johannes Thumshirn <jthumshirn@suse.de>, Keith Busch <keith.busch@intel.com>, Kent Overstreet <kent.overstreet@gmail.com>, linux-bcache@vger.kernel.org (open list:BCACHE (BLOCK LAYER CACHE)), linux-nvme@lists.infradead.org (open list:NVM EXPRESS TARGET DRIVER), linux-raid@vger.kernel.org (open list:SOFTWARE RAID (Multiple Disks) SUPPORT), linux-scsi@vger.kernel.org (open list:TARGET SUBSYSTEM), logfs@logfs.org (open list:LogFS), Mike Christie <mchristi@redhat.com>, Mike Snitzer <snitzer@redhat.com>, Rasmus Villemoes <linux@rasmusvillemoes.dk>, target-devel@vger.kernel.org (open list:TARGET SUBSYSTEM), Yijing Wang <wangyijing@huawei.com>, Zheng Liu <wenqing.lz@taobao.com>
X-Original-Date	Fri, 11 Nov 2016 20:05:28 +0800
X-Original-Message-ID	<1478865957-25252-1-git-send-email-tom.leiming@gmail.com>
X-Original-Sender	linux-kernel-owner@vger.kernel.org
Xref	csiph.com linux.kernel:1519748

Show key headers only | View raw

Hi,

This patchset cleans up direct access to bvec table.

The 1st patch passes bvec table to bio_init(), so that
direct access to bvec table in bio initialization is avoided.

For other patches, most of them uses bio_add_page()
to replace hardcode style of adding page to bvec,
and others avoids to access bio->bi_vcnt.

The most special one is to use bvec iterator helpers
to implement .get_page/.next_page for dm-io.c

One big motivation is to prepare for supporting multipage
bvec, but this patchset is one good cleanup too even not
for that purpose.

Thanks,
Ming

Ming Lei (12):
  block: bio: pass bvec table to bio_init()
  block: drbd: remove impossible failure handling
  block: floppy: use bio_add_page()
  target: avoid to access .bi_vcnt directly
  bcache: debug: avoid to access .bi_io_vec directly
  dm: crypt: use bio_add_page()
  dm: use bvec iterator helpers to implement .get_page and .next_page
  dm: dm.c: replace 'bio->bi_vcnt == 1' with !bio_multiple_segments
  fs: logfs: convert to bio_add_page() in sync_request()
  fs: logfs: use bio_add_page() in __bdev_writeseg()
  fs: logfs: use bio_add_page() in do_erase()
  fs: logfs: remove unnecesary check

 block/bio.c                        |   8 ++-
 drivers/block/drbd/drbd_receiver.c |  14 +----
 drivers/block/floppy.c             |  10 ++--
 drivers/md/bcache/debug.c          |  11 ++--
 drivers/md/bcache/io.c             |   4 +-
 drivers/md/bcache/journal.c        |   4 +-
 drivers/md/bcache/movinggc.c       |   6 +--
 drivers/md/bcache/request.c        |   2 +-
 drivers/md/bcache/super.c          |  12 ++---
 drivers/md/bcache/writeback.c      |   5 +-
 drivers/md/dm-bufio.c              |   4 +-
 drivers/md/dm-crypt.c              |   8 +--
 drivers/md/dm-io.c                 |  34 ++++++++----
 drivers/md/dm-rq.c                 |   7 ++-
 drivers/md/dm.c                    |   2 +-
 drivers/md/multipath.c             |   2 +-
 drivers/md/raid5-cache.c           |   2 +-
 drivers/md/raid5.c                 |   9 +---
 drivers/nvme/target/io-cmd.c       |   4 +-
 drivers/target/target_core_pscsi.c |   8 +--
 fs/logfs/dev_bdev.c                | 106 +++++++++++++------------------------
 include/linux/bio.h                |   3 +-
 22 files changed, 107 insertions(+), 158 deletions(-)

-- 
2.7.4

Back to linux.kernel | Previous | Next — Next in thread | Find similar | Unroll thread

Thread

[PATCH 00/12] block: cleanup direct access to bvec table Ming Lei <tom.leiming@gmail.com> - 2016-11-11 13:10 +0100
  [PATCH 02/12] block: drbd: remove impossible failure handling Ming Lei <tom.leiming@gmail.com> - 2016-11-11 13:10 +0100
  [PATCH 01/12] block: bio: pass bvec table to bio_init() Ming Lei <tom.leiming@gmail.com> - 2016-11-11 13:10 +0100
    Re: [PATCH 01/12] block: bio: pass bvec table to bio_init() Christoph Hellwig <hch@infradead.org> - 2016-11-12 19:10 +0100

csiph-web