[f2fs-dev] [PATCH] Revert "f2fs: use flush command instead of FUA for zoned device"

2024-06-13 Thread Wenjie Cheng
This reverts commit c550e25bca660ed2554cbb48d32b82d0bb98e4b1. Commit c550e25bca660ed2554cbb48d32b82d0bb98e4b1 ("f2fs: use flush command instead of FUA for zoned device") used additional flush command to keep write order. Since Commit dd291d77cc90eb6a86e9860ba8e6e38eebd57d12 ("block: Introduce zon

Re: [f2fs-dev] [PATCH v4 03/22] xfs: Use extent size granularity for iomap->io_block_size

2024-06-13 Thread John Garry via Linux-f2fs-devel
On 12/06/2024 22:47, Darrick J. Wong wrote: On Fri, Jun 07, 2024 at 02:39:00PM +, John Garry wrote: Currently iomap->io_block_size is set to the i_blocksize() value for the inode. Expand the sub-fs block size zeroing to now cover RT extents, by calling setting iomap->io_block_size as xfs_in

Re: [f2fs-dev] [PATCH v4 02/22] iomap: Allow filesystems set IO block zeroing size

2024-06-13 Thread John Garry via Linux-f2fs-devel
On 12/06/2024 22:32, Darrick J. Wong wrote: unsigned int fs_block_size = i_blocksize(inode), pad; + u64 io_block_size = iomap->io_block_size; I wonder, should iomap be nice and not require filesystems to set io_block_size themselves unless they really need it? That's what I had in v3, l

[f2fs-dev] [PATCH] f2fs: fix to use mnt_{want, drop}_write_file replace file_{start, end}_wrtie

2024-06-13 Thread Zhiguo Niu
mnt_{want,drop}_write_file is more suitable than file_{start,end}_wrtie and also is consistent with other ioctl operations. Signed-off-by: Zhiguo Niu --- fs/f2fs/file.c | 24 1 file changed, 16 insertions(+), 8 deletions(-) diff --git a/fs/f2fs/file.c b/fs/f2fs/file.c i

Re: [f2fs-dev] [PATCH v4 01/22] fs: Add generic_atomic_write_valid_size()

2024-06-13 Thread John Garry via Linux-f2fs-devel
On 12/06/2024 22:10, Darrick J. Wong wrote: On Fri, Jun 07, 2024 at 02:38:58PM +, John Garry wrote: Add a generic helper for FSes to validate that an atomic write is appropriately sized (along with the other checks). Signed-off-by: John Garry --- include/linux/fs.h | 12 1