diff options
author | Theodore Ts'o <tytso@mit.edu> | 2014-03-13 22:49:42 -0400 |
---|---|---|
committer | Theodore Ts'o <tytso@mit.edu> | 2014-03-13 22:49:42 -0400 |
commit | 38c03b34391dd25a39576073e58485e5949d29fe (patch) | |
tree | 91236e16c5d4534667aa04367c9be86cebc22af8 /fs/ext4 | |
parent | 02b9984d640873b7b3809e63f81a0d7e13496886 (diff) | |
download | op-kernel-dev-38c03b34391dd25a39576073e58485e5949d29fe.zip op-kernel-dev-38c03b34391dd25a39576073e58485e5949d29fe.tar.gz |
ext4: only call sync_filesystm() when remounting read-only
This is the only time it is required for ext4.
Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
Diffstat (limited to 'fs/ext4')
-rw-r--r-- | fs/ext4/super.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/fs/ext4/super.c b/fs/ext4/super.c index a5f1170..89baee4 100644 --- a/fs/ext4/super.c +++ b/fs/ext4/super.c @@ -4765,8 +4765,6 @@ static int ext4_remount(struct super_block *sb, int *flags, char *data) #endif char *orig_data = kstrdup(data, GFP_KERNEL); - sync_filesystem(sb); - /* Store the original options */ old_sb_flags = sb->s_flags; old_opts.s_mount_opt = sbi->s_mount_opt; @@ -4837,6 +4835,9 @@ static int ext4_remount(struct super_block *sb, int *flags, char *data) } if (*flags & MS_RDONLY) { + err = sync_filesystem(sb); + if (err < 0) + goto restore_opts; err = dquot_suspend(sb, -1); if (err < 0) goto restore_opts; |