Re: [PATCH V6 11/30] block: implement bio_pages_all() via bio_for_each_segment_all()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Christoph Hellwig <hch@xxxxxxxxxxxxx>
Subject: Re: [PATCH V6 11/30] block: implement bio_pages_all() via bio_for_each_segment_all()
From: Ming Lei <ming.lei@xxxxxxxxxx>
Date: Thu, 14 Jun 2018 09:23:54 +0800
Cc: Jens Axboe <axboe@xxxxxx>, Alexander Viro <viro@xxxxxxxxxxxxxxxxxx>, Kent Overstreet <kent.overstreet@xxxxxxxxx>, David Sterba <dsterba@xxxxxxx>, Huang Ying <ying.huang@xxxxxxxxx>, linux-kernel@xxxxxxxxxxxxxxx, linux-block@xxxxxxxxxxxxxxx, linux-fsdevel@xxxxxxxxxxxxxxx, linux-mm@xxxxxxxxx, "Theodore Ts'o" <tytso@xxxxxxx>, "Darrick J . Wong" <darrick.wong@xxxxxxxxxx>, Coly Li <colyli@xxxxxxx>, Filipe Manana <fdmanana@xxxxxxxxx>, Randy Dunlap <rdunlap@xxxxxxxxxxxxx>
In-reply-to: <20180613144412.GB4693@infradead.org>
References: <20180609123014.8861-1-ming.lei@redhat.com> <20180609123014.8861-12-ming.lei@redhat.com> <20180613144412.GB4693@infradead.org>
User-agent: Mutt/1.9.1 (2017-09-22)

On Wed, Jun 13, 2018 at 07:44:12AM -0700, Christoph Hellwig wrote:
> Given that we have a single, dubious user of bio_pages_all I'd rather
> see it as an opencoded bio_for_each_ loop in the caller.

Yeah, that is fine since there is only one user in btrfs.

Thanks,
Ming

Follow-Ups:
- Re: [PATCH V6 11/30] block: implement bio_pages_all() via bio_for_each_segment_all()
  - From: Christoph Hellwig

References:
- [PATCH V6 00/30] block: support multipage bvec
  - From: Ming Lei
- [PATCH V6 11/30] block: implement bio_pages_all() via bio_for_each_segment_all()
  - From: Ming Lei
- Re: [PATCH V6 11/30] block: implement bio_pages_all() via bio_for_each_segment_all()
  - From: Christoph Hellwig

Prev by Date: Re: [PATCH V6 00/30] block: support multipage bvec
Next by Date: Re: [PATCH 02/10] x86/cet: Introduce WRUSS instruction
Previous by thread: Re: [PATCH V6 11/30] block: implement bio_pages_all() via bio_for_each_segment_all()
Next by thread: Re: [PATCH V6 11/30] block: implement bio_pages_all() via bio_for_each_segment_all()
Index(es):
- Date
- Thread

[Index of Archives] [Linux ARM Kernel] [Linux ARM] [Linux Omap] [Fedora ARM] [IETF Annouce] [Bugtraq] [Linux OMAP] [Linux MIPS] [eCos] [Asterisk Internet PBX] [Linux API]