From: Ric Wheeler <rwheeler@redhat.com>
To: Andreas Dilger <adilger@sun.com>
Cc: Eric Sandeen <sandeen@redhat.com>,
tytso@mit.edu, 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 12:02:14 -0500 [thread overview]
Message-ID: <4B59DA16.3060906@redhat.com> (raw)
In-Reply-To: <4B7FFE9D-F110-408D-B432-7D20AEBD4689@sun.com>
On 01/22/2010 03:09 AM, Andreas Dilger wrote:
> On 2010-01-21, at 20:37, Eric Sandeen wrote:
>> That sounds fine, as do mke2fs.conf hooks, as does a nice shipped script
>> to do background checking of snapshots.
>>
>> But I still don't know why "You mounted your fs 20 times" is a good
>> proxy for "you had better check for corruption now." Have we so
>> little faith? :)
>
>
> I've thought for quite a while that 20 mounts is too often, but I'm
> reluctant to turn it off completely. I wouldn't object to increasing
> it to 60 or 80.
>
> At one time there was a patch that checked the state of the filesystem
> at mount time and only incremented only 1/5 of the time (randomly) if
> it was unmounted cleanly (not dirty, or not in recovery), but every
> time if it crashed. The reasoning was that systems which crashed are
> more likely to have memory corruption or software bugs, and ones that
> shut down cleanly are less likely to have such problems.
>
I do like the snapshot idea, but also think that we need something will
not introduce random (potentially multi-hour or multi-day) fsck runs
after an otherwise clean reboot.
If we hit this with a combination of:
Reboot time:
(1) Try to mount the file system
(1) on mount failure, fsck the failed file system
While up and running, do a periodic check with the snapshot trick.
I think that would balance the fear that we have of creeping corruption
(or at least severe corruption) against the need to be speedy when
rebooting....
ric
next prev parent reply other threads:[~2010-01-22 17:02 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 [this message]
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
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=4B59DA16.3060906@redhat.com \
--to=rwheeler@redhat.com \
--cc=adilger@sun.com \
--cc=linux-ext4@vger.kernel.org \
--cc=notting@redhat.com \
--cc=sandeen@redhat.com \
--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 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).