Re: [PATCHES/RFC v1.0.12] e2fsprogs: Next3 patch series

[Jan Kara <jack@suse.cz>]

On Tue 20-07-10 18:57:26, Amir G. wrote:
> On Tue, Jul 20, 2010 at 6:16 PM, Amir Goldstein wrote:
> > Hi guys,
> >
> > The following patches apply and tested on e2fsprogs master branch (on July 20).
> >
> > Please help me review and push these patches upstream.
> >
> > Thanks,
> > Amir.
> >
> > [PATCH 01/12] e2fsprogs: Next3 on-disk format changes
> > [PATCH 02/12] e2fsprogs: Create a big journal for Next3
> > [PATCH 03/12] e2fsprogs: Create/check exclude inode for Next3
> > [PATCH 04/12] e2fsprogs: Next3 snapshot control with chattr/lsattr -X
> > [PATCH 05/12] e2fsprogs: Avoid offline modifications to a file system with snapshots
> > [PATCH 06/12] e2fsprogs: Cleanup Next3 snapshot list when removing has_snapshot feature
> > [PATCH 07/12] e2fsprogs: Check Next3 exclude bitmap on fsck
> > [PATCH 08/12] e2fsprogs: Check Next3 snapshot list on fsck
> > [PATCH 09/12] e2fsprogs: Delete all snapshots on fsck -x
> > [PATCH 10/12] e2fsprogs: Map maximum filesystem size with a single snapshot file
> > [PATCH 11/12] e2fsprogs: Dump Next3 message buffer on fsck
> > [PATCH 12/12] e2fsprogs: Migrate old to new on-disk format
> >
> So I thought it is worth a shot to ask you personally if you will have
> time to review my Next3 patches.
> 
> In fact, the posted patches are only the small patches to e2fsprogs.
> The more challenging job is the review of the Next3 snapshot patches,
> which apply on top of ext3 (or rather a forked branch of ext3 called next3).
> They are available for download at
> http://sourceforge.net/projects/next3/files/Latest%20patch%20series
> but I can also post them to the list if you like (about 40 medium size patches).
  Well, I can have a look at those patches. But I'd like to know what is
exactly your motivation - is it that you have some customers running with
this clone and want to upstream the fs, or is it that you'd like to
contribute the cool feature you've developped, or something else? Because
on this depends where we should go from the current situation...
  To state my position: I'm not willing to merge the feature into ext3
because it's basically in a maintenance mode so we don't accept larger
features to it anymore (for stability reasons).
  You could, of course, copy ext3 code base and create a separate next3
filesystem. But such code duplication would be generally frowned upon and I
personally wouldn't like to take the burden of maintaining such code so
you'd have to do it. Moreover you have to port all ext3 fixes to your code
and you have a problem that as time progresses, new features are added to
ext4, not ext3, so I think it would be less and less attractive to run
Next3 instead of ext4... So this doesn't seem like an ideal solution either.
  For future, the most promising to me would be to change the
implementation to work with ext4 and merge it there. I understand there are
technical issues with this and I'm not sure how hard they would be to
solve. But for me as a filesystem developper this would seem like a
direction where it's worth to invest some time and energy and I can help
with that (and I believe other ext4 developpers might lend a hand as well).
  Just my thoughts... Anyway, I've added to my todo an item to have a look
at your patches so that I have better idea what we are discussing about.

								Honza
-- 
Jan Kara <jack@suse.cz>
SUSE Labs, CR