From: Jens Axboe <axboe@kernel.dk>
To: Kent Overstreet <kmo@daterainc.com>
Cc: Christoph Hellwig <hch@infradead.org>,
linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org,
snitzer@redhat.com
Subject: Re: [GIT PULL] Immutable biovecs
Date: Tue, 26 Nov 2013 17:59:54 -0700 [thread overview]
Message-ID: <20131127005954.GH9759@kernel.dk> (raw)
In-Reply-To: <20131127004422.GB21305@kmo>
On Tue, Nov 26 2013, Kent Overstreet wrote:
> On Mon, Nov 25, 2013 at 10:05:58PM -0800, Christoph Hellwig wrote:
> > On Mon, Nov 25, 2013 at 01:52:16PM -0800, Kent Overstreet wrote:
> > > Jens - here's immutable biovecs, rebased and ready for 3.14. Changes since the
> > > last version of the series:
> >
> > Can you do a resend of the patch series to all involved lists first so
> > we can have a detailed look at the current version?
>
> Sure thing.
>
> > > * bio_clone_bioset() retains the old behaviour, as previously discussed -
> > > bio_clone_fast() is being used by bcache, dm and the new bio_split().
> >
> > Any chance to have a more descriptive name than bio_clone_fast? Also
> > without having the actual patches in front of me, did you make sure to
> > document the different in semantics in detail?
>
> I haven't been able to think of anything I don't hate, I'm open to
> suggestions.
Don't think it's super important, as it will die off pretty quickly
anyway. _fast implies it takes certain shortcuts and you need to be more
careful, so I don't think it's that bad.
--
Jens Axboe
next prev parent reply other threads:[~2013-11-27 0:59 UTC|newest]
Thread overview: 43+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-11-25 21:52 [GIT PULL] Immutable biovecs Kent Overstreet
2013-11-26 6:05 ` Christoph Hellwig
2013-11-27 0:44 ` Kent Overstreet
2013-11-27 0:45 ` [PATCH 01/25] block: submit_bio_wait() conversions Kent Overstreet
2013-11-27 0:45 ` [PATCH 02/25] block: Convert various code to bio_for_each_segment() Kent Overstreet
2013-11-27 0:45 ` [PATCH 03/25] bcache: Kill unaligned bvec hack Kent Overstreet
[not found] ` <1385513128-5035-1-git-send-email-kmo-PEzghdH756F8UrSeD/g0lQ@public.gmane.org>
2013-11-27 0:45 ` [PATCH 04/25] block: Abstract out bvec iterator Kent Overstreet
2013-11-27 0:45 ` [Drbd-dev] " Kent Overstreet
2013-11-27 0:45 ` Kent Overstreet
2013-11-27 18:51 ` Geoff Levand
2014-01-17 8:00 ` Geert Uytterhoeven
2014-01-17 8:00 ` Geert Uytterhoeven
2013-11-27 0:45 ` [PATCH 05/25] dm: Use bvec_iter for dm_bio_record() Kent Overstreet
2013-11-27 0:45 ` [PATCH 06/25] block: Convert bio_iovec() to bvec_iter Kent Overstreet
2013-11-27 0:45 ` [PATCH 07/25] block: Convert bio_for_each_segment() " Kent Overstreet
2013-11-27 0:45 ` Kent Overstreet
2013-11-27 0:45 ` [Drbd-dev] " Kent Overstreet
2013-11-27 0:45 ` Kent Overstreet
2013-11-27 0:45 ` [Cluster-devel] " Kent Overstreet
2013-11-27 0:45 ` Kent Overstreet
2013-11-27 0:45 ` [Drbd-dev] [PATCH 08/25] block: Immutable bio vecs Kent Overstreet
2013-11-27 0:45 ` Kent Overstreet
2013-11-27 0:45 ` Kent Overstreet
2013-11-27 0:45 ` [PATCH 09/25] block: Convert bio_copy_data() to bvec_iter Kent Overstreet
2013-11-27 0:45 ` [PATCH 10/25] bio-integrity: Convert " Kent Overstreet
2013-11-27 0:45 ` [PATCH 11/25] block: Kill bio_segments()/bi_vcnt usage Kent Overstreet
2013-11-27 0:45 ` [PATCH 12/25] block: Convert drivers to immutable biovecs Kent Overstreet
2013-11-27 0:45 ` Kent Overstreet
2013-11-27 0:45 ` [PATCH 13/25] aoe: Convert " Kent Overstreet
2013-11-27 0:45 ` [PATCH 14/25] ceph: " Kent Overstreet
2013-11-27 0:45 ` [PATCH 15/25] block: Kill bio_iovec_idx(), __bio_iovec() Kent Overstreet
2013-11-27 0:45 ` [PATCH 16/25] block: Refactor bio_clone_bioset() for immutable biovecs Kent Overstreet
2013-11-27 0:45 ` [PATCH 17/25] block: Add bio_clone_fast() Kent Overstreet
2013-11-27 0:45 ` [PATCH 18/25] rbd: Refactor bio cloning Kent Overstreet
2013-11-27 0:45 ` [PATCH 19/25] dm: Refactor for new bio cloning/splitting Kent Overstreet
2013-11-27 0:45 ` [PATCH 20/25] block: Don't save/copy bvec array anymore Kent Overstreet
2013-11-27 0:45 ` [PATCH 21/25] block: Remove bi_idx hacks Kent Overstreet
2013-11-27 0:45 ` [PATCH 22/25] block: Generic bio chaining Kent Overstreet
2013-11-27 0:45 ` [PATCH 23/25] block: Rename bio_split() -> bio_pair_split() Kent Overstreet
2013-11-27 0:45 ` [PATCH 24/25] block: Introduce new bio_split() Kent Overstreet
2013-11-27 0:45 ` [PATCH 25/25] block: Kill bio_pair_split() Kent Overstreet
2013-11-27 0:59 ` Jens Axboe [this message]
-- strict thread matches above, loose matches on Subject: below --
2013-10-15 20:20 [GIT PULL] Immutable biovecs Kent Overstreet
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20131127005954.GH9759@kernel.dk \
--to=axboe@kernel.dk \
--cc=hch@infradead.org \
--cc=kmo@daterainc.com \
--cc=linux-fsdevel@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=snitzer@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.