Re: [PATCH 3/3] e2fsprogs: Support for large inode migration.

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

 



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
-
To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Reiser Filesystem Development]     [Ceph FS]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux FS]     [Yosemite National Park]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Device Mapper]     [Linux Media]

  Powered by Linux