All of lore.kernel.org
 help / color / mirror / Atom feed
From: Josh Triplett <josh@joshtriplett.org>
To: "Theodore Y. Ts'o" <tytso@mit.edu>
Cc: linux-ext4@vger.kernel.org
Subject: Re: Use case for EXT4_INODE_HUGE_FILE / EXT4_HUGE_FILE_FL?
Date: Tue, 7 Apr 2020 01:09:25 -0700	[thread overview]
Message-ID: <20200407080925.GA675720@localhost> (raw)
In-Reply-To: <20200407033031.GT45598@mit.edu>

On Mon, Apr 06, 2020 at 11:30:31PM -0400, Theodore Y. Ts'o wrote:
> On Mon, Apr 06, 2020 at 03:45:34PM -0700, Josh Triplett wrote:
> > Under what circumstances can an inode ever end up with EXT4_HUGE_FILE_FL
> > set? (Other than in an artificially constructed filesystem.)
> > 
> > Was EXT4_HUGE_FILE_FL just added for future extensibility, in case a
> > future file storage mechanism allows storing files bigger than 2**32
> > blocks?
> 
> Yes. basically.  When we added the huge_file feature, which introduced
> the i_blocks_hi field, the thinking was to add EXT4_HUGE_FILE_FL so
> that we could painlessly upgrade a file system from ext3 (w/o the huge
> file feature) to enabling the feature without having to rewrite all of
> the inodes.  However, we also didn't want to artificially limit
> ourselves to 2**57 file sizes, so we also added the EXT4_HUGE_FILE_FL
> flag.

Thanks for the explanation! That makes sense.

> It hasn't gotten a huge amount of testing in a while, but it would be
> relatively easy to add debugging code (triggered via a mount option or
> a sysfs file) which forces the use of EXT4_HUGE_FILE_FL all the time.

That does seem like a good idea. It would also be nice to have an e2fsck
option to rewrite all inodes to use EXT4_HUGE_FILE_FL.

I think I'll avoid poking that code for now, though, since I don't
currently have a need for files anywhere near that large.

      reply	other threads:[~2020-04-07  8:09 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-04-06 22:45 Use case for EXT4_INODE_HUGE_FILE / EXT4_HUGE_FILE_FL? Josh Triplett
2020-04-07  3:30 ` Theodore Y. Ts'o
2020-04-07  8:09   ` Josh Triplett [this message]

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=20200407080925.GA675720@localhost \
    --to=josh@joshtriplett.org \
    --cc=linux-ext4@vger.kernel.org \
    --cc=tytso@mit.edu \
    /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.