f2fs: don't drop any page on f2fs_cp_error() case
authorJaegeuk Kim <jaegeuk@kernel.org>
Sat, 5 May 2018 01:04:22 +0000 (18:04 -0700)
committerJaegeuk Kim <jaegeuk@kernel.org>
Thu, 31 May 2018 18:31:49 +0000 (11:31 -0700)
We still provide readdir() after shtudown, so we should keep pages to avoid
additional IOs.

Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
fs/f2fs/node.c

index 437a66ba56fd3fd6a5155ef3e126b96fcdbe461e..62ce49984a9e6979ae17b9fe0c83359f5db3c982 100644 (file)
@@ -1376,11 +1376,8 @@ static int __write_node_page(struct page *page, bool atomic, bool *submitted,
 
        trace_f2fs_writepage(page, NODE);
 
-       if (unlikely(f2fs_cp_error(sbi))) {
-               dec_page_count(sbi, F2FS_DIRTY_NODES);
-               unlock_page(page);
-               return 0;
-       }
+       if (unlikely(f2fs_cp_error(sbi)))
+               goto redirty_out;
 
        if (unlikely(is_sbi_flag_set(sbi, SBI_POR_DOING)))
                goto redirty_out;