From: tytso@mit.edu
To: Ric Wheeler <rwheeler@redhat.com>
Cc: Andreas Dilger <adilger@sun.com>,
Eric Sandeen <sandeen@redhat.com>,
ext4 development <linux-ext4@vger.kernel.org>,
Bill Nottingham <notting@redhat.com>
Subject: Re: [PATCH] default max mount count to unused
Date: Fri, 22 Jan 2010 18:18:11 -0500 [thread overview]
Message-ID: <20100122231811.GL21263@thunk.org> (raw)
In-Reply-To: <4B59F50C.20601@redhat.com>
On Fri, Jan 22, 2010 at 01:57:16PM -0500, Ric Wheeler wrote:
>
> I think that we have to move towards an assumption that our
> journalling code actually works - the goal should be that we can
> *always* mount after a crash or clean reboot. That should be the
> basic test case - pound on a file system, drop power to the storage
> (and or server) and then on reboot, try to remount. Verification
> would be in the QA test case to unmount and fsck to make sure our
> journal was robust.
The original reason for the periodic fsck was not a fear that the
journalling system worked --- it was a concern that the hardware was
reliable. (Ted's law of PC class hardware: it's crap. :-) That was
the reason for the periodic fsck in the BSD days, and it's the same
now.
That being said, I agree that 20-40 reboots (it's actually randomized
by mke2fs these days; the setting in libext2fs isn't the whole story)
is a ver y rough metric. I'd much rather do the checking periodically
via snapshots in cron, at which point the reboot counter becomes moot
(the snapshot check zero's the mount count and sets the last checked
time correctly).
- Ted
prev parent reply other threads:[~2010-01-22 23:18 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-01-20 22:37 [PATCH] default max mount count to unused Eric Sandeen
2010-01-22 0:22 ` Andreas Dilger
2010-01-22 1:37 ` tytso
2010-01-22 17:42 ` Eric Sandeen
2010-01-22 18:35 ` Andreas Dilger
2010-01-22 1:29 ` tytso
2010-01-22 3:37 ` Eric Sandeen
2010-01-22 8:09 ` Andreas Dilger
2010-01-22 17:02 ` Ric Wheeler
2010-01-22 18:40 ` Andreas Dilger
2010-01-22 18:57 ` Ric Wheeler
2010-01-22 19:06 ` Andreas Dilger
2010-01-22 19:59 ` Eric Sandeen
2010-01-22 20:58 ` Valerie Aurora
2010-01-22 23:18 ` tytso [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=20100122231811.GL21263@thunk.org \
--to=tytso@mit.edu \
--cc=adilger@sun.com \
--cc=linux-ext4@vger.kernel.org \
--cc=notting@redhat.com \
--cc=rwheeler@redhat.com \
--cc=sandeen@redhat.com \
/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).