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, 28 Jun 2018 22:49:24 +0000 (15:49 -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 a367e80640607fc4fe05bc17d8ec54cca23ae54b..1e489d10900435c79dabad94f5e8fabfc815fc7e 100644 (file)
@@ -1380,11 +1380,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;