Commit 9e1e6df4 authored by Jaegeuk Kim's avatar Jaegeuk Kim

f2fs: put directory inodes before checkpoint in roll-forward recovery

Before checkpoint, we'd be better drop any inodes.
Signed-off-by: default avatarJaegeuk Kim <jaegeuk@kernel.org>
parent a468f0ef
...@@ -630,6 +630,9 @@ int recover_fsync_data(struct f2fs_sb_info *sbi, bool check_only) ...@@ -630,6 +630,9 @@ int recover_fsync_data(struct f2fs_sb_info *sbi, bool check_only)
set_ckpt_flags(sbi->ckpt, CP_ERROR_FLAG); set_ckpt_flags(sbi->ckpt, CP_ERROR_FLAG);
mutex_unlock(&sbi->cp_mutex); mutex_unlock(&sbi->cp_mutex);
/* let's drop all the directory inodes for clean checkpoint */
destroy_fsync_dnodes(&dir_list);
if (!err && need_writecp) { if (!err && need_writecp) {
struct cp_control cpc = { struct cp_control cpc = {
.reason = CP_RECOVERY, .reason = CP_RECOVERY,
...@@ -637,7 +640,6 @@ int recover_fsync_data(struct f2fs_sb_info *sbi, bool check_only) ...@@ -637,7 +640,6 @@ int recover_fsync_data(struct f2fs_sb_info *sbi, bool check_only)
err = write_checkpoint(sbi, &cpc); err = write_checkpoint(sbi, &cpc);
} }
destroy_fsync_dnodes(&dir_list);
kmem_cache_destroy(fsync_entry_slab); kmem_cache_destroy(fsync_entry_slab);
return ret ? ret: err; return ret ? ret: err;
} }
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment