From: Theodore Tso <tytso@mit.edu>
To: "Aneesh Kumar K.V" <aneesh.kumar@linux.vnet.ibm.com>
Cc: linux-ext4@vger.kernel.org
Subject: Re: [PATCH 3/3] e2fsprogs: Support for large inode migration.
Date: Fri, 27 Jul 2007 11:34:19 -0400 [thread overview]
Message-ID: <20070727153419.GA13515@thunk.org> (raw)
In-Reply-To: <46A95F93.3020001@linux.vnet.ibm.com>
On Fri, Jul 27, 2007 at 08:29:31AM +0530, Aneesh Kumar K.V wrote:
> What are the issues you see with PATCH 1 and PATCH 2 which implement
> Undo I/O Manager and undoe2fs other than it is not hooked into any of
> the existing tools. I will try to add it to mke2fs as you suggested. But
> should that prevent it from going in ?
The main issue is that it's not used by an in-tree caller. If you can
add it to mke2fs, that would be great; otherwise, it's something
that's on my todo list so we can merge the undo manager. The other
possibility is that I might use this as justification for creating a
"pu" (proposed update) branch which has the property that patches on
the pu branch can get removed or modified at any time, and the "pu"
branch can get rewound or rebased at any time.
See the section of text starting at line #68 (and going on to line
#160) in the following git's maintainer notes for a much more
comprehensive description of "maint", "master", "next", and "pu",
which is the direction in which I plan to take the e2fsprogs git tree:
http://git.kernel.org/?p=git/git.git;a=blob;f=MaintNotes;h=8bf0352adf2b4ac775d6100ad937600ecb5be5f2;hb=962753f75390f5c5ea23a2d1e6996f6027003478
But yes, the main reason why I haven't merged the undo manager is
because we don't have an in-tree user of that interface. We don't
even have test cases in the tree, either (also on my todo list, but
feel free to beat me to it).
- Ted
next prev parent reply other threads:[~2007-07-27 15:34 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-07-25 5:36 e2fsprogs: Undo I/O Manager and large inode migration support in tune2fs Aneesh Kumar K.V
[not found] ` <bee58d48110eee4d5cd133167245b99644148d96.1185341470.git.aneesh.kumar@linux.vnet.ibm.com>
2007-07-25 5:36 ` [PATCH 1/3] e2fsprogs: Add undo I/O manager Aneesh Kumar K.V
[not found] ` <47f96570519d76b8d59f92b729a0a48c4a1b68d8.1185341470.git.aneesh.kumar@linux.vnet.ibm.com>
2007-07-25 5:36 ` [PATCH 2/3] e2fsprogs: Add undoe2fs Aneesh Kumar K.V
[not found] ` <3ae4c55b831a13f9fbb9a187efcd65d29434bf09.1185341470.git.aneesh.kumar@linux.vnet.ibm.com>
2007-07-25 5:36 ` [PATCH 3/3] e2fsprogs: Support for large inode migration Aneesh Kumar K.V
2007-07-25 14:32 ` Theodore Tso
2007-07-25 19:46 ` Andreas Dilger
2007-07-26 14:58 ` Theodore Tso
2007-07-26 11:45 ` Aneesh Kumar K.V
2007-07-26 16:13 ` Theodore Tso
2007-07-27 2:59 ` Aneesh Kumar K.V
2007-07-27 15:34 ` Theodore Tso [this message]
2007-07-27 19:03 ` Aneesh Kumar K.V
2007-07-25 19:49 ` Andreas Dilger
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=20070727153419.GA13515@thunk.org \
--to=tytso@mit.edu \
--cc=aneesh.kumar@linux.vnet.ibm.com \
--cc=linux-ext4@vger.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).