Re: [f2fs-dev] [PATCH v3] f2fs: zone: don't block IO if there is remained open zone

2024-04-14 Thread Chao Yu
On 2024/4/14 23:19, Jaegeuk Kim wrote: It seems this caused kernel hang. Chao, have you tested this patch enough? Jaegeuk, Oh, I've checked this patch w/ fsstress before submitting it, but missed the SPO testcase... do you encounter kernel hang w/ SPO testcase? Anyway, let me test it more.

Re: [f2fs-dev] [PATCH V3] f2fs: add REQ_TIME time update for some user behaviors

2024-04-14 Thread patchwork-bot+f2fs
Hello: This patch was applied to jaegeuk/f2fs.git (dev) by Jaegeuk Kim : On Wed, 20 Mar 2024 14:22:16 +0800 you wrote: > some user behaviors requested filesystem operations, which > will cause filesystem not idle. > Meanwhile adjust some f2fs_update_time(REQ_TIME) positions. > > Signed-off-by:

Re: [f2fs-dev] [PATCH] f2fs: fix to detect inconsistent nat entry during truncation

2024-04-14 Thread patchwork-bot+f2fs
Hello: This patch was applied to jaegeuk/f2fs.git (dev) by Jaegeuk Kim : On Fri, 22 Mar 2024 22:59:55 +0800 you wrote: > As Roman Smirnov reported as below: > > " > There is a possible bug in f2fs_truncate_inode_blocks(): > > if (err < 0 && err != -ENOENT) > goto

Re: [f2fs-dev] [PATCH 1/2] f2fs: introduce map_is_mergeable() for cleanup

2024-04-14 Thread patchwork-bot+f2fs
Hello: This series was applied to jaegeuk/f2fs.git (dev) by Jaegeuk Kim : On Mon, 25 Mar 2024 23:27:25 +0800 you wrote: > No logic changes. > > Signed-off-by: Chao Yu > --- > fs/f2fs/data.c | 24 ++-- > 1 file changed, 18 insertions(+), 6 deletions(-) Here is the summary

Re: [f2fs-dev] [PATCH] f2fs: Prevent s_writer rw_sem count mismatch in f2fs_evict_inode

2024-04-14 Thread patchwork-bot+f2fs
Hello: This patch was applied to jaegeuk/f2fs.git (dev) by Jaegeuk Kim : On Fri, 22 Mar 2024 13:16:39 +0900 you wrote: > If f2fs_evict_inode is called between freeze_super and thaw_super, the > s_writer rwsem count may become negative, resulting in hang. > > CPU1 CPU2 > >

Re: [f2fs-dev] [PATCH 1/3] f2fs: use folio_test_writeback

2024-04-14 Thread patchwork-bot+f2fs
Hello: This series was applied to jaegeuk/f2fs.git (dev) by Jaegeuk Kim : On Tue, 9 Apr 2024 20:34:09 + you wrote: > Let's convert PageWriteback to folio_test_writeback. > > Signed-off-by: Jaegeuk Kim > --- > fs/f2fs/compress.c | 2 +- > fs/f2fs/data.c | 3 +-- > fs/f2fs/f2fs.h

Re: [f2fs-dev] [PATCH] f2fs: support .shutdown in f2fs_sops

2024-04-14 Thread patchwork-bot+f2fs
Hello: This patch was applied to jaegeuk/f2fs.git (dev) by Jaegeuk Kim : On Thu, 29 Feb 2024 22:38:38 +0800 you wrote: > Support .shutdown callback in f2fs_sops, then, it can be called to > shut down the file system when underlying block device is marked dead. > > Signed-off-by: Chao Yu > ---

Re: [f2fs-dev] [PATCH 1/2] f2fs: fix to relocate check condition in f2fs_fallocate()

2024-04-14 Thread patchwork-bot+f2fs
Hello: This series was applied to jaegeuk/f2fs.git (dev) by Jaegeuk Kim : On Wed, 3 Apr 2024 22:24:19 +0800 you wrote: > compress and pinfile flag should be checked after inode lock held to > avoid race condition, fix it. > > Fixes: 4c8ff7095bef ("f2fs: support data compression") > Fixes:

Re: [f2fs-dev] [PATCH 1/2] f2fs: compress: fix to relocate check condition in f2fs_{release, reserve}_compress_blocks()

2024-04-14 Thread patchwork-bot+f2fs
Hello: This series was applied to jaegeuk/f2fs.git (dev) by Jaegeuk Kim : On Sun, 7 Apr 2024 15:26:03 +0800 you wrote: > Compress flag should be checked after inode lock held to avoid > racing w/ f2fs_setflags_common(), fix it. > > Fixes: 4c8ff7095bef ("f2fs: support data compression") >

