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 -- dm-devel mailing list dm-devel@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/dm-devel