diff options
| author | Theodore Ts'o <tytso@mit.edu> | 2014-03-14 06:49:42 +0400 |
|---|---|---|
| committer | Theodore Ts'o <tytso@mit.edu> | 2014-03-14 06:49:42 +0400 |
| commit | 38c03b34391dd25a39576073e58485e5949d29fe (patch) | |
| tree | 91236e16c5d4534667aa04367c9be86cebc22af8 | |
| parent | 02b9984d640873b7b3809e63f81a0d7e13496886 (diff) | |
| download | linux-38c03b34391dd25a39576073e58485e5949d29fe.tar.xz | |
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>
| -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 a5f1170048bd..89baee42f353 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; |