All of lore.kernel.org
 help / color / mirror / Atom feed
From: Theodore Tso <tytso@mit.edu>
To: "Jose R. Santos" <jrs@us.ibm.com>, Andreas Dilger <adilger@sun.com>
Cc: linux-ext4 <linux-ext4@vger.kernel.org>
Subject: Re: Ininitial e2fsprogs TODO list (please expand)
Date: Wed, 16 Apr 2008 23:36:16 -0400	[thread overview]
Message-ID: <20080417033616.GA2850@mit.edu> (raw)
In-Reply-To: <20080416033002.GZ3106@webber.adilger.int> <20080415115216.1f6868de@gara.konoha.net>

On Tue, Apr 15, 2008 at 11:52:16AM -0500, Jose R. Santos wrote:
> As discuss on the call yesterday, some folks (my self included) really
> want a TODO list to help them keep track of what things are left undone
> in e2fsprogs as we try to get ext4 out the door.  Here is my initial
> list of items that still need addressing.  Hopefully we can expand this
> list and document it somewhere like the ext4 wiki or the SourceForge
> bug tracker.
> 
> - Rename uninit_groups to uninit_bg to be consistent with other
> defined features.  Retain the old name for historical purpose.

Yes.  Although until we actually don't do lazy initialization of the
inode table, I still think the name is a bit of a misnomer.  It really
is more about checksuming the block group descriptors and a faster
fsck, but whether or not we initialize the block groups or not is
pretty much a non-issue.

> - The return value of ext2fs_super_and_bgd_loc() is not to be trusted.
> Document this in the source code.
> 
> - Make sure ext2fs_super_and_bgd_loc() does not get used anywhere where
> the return value is expected to be accurate (aside from mke2fs).
> 
> - Remove lazy_bg feature from being set in mke2fs.  Feature has been
> declare a dangerous hack by its creator, remove it to avoid people
> building on top of it.

.... and to replace it, add a configuration parameter to
/etc/e2fsck.conf which controls whether or not the inode table and
bitmap blocks should be uninitialized when using uninit groups.  It
will default to off for now, until the kernel support can be
implemented.

> - Add flex_bg meta-data grouping support.

Once it is demonstrated to work correctly in all circumstances.  :-)

     	   		   		     - Ted

  parent reply	other threads:[~2008-04-17  3:36 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-04-15 16:52 Ininitial e2fsprogs TODO list (please expand) Jose R. Santos
2008-04-16  3:30 ` Andreas Dilger
2008-04-16  4:35   ` Jose R. Santos
2008-04-17  3:26     ` Andreas Dilger
2008-04-17  3:36 ` Theodore Tso [this message]
2008-04-20 23:47 ` Theodore Tso
2008-04-21 13:11   ` Theodore Tso
2008-04-21 21:29     ` Eric Sandeen

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=20080417033616.GA2850@mit.edu \
    --to=tytso@mit.edu \
    --cc=adilger@sun.com \
    --cc=jrs@us.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 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.