From: Pierre Ossman <drzeus-list@drzeus.cx>
To: "Jörn Engel" <joern@wohnheim.fh-wedel.de>
Cc: LKML <linux-kernel@vger.kernel.org>
Subject: Re: Flash erase groups and filesystems
Date: Wed, 17 Aug 2005 13:35:11 +0200 [thread overview]
Message-ID: <430320EF.3070907@drzeus.cx> (raw)
In-Reply-To: <20050816185230.GA2931@wohnheim.fh-wedel.de>
Jörn Engel wrote:
>On Tue, 16 August 2005 20:13:36 +0200, Jörn Engel wrote:
>
>
>>Yes. Most filesystems expect to find either 1) old data or 2) new
>>data. Blocks full of 0xff are non-expected.
>>
>>
>
>Maybe this isn't obvious. Because of this expectation, it is
>absolutely not safe to pre-erase blocks, just because the fs will
>write them anyway. Unless you can guarantee that the write will
>always succeed, even in case of power outage, you just broke the
>expectation.
>
>
>
Darn. I suspected as much. I'll guess the erase function will have to be
scrapped for now...
Whilst we're on the subject, do the filesystems assume that the device
can tell them exactly where the write failed? I.e. if the driver knows
that 5 sectors were written correctly, but that it failed somewhere
beyond that. It might have failed at sector 6, but it might also have
failed at sector 10. The assumption that sectors contain either old or
new data is still true, we're just unsure which. This can be the case
when you feed a controller a lot of data and it can only report back
success or failure.
>Fixing all filesystem is also not an option, even ignoring the
>question whether such a change would be a fix, a change of behaviour
>or a plain bug.
>
>So the only remaining option is to add a new interface that lets
>filesystems decide to support pre-erase in some form. And one such
>interface would be the "forget" operation. Nice attribute of forget
>is the fact that it would also help some FTL layers in the kernel.
>There is nothing MMC-specific about it.
>
>
>
A bit too much work for me right now. But I'll be there with my erase
patch when someone implements it. :)
Rgds
Pierre
next prev parent reply other threads:[~2005-08-17 11:35 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2005-08-15 20:21 Flash erase groups and filesystems Pierre Ossman
2005-08-16 16:27 ` Jörn Engel
2005-08-16 17:09 ` Pierre Ossman
2005-08-16 18:13 ` Jörn Engel
2005-08-16 18:52 ` Jörn Engel
2005-08-17 11:35 ` Pierre Ossman [this message]
2005-08-17 11:45 ` Jörn Engel
2005-08-17 14:35 ` Pavel Machek
2005-08-18 11:09 ` linux-os (Dick Johnson)
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=430320EF.3070907@drzeus.cx \
--to=drzeus-list@drzeus.cx \
--cc=joern@wohnheim.fh-wedel.de \
--cc=linux-kernel@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