Re: [PATCH 09/27] block: Use bio_sectors() more consistently

From: Ed Cashin
Date: Wed Feb 20 2013 - 11:47:08 EST


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@xxxxxxxxxx>
> CC: Jens Axboe <axboe@xxxxxxxxx>
> CC: "Ed L. Cashin" <ecashin@xxxxxxxxxx>
> CC: Nick Piggin <npiggin@xxxxxxxxx>
> CC: Jiri Kosina <jkosina@xxxxxxx>
> CC: Jim Paris <jim@xxxxxxxx>
> CC: Geoff Levand <geoff@xxxxxxxxxxxxx>
> CC: Alasdair Kergon <agk@xxxxxxxxxx>
> CC: dm-devel@xxxxxxxxxx
> CC: Neil Brown <neilb@xxxxxxx>
> CC: Steven Rostedt <rostedt@xxxxxxxxxxx>
> Acked-by: Ed Cashin <ecashin@xxxxxxxxxx>
> ---
> 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@xxxxxxxxxx


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/