diff options
author | Eric Sandeen <sandeen@redhat.com> | 2012-05-24 12:00:37 -0500 |
---|---|---|
committer | Jan Kara <jack@suse.cz> | 2012-07-09 12:03:11 +0200 |
commit | f007dbf8e51f4a0910194eebc2aa119eb861893e (patch) | |
tree | 568ef2396a9396ac7adf69d18811f40a43302b0d /fs/ext3/super.c | |
parent | f3da93105b6963a2be2a56dee27fdc88ac4ad769 (diff) | |
download | op-kernel-dev-f007dbf8e51f4a0910194eebc2aa119eb861893e.zip op-kernel-dev-f007dbf8e51f4a0910194eebc2aa119eb861893e.tar.gz |
ext3: force ro mount if ext3_setup_super() fails
If ext3_setup_super() fails i.e. due to a too-high revision,
the error is logged in dmesg but the fs is not mounted RO as
indicated.
Tested by:
[164152.114551] EXT3-fs (sdb6): error: revision level too high, forcing read-only mode
/dev/sdb6 /mnt/test2 ext3 rw,seclabel,relatime,errors=continue,user_xattr,acl,barrier=1,data=ordered 0 0
^^
Signed-off-by: Eric Sandeen <sandeen@redhat.com>
Reviewed-by: Andreas Dilger <adilger@whamcloud.com>
Signed-off-by: Jan Kara <jack@suse.cz>
Diffstat (limited to 'fs/ext3/super.c')
-rw-r--r-- | fs/ext3/super.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/fs/ext3/super.c b/fs/ext3/super.c index 8c3a44b..b4e1992 100644 --- a/fs/ext3/super.c +++ b/fs/ext3/super.c @@ -2058,7 +2058,8 @@ static int ext3_fill_super (struct super_block *sb, void *data, int silent) goto failed_mount3; } - ext3_setup_super (sb, es, sb->s_flags & MS_RDONLY); + if (ext3_setup_super(sb, es, sb->s_flags & MS_RDONLY)) + sb->s_flags |= MS_RDONLY; EXT3_SB(sb)->s_mount_state |= EXT3_ORPHAN_FS; ext3_orphan_cleanup(sb, es); |