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

Jaegeuk Kim · Jaegeuk Kim · commit 9e1e6df412a2 · 2016-09-30T10:05:49.000-07:00
Before checkpoint, we'd be better drop any inodes.

Signed-off-by: Jaegeuk Kim &lt;jaegeuk@kernel.org&gt;
diff --git a/fs/f2fs/recovery.c b/fs/f2fs/recovery.c
@@ -630,14 +630,16 @@ int recover_fsync_data(struct f2fs_sb_info *sbi, bool check_only)
 		set_ckpt_flags(sbi->ckpt, CP_ERROR_FLAG);
 	mutex_unlock(&sbi->cp_mutex);
 
+	/* let's drop all the directory inodes for clean checkpoint */
+	destroy_fsync_dnodes(&dir_list);
+
 	if (!err && need_writecp) {
 		struct cp_control cpc = {
 			.reason = CP_RECOVERY,
 		};
 		err = write_checkpoint(sbi, &cpc);
 	}
 
-	destroy_fsync_dnodes(&dir_list);
 	kmem_cache_destroy(fsync_entry_slab);
 	return ret ? ret: err;
 }
