[dm-devel] [PATCH 09/27] block: Use bio_sectors() more consistently

Ed Cashin ecashin at coraid.com
Wed Feb 20 16:43:34 UTC 2013


That looks OK to me.

(I notice that while back in October it included changes to aoe, the current patch does not, which is fine.)

On Feb 19, 2013, at 7:22 PM, Kent Overstreet wrote:

> Bunch of places in the code weren't using it where they could be -
> this'll reduce the size of the patch that puts bi_sector/bi_size/bi_idx
> into a struct bvec_iter.
> 
> Signed-off-by: Kent Overstreet <koverstreet at google.com>
> CC: Jens Axboe <axboe at kernel.dk>
> CC: "Ed L. Cashin" <ecashin at coraid.com>
> CC: Nick Piggin <npiggin at kernel.dk>
> CC: Jiri Kosina <jkosina at suse.cz>
> CC: Jim Paris <jim at jtan.com>
> CC: Geoff Levand <geoff at infradead.org>
> CC: Alasdair Kergon <agk at redhat.com>
> CC: dm-devel at redhat.com
> CC: Neil Brown <neilb at suse.de>
> CC: Steven Rostedt <rostedt at goodmis.org>
> Acked-by: Ed Cashin <ecashin at coraid.com>
> ---
> drivers/block/pktcdvd.c      |  2 +-
> drivers/md/dm-raid1.c        |  2 +-
> drivers/md/raid0.c           |  6 +++---
> drivers/md/raid1.c           | 17 ++++++++---------
> drivers/md/raid10.c          | 24 +++++++++++-------------
> drivers/md/raid5.c           |  8 ++++----
> fs/btrfs/volumes.c           |  2 +-
> include/trace/events/block.h | 12 ++++++------
> 8 files changed, 35 insertions(+), 38 deletions(-)
...

-- 
  Ed Cashin
  ecashin at coraid.com






More information about the dm-devel mailing list