summaryrefslogtreecommitdiffstats
path: root/fs/f2fs/super.c
diff options
context:
space:
mode:
authorChao Yu <chao2.yu@samsung.com>2015-06-08 07:28:03 +0200
committerJaegeuk Kim <jaegeuk@kernel.org>2015-06-08 20:20:51 +0200
commitc5bda1c8b13ad7840ae0e8c9a2926df686ba6e06 (patch)
tree8de3110094792cbbab37362724cf30f6fe758551 /fs/f2fs/super.c
parentf2fs: setting discard option in parse_options() (diff)
downloadlinux-c5bda1c8b13ad7840ae0e8c9a2926df686ba6e06.tar.xz
linux-c5bda1c8b13ad7840ae0e8c9a2926df686ba6e06.zip
f2fs: skip committing valid superblock
In recovery procedure for superblock, we try to write data of valid superblock into invalid one for recovery, work should be finished here, but then still we will write the valid one with its original data. This operation is not needed. Let's skip doing this unnecessary work. Signed-off-by: Chao Yu <chao2.yu@samsung.com> Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
Diffstat (limited to '')
-rw-r--r--fs/f2fs/super.c8
1 files changed, 5 insertions, 3 deletions
diff --git a/fs/f2fs/super.c b/fs/f2fs/super.c
index d54c54f6a49d..a06b0b46fe69 100644
--- a/fs/f2fs/super.c
+++ b/fs/f2fs/super.c
@@ -1043,7 +1043,7 @@ out:
return 0;
}
-int f2fs_commit_super(struct f2fs_sb_info *sbi)
+int f2fs_commit_super(struct f2fs_sb_info *sbi, bool recover)
{
struct buffer_head *sbh = sbi->raw_super_buf;
sector_t block = sbh->b_blocknr;
@@ -1055,7 +1055,9 @@ int f2fs_commit_super(struct f2fs_sb_info *sbi)
err = sync_dirty_buffer(sbh);
sbh->b_blocknr = block;
- if (err)
+
+ /* if we are in recovery path, skip writing valid superblock */
+ if (recover || err)
goto out;
/* write current valid superblock */
@@ -1289,7 +1291,7 @@ try_onemore:
/* recover broken superblock */
if (recovery && !f2fs_readonly(sb) && !bdev_read_only(sb->s_bdev)) {
f2fs_msg(sb, KERN_INFO, "Recover invalid superblock");
- f2fs_commit_super(sbi);
+ f2fs_commit_super(sbi, true);
}
return 0;