From: Mike Snitzer <snitzer@redhat.com>
To: Kent Overstreet <kmo@daterainc.com>
Cc: axboe@kernel.dk, linux-kernel@vger.kernel.org, hch@infradead.org,
dm-devel@redhat.com, tj@kernel.org,
Alasdair Kergon <agk@redhat.com>
Subject: Re: [PATCH 17/23] dm: Refactor for new bio cloning/splitting
Date: Tue, 29 Oct 2013 20:29:36 -0400 [thread overview]
Message-ID: <20131030002935.GA3449@redhat.com> (raw)
In-Reply-To: <20131030001959.GB27129@kmo>
On Tue, Oct 29 2013 at 8:19pm -0400,
Kent Overstreet <kmo@daterainc.com> wrote:
> On Tue, Oct 29, 2013 at 08:09:08PM -0400, Mike Snitzer wrote:
> >
> > But now that I look closer, the above dm-thin.c hunks belong in:
> > [PATCH 19/23] block: Generic bio chaining
> >
> > Seems these changes were mistakenly folded into this 17/23 patch?
>
> ...Whoops, fixed. Here's the new versions if you want to take a look:
>
> http://evilpiepirate.org/git/linux-bcache.git/log/?h=for-jens
> http://evilpiepirate.org/git/linux-bcache.git/commit/?h=for-jens&id=74f0f2084f88bb38403ca80bb98976f8219f9262
Yeap, looks good (aside from the dm patches not having my Reviewed-by).
Not sure if Jens can add it?.. that said, I don't mind the commits not
having my Reviewed-by but it does lose some context for your changes
having had some other eyeballs on them.
WARNING: multiple messages have this Message-ID (diff)
From: Mike Snitzer <snitzer@redhat.com>
To: Kent Overstreet <kmo@daterainc.com>
Cc: axboe@kernel.dk, dm-devel@redhat.com,
linux-kernel@vger.kernel.org, hch@infradead.org, tj@kernel.org,
Alasdair Kergon <agk@redhat.com>
Subject: Re: [PATCH 17/23] dm: Refactor for new bio cloning/splitting
Date: Tue, 29 Oct 2013 20:29:36 -0400 [thread overview]
Message-ID: <20131030002935.GA3449@redhat.com> (raw)
In-Reply-To: <20131030001959.GB27129@kmo>
On Tue, Oct 29 2013 at 8:19pm -0400,
Kent Overstreet <kmo@daterainc.com> wrote:
> On Tue, Oct 29, 2013 at 08:09:08PM -0400, Mike Snitzer wrote:
> >
> > But now that I look closer, the above dm-thin.c hunks belong in:
> > [PATCH 19/23] block: Generic bio chaining
> >
> > Seems these changes were mistakenly folded into this 17/23 patch?
>
> ...Whoops, fixed. Here's the new versions if you want to take a look:
>
> http://evilpiepirate.org/git/linux-bcache.git/log/?h=for-jens
> http://evilpiepirate.org/git/linux-bcache.git/commit/?h=for-jens&id=74f0f2084f88bb38403ca80bb98976f8219f9262
Yeap, looks good (aside from the dm patches not having my Reviewed-by).
Not sure if Jens can add it?.. that said, I don't mind the commits not
having my Reviewed-by but it does lose some context for your changes
having had some other eyeballs on them.
next prev parent reply other threads:[~2013-10-30 0:29 UTC|newest]
Thread overview: 48+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-10-29 20:17 [PATCH] Immutable biovecs Kent Overstreet
2013-10-29 20:17 ` [PATCH 01/23] block: Use rw_copy_check_uvector() Kent Overstreet
2013-10-29 20:17 ` [PATCH 02/23] block: Consolidate duplicated bio_trim() implementations Kent Overstreet
2013-10-29 20:17 ` [PATCH 03/23] bcache: Kill unaligned bvec hack Kent Overstreet
2013-10-29 20:17 ` [PATCH 04/23] block: Abstract out bvec iterator Kent Overstreet
2013-10-29 20:17 ` [Drbd-dev] " Kent Overstreet
2013-10-29 20:17 ` Kent Overstreet
2013-10-31 3:29 ` NeilBrown
2013-10-31 3:29 ` [Drbd-dev] " NeilBrown
2013-10-31 3:29 ` NeilBrown
2013-10-31 17:14 ` Ed Cashin
2013-10-31 17:14 ` [Drbd-dev] " Ed Cashin
2013-10-31 17:14 ` Ed Cashin
2013-10-29 20:17 ` [PATCH 05/23] dm: Use bvec_iter for dm_bio_record() Kent Overstreet
2013-10-29 20:17 ` [PATCH 06/23] block: Convert bio_iovec() to bvec_iter Kent Overstreet
2013-10-29 20:17 ` Kent Overstreet
2013-10-29 20:18 ` [PATCH 07/23] block: Convert bio_for_each_segment() " Kent Overstreet
2013-10-29 20:18 ` Kent Overstreet
2013-10-29 20:18 ` Kent Overstreet
2013-10-29 20:18 ` Kent Overstreet
2013-10-29 20:18 ` [Drbd-dev] " Kent Overstreet
2013-10-29 20:18 ` Kent Overstreet
2013-10-29 20:18 ` [Cluster-devel] " Kent Overstreet
2013-10-29 20:18 ` [Drbd-dev] [PATCH 08/23] block: Immutable bio vecs Kent Overstreet
2013-10-29 20:18 ` Kent Overstreet
2013-10-29 20:18 ` [PATCH 09/23] block: Convert bio_copy_data() to bvec_iter Kent Overstreet
2013-10-29 20:18 ` [PATCH 10/23] bio-integrity: Convert " Kent Overstreet
2013-10-29 20:18 ` [PATCH 11/23] block: Kill bio_segments()/bi_vcnt usage Kent Overstreet
2013-10-29 20:18 ` [PATCH 12/23] block: Convert drivers to immutable biovecs Kent Overstreet
2013-10-29 20:18 ` [PATCH 13/23] aoe: Convert " Kent Overstreet
2013-10-29 20:18 ` [PATCH 14/23] ceph: " Kent Overstreet
2013-10-29 20:18 ` [PATCH 15/23] block: Kill bio_iovec_idx(), __bio_iovec() Kent Overstreet
2013-10-29 20:18 ` [PATCH 16/23] rbd: Refactor bio cloning, don't clone biovecs Kent Overstreet
2013-10-29 20:18 ` [PATCH 17/23] dm: Refactor for new bio cloning/splitting Kent Overstreet
2013-10-29 23:04 ` Mike Snitzer
2013-10-30 0:09 ` Mike Snitzer
2013-10-30 0:19 ` Kent Overstreet
2013-10-30 0:29 ` Mike Snitzer [this message]
2013-10-30 0:29 ` Mike Snitzer
2013-10-31 14:05 ` Jens Axboe
2013-10-29 20:18 ` [PATCH 18/23] block: Remove bi_idx hacks Kent Overstreet
2013-10-29 20:18 ` [PATCH 19/23] block: Generic bio chaining Kent Overstreet
2013-10-29 20:18 ` [PATCH 20/23] block: Rename bio_split() -> bio_pair_split() Kent Overstreet
2013-10-29 20:18 ` [PATCH 21/23] block: Introduce new bio_split() Kent Overstreet
2013-10-29 20:18 ` [PATCH 22/23] block: Kill bio_pair_split() Kent Overstreet
2013-10-29 20:18 ` [PATCH 23/23] block: Don't save/copy bvec array anymore, share when cloning Kent Overstreet
2013-10-29 20:36 ` [PATCH] Immutable biovecs Jens Axboe
2013-10-30 0:06 ` 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=20131030002935.GA3449@redhat.com \
--to=snitzer@redhat.com \
--cc=agk@redhat.com \
--cc=axboe@kernel.dk \
--cc=dm-devel@redhat.com \
--cc=hch@infradead.org \
--cc=kmo@daterainc.com \
--cc=linux-kernel@vger.kernel.org \
--cc=tj@kernel.org \
/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.