From: Eric Sandeen <sandeen@redhat.com>
To: ext4 development <linux-ext4@vger.kernel.org>
Cc: Bill Nottingham <notting@redhat.com>
Subject: [PATCH] default max mount count to unused
Date: Wed, 20 Jan 2010 16:37:25 -0600 [thread overview]
Message-ID: <4B5785A5.2010505@redhat.com> (raw)
From: Bill Nottingham <notting@redhat.com>
Anaconda has been setting the max mount count on the root fs
to -1 (unused) for ages.
I (Eric) tend to agree that using mount count as a proxy for potential
for corruption seems odd. And waiting for fsck on a reboot just because
it's number 20 (or so) is painful. Can we just turn it off by default?
I wouldn't mind killing the periodic check as well, but consider
this a trial balloon. :)
Signed-off-by: Eric Sandeen <sandeen@redhat.com>
---
diff --git a/doc/libext2fs.texinfo b/doc/libext2fs.texinfo
index 19899bc..a0439bf 100644
--- a/doc/libext2fs.texinfo
+++ b/doc/libext2fs.texinfo
@@ -319,7 +319,8 @@ skip the filesystem check if the number of times that the filesystem has
been mounted is less than @code{s_max_mnt_count} and if the interval
between the last time a filesystem check was performed and the current
time is less than @code{s_checkinterval} (see below). The default value
-of @code{s_max_mnt_count} is 20.
+of @code{s_max_mnt_count} is -1 (which means that this check is not
+done).
@item s_checkinterval
This field defines the minimal interval between filesystem checks. See
diff --git a/lib/ext2fs/ext2_fs.h b/lib/ext2fs/ext2_fs.h
index 114b001..b98d6e8 100644
--- a/lib/ext2fs/ext2_fs.h
+++ b/lib/ext2fs/ext2_fs.h
@@ -484,7 +484,7 @@ struct ext2_inode_large {
/*
* Maximal mount counts between two filesystem checks
*/
-#define EXT2_DFL_MAX_MNT_COUNT 20 /* Allow 20 mounts */
+#define EXT2_DFL_MAX_MNT_COUNT -1 /* Don't use mount check */
#define EXT2_DFL_CHECKINTERVAL 0 /* Don't use interval check */
/*
next reply other threads:[~2010-01-20 22:37 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-01-20 22:37 Eric Sandeen [this message]
2010-01-22 0:22 ` [PATCH] default max mount count to unused Andreas Dilger
2010-01-22 1:37 ` tytso
2010-01-22 1:37 ` [linux-lvm] " 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
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=4B5785A5.2010505@redhat.com \
--to=sandeen@redhat.com \
--cc=linux-ext4@vger.kernel.org \
--cc=notting@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 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.