linux-kernel - Re: [PATCH 09/27] block: Use bio

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Wed, 20 Feb 2013 16:43:34 +0000
From:	Ed Cashin <ecashin@...aid.com>
To:	Kent Overstreet <koverstreet@...gle.com>
CC:	"<linux-kernel@...r.kernel.org>" <linux-kernel@...r.kernel.org>,
	"<axboe@...nel.dk>" <axboe@...nel.dk>,
	Nick Piggin <npiggin@...nel.dk>,
	"Jiri Kosina" <jkosina@...e.cz>, Jim Paris <jim@...n.com>,
	Geoff Levand <geoff@...radead.org>,
	Alasdair Kergon <agk@...hat.com>,
	"<dm-devel@...hat.com>" <dm-devel@...hat.com>,
	Neil Brown <neilb@...e.de>,
	Steven Rostedt <rostedt@...dmis.org>
Subject: Re: [PATCH 09/27] block: Use bio_sectors() more consistently

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@...gle.com>
> CC: Jens Axboe <axboe@...nel.dk>
> CC: "Ed L. Cashin" <ecashin@...aid.com>
> CC: Nick Piggin <npiggin@...nel.dk>
> CC: Jiri Kosina <jkosina@...e.cz>
> CC: Jim Paris <jim@...n.com>
> CC: Geoff Levand <geoff@...radead.org>
> CC: Alasdair Kergon <agk@...hat.com>
> CC: dm-devel@...hat.com
> CC: Neil Brown <neilb@...e.de>
> CC: Steven Rostedt <rostedt@...dmis.org>
> Acked-by: Ed Cashin <ecashin@...aid.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@...aid.com


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