From: Ted Ts'o <tytso@mit.edu>
To: Yongqiang Yang <xiaoqiangnk@gmail.com>
Cc: linux-ext4@vger.kernel.org, achender@linux.vnet.ibm.com, cmm@us.ibm.com
Subject: Re: [PATCH REC 0/3] ext4:Factor common code from convert and split unwritten.
Date: Thu, 14 Apr 2011 10:14:24 -0400 [thread overview]
Message-ID: <20110414141424.GG3682@thunk.org> (raw)
In-Reply-To: <1302759651-21222-1-git-send-email-xiaoqiangnk@gmail.com>
On Wed, Apr 13, 2011 at 10:40:48PM -0700, Yongqiang Yang wrote:
> These patches factor common code from ext4_ext_convert_to_initialized() and
> ext4_split_unwritten_extents(), so that extent-move-on-write in snapshot and
> punch-hole can be built on the common code as well.
>
> ext4:Add a function merging extent right and left.
> ext4:Add two functions splitting an extent.
> ext4:Reimplement convert and split_unwritten.
Hi Yongqiang,
I'd like to hear back from Mingming and Allison about whether these
patches conflict with their punch patches, since my understanding was
they were planning on doing very similar things (and at least would be
working in the very similar parts of the code).
I agree that the extents code here could definitely use this cleanup,
and I'm really glad that you worked on this; thank you! I just want
to make sure work you are doing is coordinated with the work that
Allison and Mingming are working on.
- Ted
next prev parent reply other threads:[~2011-04-14 14:14 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-04-14 5:40 [PATCH REC 0/3] ext4:Factor common code from convert and split unwritten Yongqiang Yang
2011-04-14 5:40 ` [PATCH RFC 1/3] ext4:Add a function merging extent right and left Yongqiang Yang
2011-04-14 19:03 ` Mingming Cao
2011-04-15 1:12 ` Yongqiang Yang
2011-04-15 16:39 ` Mingming Cao
2011-04-14 5:40 ` [PATCH RFC 2/3] ext4:Add two functions splitting an extent Yongqiang Yang
2011-04-20 17:21 ` Allison Henderson
2011-04-20 18:13 ` Andreas Dilger
2011-04-20 20:52 ` Allison Henderson
2011-04-21 1:18 ` Yongqiang Yang
2011-04-14 5:40 ` [PATCH RFC 3/3] ext4:Reimplement convert and split_unwritten Yongqiang Yang
2011-04-21 1:21 ` Yongqiang Yang
2011-04-21 8:28 ` Allison Henderson
2011-04-21 9:10 ` Yongqiang Yang
2011-04-21 21:13 ` Allison Henderson
2011-04-14 14:14 ` Ted Ts'o [this message]
2011-04-14 14:34 ` [PATCH REC 0/3] ext4:Factor common code from convert and split unwritten Yongqiang Yang
2011-04-14 18:45 ` Mingming Cao
2011-04-14 14:36 ` Amir Goldstein
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=20110414141424.GG3682@thunk.org \
--to=tytso@mit.edu \
--cc=achender@linux.vnet.ibm.com \
--cc=cmm@us.ibm.com \
--cc=linux-ext4@vger.kernel.org \
--cc=xiaoqiangnk@gmail.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.