Re: [f2fs-dev] [PATCH] f2fs: fix to wait on page writeback in __clone_blkaddrs()

2024-04-14 Thread patchwork-bot+f2fs
Hello: This patch was applied to jaegeuk/f2fs.git (dev) by Jaegeuk Kim : On Tue, 26 Mar 2024 19:28:45 +0800 you wrote: > In below race condition, dst page may become writeback status > in __clone_blkaddrs(), it needs to wait writeback before update, > fix it. > > Thread A

Re: [f2fs-dev] [PATCH 1/2] f2fs: prevent writing without fallocate() for pinned files

2024-04-14 Thread patchwork-bot+f2fs
Hello: This patch was applied to jaegeuk/f2fs.git (dev) by Jaegeuk Kim : On Thu, 11 Apr 2024 10:54:10 -0700 you wrote: > From: Daeho Jeong > > In a case writing without fallocate(), we can't guarantee it's allocated > in the conventional area for zoned stroage. To make it consistent across >

Re: [f2fs-dev] [PATCH] f2fs: remove clear SB_INLINECRYPT flag in default_options

2024-04-14 Thread patchwork-bot+f2fs
Hello: This patch was applied to jaegeuk/f2fs.git (dev) by Jaegeuk Kim : On Tue, 26 Mar 2024 14:10:43 +0800 you wrote: > In f2fs_remount, SB_INLINECRYPT flag will be clear and re-set. > If create new file or open file during this gap, these files > will not use inlinecrypt. Worse case, it may

Re: [f2fs-dev] [PATCH 2/2] f2fs: allow direct io of pinned files for zoned storage

2024-04-14 Thread patchwork-bot+f2fs
Hello: This patch was applied to jaegeuk/f2fs.git (dev) by Jaegeuk Kim : On Thu, 11 Apr 2024 11:37:53 -0700 you wrote: > From: Daeho Jeong > > Since the allocation happens in conventional LU for zoned storage, we > can allow direct io for that. > > Signed-off-by: Daeho Jeong > > [...] Here

Re: [f2fs-dev] [PATCH v2] f2fs: multidev: fix to recognize valid zero block address

2024-04-14 Thread patchwork-bot+f2fs
Hello: This patch was applied to jaegeuk/f2fs.git (dev) by Jaegeuk Kim : On Wed, 27 Mar 2024 15:42:23 +0800 you wrote: > As reported by Yi Zhang in mailing list [1], kernel warning was catched > during zbd/010 test as below: > > ./check zbd/010 > zbd/010 (test gap zone support with F2FS)

Re: [f2fs-dev] [PATCH] f2fs: write missing last sum blk of file pinning section

2024-04-14 Thread patchwork-bot+f2fs
Hello: This patch was applied to jaegeuk/f2fs.git (dev) by Jaegeuk Kim : On Tue, 9 Apr 2024 16:34:11 -0700 you wrote: > From: Daeho Jeong > > While do not allocating a new section in advance for file pinning area, I > missed that we should write the sum block for the last segment of a file >

[f2fs-dev] Patchwork summary for: f2fs

2024-04-14 Thread patchwork-bot+f2fs
Hello: The following patches were marked "accepted", because they were applied to jaegeuk/f2fs.git (dev): Patch: [f2fs-dev] f2fs: fix to wait on page writeback in __clone_blkaddrs() Submitter: Chao Yu Committer: Jaegeuk Kim Patchwork:

Re: [f2fs-dev] [PATCH v7] f2fs: fix zoned block device information initialization

2024-04-14 Thread patchwork-bot+f2fs
Hello: This patch was applied to jaegeuk/f2fs.git (dev) by Jaegeuk Kim : On Sun, 7 Apr 2024 15:21:23 +0800 you wrote: > From: Wenjie Qi > > If the max open zones of zoned devices are less than > the active logs of F2FS, the device may error due to > insufficient zone resources when multiple

Re: [f2fs-dev] [PATCH v3] f2fs: zone: don't block IO if there is remained open zone

2024-04-14 Thread Jaegeuk Kim
It seems this caused kernel hang. Chao, have you tested this patch enough? On 04/13, Chao Yu wrote: > On 2024/4/13 5:11, Jaegeuk Kim wrote: > > On 04/07, Chao Yu wrote: > > > max open zone may be larger than log header number of f2fs, for > > > such case, it doesn't need to wait last IO in