4.4-stable review patch.  If anyone has any objections, please let me know.

------------------

From: Jaegeuk Kim <jaeg...@kernel.org>

commit 9e1e6df412a28cdbbd2909de5c6189eda4a3383d upstream.

Before checkpoint, we'd be better drop any inodes.

Signed-off-by: Jaegeuk Kim <jaeg...@kernel.org>
Signed-off-by: Ben Hutchings <ben.hutchi...@codethink.co.uk>
Signed-off-by: Greg Kroah-Hartman <gre...@linuxfoundation.org>
---
 fs/f2fs/recovery.c |    4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

--- a/fs/f2fs/recovery.c
+++ b/fs/f2fs/recovery.c
@@ -597,6 +597,9 @@ out:
                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,
@@ -604,7 +607,6 @@ out:
                write_checkpoint(sbi, &cpc);
        }
 
-       destroy_fsync_dnodes(&dir_list);
        kmem_cache_destroy(fsync_entry_slab);
        return ret ? ret: err;
 }


Reply via email to