All of lore.kernel.org
 help / color / mirror / Atom feed
From: Arnd Bergmann <arnd.bergmann@linaro.org>
To: Nicolas Pitre <nicolas.pitre@linaro.org>
Cc: "Ted Ts'o" <tytso@mit.edu>,
	Alex Lemberg <Alex.Lemberg@sandisk.com>,
	HYOJIN JEONG <syr.jeong@samsung.com>,
	Saugata Das <saugata.das@linaro.org>,
	Artem Bityutskiy <dedekind1@gmail.com>,
	Saugata Das <saugata.das@stericsson.com>,
	linux-ext4@vger.kernel.org, linux-fsdevel@vger.kernel.org,
	linux-mmc@vger.kernel.org, patches@linaro.org, venkat@linaro.org,
	"Luca Porzio (lporzio)" <lporzio@micron.com>
Subject: Re: [PATCH 2/3] ext4: Context support
Date: Fri, 15 Jun 2012 09:19:23 +0000	[thread overview]
Message-ID: <201206150919.23484.arnd.bergmann@linaro.org> (raw)
In-Reply-To: <alpine.LFD.2.02.1206141505300.23555@xanadu.home>

On Thursday 14 June 2012, Nicolas Pitre wrote:
> On Thu, 14 Jun 2012, Ted Ts'o wrote:
>
> > The reason why I talk about making it work automatically at mke2fs
> > time is that the vast majority of created file systems (where a
> > specially created fs by a handset vendor counts as "one", even if it
> > then gets stamped on millions of devices), the end user is someone
> > naive/oblivious, so the right thing has to happen by default in the
> > common case of running mke2fs on the storage device where the file
> > system gets used.
> 
> Absolutely.  However it is fair to say that less than 0.01% of total end 
> users will even think of running mke2fs on their device.  So another 
> strategy that can be executed at run time when the fs is live would be 
> required too.

The trouble is that detecting the erase block size requires us to
write specific patterns to the device, which is generally a bad
idea after the file system has been created.

I think the best we can do is

* default to "unspecified" as before
* if "unspecified", make the file system ask the block device. in
  case of eMMC, that will usually be reliable
* Add an option to mkfs and tunefs to hardcode a specific size for
  users that know the size and can't rely on the blockdev reporting
  it correctly to the file system.
* Add an option to mkfs to autodetect the size for the drive it's
  run on.

	Arnd

  reply	other threads:[~2012-06-15  9:19 UTC|newest]

Thread overview: 45+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-06-11 10:46 [PATCH 1/3] block: Context support Saugata Das
2012-06-11 10:46 ` Saugata Das
2012-06-11 10:46 ` [PATCH 2/3] ext4: " Saugata Das
2012-06-11 10:46   ` Saugata Das
2012-06-11 11:41   ` Artem Bityutskiy
2012-06-11 12:27     ` Ted Ts'o
2012-06-12 12:21       ` Saugata Das
2012-06-12 12:32         ` Ted Ts'o
2012-06-12 13:29         ` Arnd Bergmann
2012-06-12 14:26           ` Saugata Das
2012-06-12 14:55             ` Arnd Bergmann
2012-06-12 18:19               ` Ted Ts'o
2012-06-12 20:07                 ` Arnd Bergmann
2012-06-12 20:41                   ` Ted Ts'o
2012-06-13 19:44                     ` Arnd Bergmann
2012-06-13 20:00                       ` Ted Ts'o
2012-06-13 20:43                         ` Arnd Bergmann
2012-06-14  2:07                           ` Ted Ts'o
2012-06-14 16:14                             ` Nicolas Pitre
2012-06-14 16:24                               ` Artem Bityutskiy
2012-06-14 17:05                               ` Ted Ts'o
2012-06-14 19:08                                 ` Nicolas Pitre
2012-06-15  9:19                                   ` Arnd Bergmann [this message]
2012-06-15 21:30                                     ` Ted Ts'o
2012-06-16  6:49                                       ` Arnd Bergmann
2012-06-14 21:55                             ` Arnd Bergmann
2012-06-15  5:18                               ` Andreas Dilger
2012-06-15  9:25                                 ` Arnd Bergmann
2012-06-15  9:40                                   ` Andreas Dilger
2012-06-15 10:54                                     ` Arnd Bergmann
2012-06-15 22:04                               ` Ted Ts'o
2012-06-15 22:25                                 ` Andreas Dilger
2012-06-16  7:14                                   ` Arnd Bergmann
2012-06-16  7:28                                     ` Arnd Bergmann
2012-06-16  7:26                                 ` Arnd Bergmann
2012-06-16 13:49                                   ` Ted Ts'o
2012-06-16 17:41                                     ` Arnd Bergmann
2012-06-18 17:42                                       ` Ted Ts'o
2012-06-19 15:17                                         ` Arnd Bergmann
2012-06-20 15:09                                           ` Luca Porzio (lporzio)
2012-06-20 15:46                                             ` Arnd Bergmann
2012-06-22 13:29                       ` Artem Bityutskiy
2012-06-22 14:07                         ` Luca Porzio (lporzio)
2012-06-11 10:46 ` [PATCH 3/3] mmc: " Saugata Das
2012-06-11 10:46   ` Saugata Das

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=201206150919.23484.arnd.bergmann@linaro.org \
    --to=arnd.bergmann@linaro.org \
    --cc=Alex.Lemberg@sandisk.com \
    --cc=dedekind1@gmail.com \
    --cc=linux-ext4@vger.kernel.org \
    --cc=linux-fsdevel@vger.kernel.org \
    --cc=linux-mmc@vger.kernel.org \
    --cc=lporzio@micron.com \
    --cc=nicolas.pitre@linaro.org \
    --cc=patches@linaro.org \
    --cc=saugata.das@linaro.org \
    --cc=saugata.das@stericsson.com \
    --cc=syr.jeong@samsung.com \
    --cc=tytso@mit.edu \
    --cc=venkat@linaro.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.