f2fs: use flush command instead of FUA for zoned device
authorJaegeuk Kim <jaegeuk@kernel.org>
Mon, 18 Apr 2022 23:57:44 +0000 (16:57 -0700)
committerJaegeuk Kim <jaegeuk@kernel.org>
Fri, 6 May 2022 17:18:11 +0000 (10:18 -0700)
The block layer for zoned disk can reorder the FUA'ed IOs. Let's use flush
command to keep the write order.

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

index f08e6208e183382e4abac62691c29d0a0bc8fa00..eae2e79080724f1b158ab8798ca9971279ea63ba 100644 (file)
@@ -372,7 +372,8 @@ sync_nodes:
        f2fs_remove_ino_entry(sbi, ino, APPEND_INO);
        clear_inode_flag(inode, FI_APPEND_WRITE);
 flush_out:
-       if (!atomic && F2FS_OPTION(sbi).fsync_mode != FSYNC_MODE_NOBARRIER)
+       if ((!atomic && F2FS_OPTION(sbi).fsync_mode != FSYNC_MODE_NOBARRIER) ||
+           (atomic && !test_opt(sbi, NOBARRIER) && f2fs_sb_has_blkzoned(sbi)))
                ret = f2fs_issue_flush(sbi, inode->i_ino);
        if (!ret) {
                f2fs_remove_ino_entry(sbi, ino, UPDATE_INO);
index c45d341dcf6e536fd1f5458621dcdb61cfa9f592..144f9f9666904e9f2aa0d3dba215100632eceba3 100644 (file)
@@ -1631,7 +1631,7 @@ static int __write_node_page(struct page *page, bool atomic, bool *submitted,
                goto redirty_out;
        }
 
-       if (atomic && !test_opt(sbi, NOBARRIER))
+       if (atomic && !test_opt(sbi, NOBARRIER) && !f2fs_sb_has_blkzoned(sbi))
                fio.op_flags |= REQ_PREFLUSH | REQ_FUA;
 
        /* should add to global list before clearing PAGECACHE status */