f2fs: fix to force keeping write barrier for strict fsync mode
authorChao Yu <chao@kernel.org>
Tue, 20 Jul 2021 01:03:29 +0000 (09:03 +0800)
committerJaegeuk Kim <jaegeuk@kernel.org>
Mon, 2 Aug 2021 18:24:26 +0000 (11:24 -0700)
commit2787991516468bfafafb9bf2b45a848e6b202e7c
tree6da9bc0931b9c7c944bb910481568bcdc43b1e51
parent277afbde6ca2b38729683fc17c031b4bc942068d
f2fs: fix to force keeping write barrier for strict fsync mode

[1] https://www.mail-archive.com/linux-f2fs-devel@lists.sourceforge.net/msg15126.html

As [1] reported, if lower device doesn't support write barrier, in below
case:

- write page #0; persist
- overwrite page #0
- fsync
 - write data page #0 OPU into device's cache
 - write inode page into device's cache
 - issue flush

If SPO is triggered during flush command, inode page can be persisted
before data page #0, so that after recovery, inode page can be recovered
with new physical block address of data page #0, however there may
contains dummy data in new physical block address.

Then what user will see is: after overwrite & fsync + SPO, old data in
file was corrupted, if any user do care about such case, we can suggest
user to use STRICT fsync mode, in this mode, we will force to use atomic
write sematics to keep write order in between data/node and last node,
so that it avoids potential data corruption during fsync().

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