From: Theodore Ts'o <tytso@mit.edu>
To: Andreas Dilger <adilger@dilger.ca>
Cc: Ext4 Developers List <linux-ext4@vger.kernel.org>
Subject: Re: [PATCH v2] Add support for new compat feature "super_sparse"
Date: Thu, 16 Jan 2014 15:54:46 -0500 [thread overview]
Message-ID: <20140116205446.GA12104@thunk.org> (raw)
In-Reply-To: <9E6FFD6C-D0E8-4B2D-A6F6-9835F6001786@dilger.ca>
On Thu, Jan 16, 2014 at 01:21:47PM -0700, Andreas Dilger wrote:
>
> I'm OK with this in theory, but it would make it harder to know what
> features are actually enabled, especially if "ext4_default_set" is
> changing over time. Also, while this might be OK for "dumpe2fs"
> output, it shouldn't be used for the debugfs "features" command
> output, since that would break the ability to determine what features
> are actually implemented.
Yeah, I think if we were going to use sets, the sets would have to be
invariant over time. So that probably means we'd have to do things
like ext4_set_v3, ext4_set_v4, etc. And I think we'd want to have
options to both debugfs's "features" and commands to dumpe2fs which
either shows the full feature set, or the compressed version using
feature sets. There are some interesting UI design issues hiding
here, which is one of the reasons I haven't pursued this seriously for
the past couple of years.
> > I'm not sure what what you mean by "conflict with the backup
> > descriptors in #0 and #1"?
>
> In 4kB blocksize filesystems with 64-bit group descriptors, there
> are 64 group descriptors per block, so for the 32k blocks in group
> #0 this means a maximum of 32767 * 64 ~= 2M groups = 255TB before
> the group #0 group descriptors collide with the group #1 superblock
> and group #1 descriptor backups.
Ah.... yes, good point. I suspect that we'd definitely want to use
bigalloc for a file system as big as 256TB, but still, this is
something we should try to fix in the future "sparse_super2" feature.
I wonder if the right answer is that we should have two fields in the
superblock which describes which block groups have the backup
superblocks, and then the tools which do automated searching for the
bitmaps would simply search the first couple of block groups looking
for the backup superblock.
If these fields is zero, then we can also skip having the backup
superblock --- which is actually what I'd probably use at Google,
because if the file system is that badly damaged, it's not worth it to
fix it. Better to simply fix the file system by using mke2fs, and
relying on the redundancies at the cluster file system level.
- Ted
next prev parent reply other threads:[~2014-01-16 20:54 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-01-12 3:23 [PATCH RFC] Add support for new compat feature "one_backup_sb" Theodore Ts'o
2014-01-13 13:27 ` Carlos Maiolino
2014-01-13 14:06 ` Theodore Ts'o
2014-01-13 16:19 ` Theodore Ts'o
2014-01-13 20:41 ` Andreas Dilger
2014-01-13 22:02 ` Theodore Ts'o
2014-01-14 5:54 ` [PATCH v2] Add support for new compat feature "super_sparse" Theodore Ts'o
2014-01-14 11:21 ` Andreas Dilger
2014-01-14 16:08 ` Theodore Ts'o
2014-01-16 20:21 ` Andreas Dilger
2014-01-16 20:54 ` Theodore Ts'o [this message]
2014-01-14 18:42 ` Darrick J. Wong
2014-01-13 16:42 ` [PATCH RFC] Add support for new compat feature "one_backup_sb" Carlos Maiolino
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=20140116205446.GA12104@thunk.org \
--to=tytso@mit.edu \
--cc=adilger@dilger.ca \
--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).