From: Jeff Garzik <jeff@garzik.org>
To: Andrew Morton <akpm@osdl.org>
Cc: Mingming Cao <cmm@us.ibm.com>,
linux-kernel@vger.kernel.org, ext2-devel@lists.sourceforge.net,
linux-fsdevel@vger.kernel.org
Subject: Re: [PATCH 1/5] Forking ext4 filesystem from ext3 filesystem
Date: Thu, 10 Aug 2006 16:52:49 -0400 [thread overview]
Message-ID: <44DB9CA1.2050306@garzik.org> (raw)
In-Reply-To: <20060810133338.8d1f6061.akpm@osdl.org>
Andrew Morton wrote:
> On Thu, 10 Aug 2006 16:22:26 -0400
> Jeff Garzik <jeff@garzik.org> wrote:
>
>> I strongly disagree that ext3 should be subject to a spring cleaning.
>> Comments, whitespace, very very minor things, sure. Trying to get rid
>> of brelse() when _many_ other filesystems also use it? ext4 material.
>
> We should seek to minimise the difficulty of cross-porting bugfixes and
> enhancements. Putting cleanups in only ext4 works against that.
>
> ext3 will be around for many years yet. We cannot just let it rot due to
> some false belief that performing routine maintenance against it will for
> some magical reason cause it to break.
Because ext4 is impending, you want to push a bunch of cleanups into
ext3 over a short span of time. That's not routine maintenance at all.
We're not talking about routine maintenance. In your words, we are
talking about spring cleaning.
Why not let the devel/stable system work its magic? If the cleanups are
viable, proving that first in ext4 should give us more confidence to put
them into ext3.
Cross-porting bugfixes and cleanups will _obviously_ be quite easy,
during the first few months of ext4's life.
Just look at ext2->ext3 history. Regardless of when you make the split,
there will be a bunch of stuff people wish to backport after the split
occurs. Given that, it makes more sense to testbed the changes in ext4
first.
Jeff
next prev parent reply other threads:[~2006-08-10 20:52 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-08-10 1:17 [PATCH 1/5] Forking ext4 filesystem from ext3 filesystem Mingming Cao
2006-08-10 6:39 ` Andrew Morton
2006-08-10 16:41 ` Mingming Cao
2006-08-10 16:48 ` Jörn Engel
2006-08-10 18:18 ` Andrew Morton
2006-08-10 20:22 ` Jeff Garzik
2006-08-10 20:33 ` Andrew Morton
2006-08-10 20:52 ` Jeff Garzik [this message]
2006-08-10 17:44 ` Theodore Tso
2006-08-10 18:51 ` Badari Pulavarty
2006-08-10 19:23 ` Andrew Morton
2006-08-10 19:36 ` Dave Kleikamp
2006-08-10 19:54 ` Andrew Morton
2006-08-10 20:12 ` Jeff Garzik
2006-08-10 20:13 ` Jeff Garzik
2006-08-10 20:27 ` Andrew Morton
2006-08-10 21:00 ` Jeff Garzik
2006-08-10 21:11 ` Alex Tomas
2006-08-10 22:18 ` [Ext2-devel] " Andrew Morton
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=44DB9CA1.2050306@garzik.org \
--to=jeff@garzik.org \
--cc=akpm@osdl.org \
--cc=cmm@us.ibm.com \
--cc=ext2-devel@lists.sourceforge.net \
--cc=linux-fsdevel@vger.kernel.org \
--cc=linux-kernel@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).