All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Theodore Ts'o" <tytso@mit.edu>
To: "Toralf Förster" <toralf.foerster@gmx.de>
Cc: Linux Kernel <linux-kernel@vger.kernel.org>
Subject: Re: RFC: kconfig option CONFIG_LBDAF should be auto-selected if EXT4 is chosen
Date: Sat, 27 Oct 2012 04:24:13 -0400	[thread overview]
Message-ID: <20121027082413.GB12045@thunk.org> (raw)
In-Reply-To: <508B94AE.9050505@gmx.de>

On Sat, Oct 27, 2012 at 10:00:46AM +0200, Toralf Förster wrote:
> 
> otherwise booting an EXT4 image won't work b/c the root fs can't be
> mounted read-write - and as a side effect no syslog messages might
> point the user to this issue (at least in my setup) ...

This is only a problem on 32-bit architectues. and it's actually
documented in the Kconfig help message, and it's enabled by default.
So someone would have had to have gone out of their way to turn off
CONFIG_LBDAF.  This is the first time I've heard of someone getting
surprised by this; do you know how CONFIG_LBDAF got disabled?  Did you
just disable it without reading the help message?

What we might be able to do instead is to allow the mount to succeed,
but to simply fail any attempts to open(2) or stat(2) files larger
than 2TB with an EFBIG error.

BTW, technically ext4 does not _require_ CONFIG_LBDAF.  It's just that
the default mke2fs.conf enables huge_file, so most ext4 file systems
enable it by default.  I could imagine systems (like 32-bit Android
handsets) where it's not intended to ever support storage devices
larger than 2TB, so they might very well wnat CONFIG_EXT4 &&
!CONFIG_LBDAF.  That's a valid choice, although I do admit some users
might get surprised by this.

Regards,

					- Ted





  reply	other threads:[~2012-10-27  8:24 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <508A967F.4080802@gmx.de>
2012-10-27  8:00 ` RFC: kconfig option CONFIG_LBDAF should be auto-selected if EXT4 is chosen Toralf Förster
2012-10-27  8:24   ` Theodore Ts'o [this message]
2012-10-27  8:40     ` Toralf Förster

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=20121027082413.GB12045@thunk.org \
    --to=tytso@mit.edu \
    --cc=linux-kernel@vger.kernel.org \
    --cc=toralf.foerster@gmx.de \
    /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.