Re: [f2fs-dev] [PATCH v8] fsck.f2fs: Detect and fix looped node chain efficiently

2023-06-16 Thread Chao Yu
On 2023/6/14 17:27, Chunhai Guo wrote: Hi Jaegeuk, Could you please help to confirm if this patch has been merged? I cannot see the patch in the dev-test or dev branch. Thanks. On 2023/5/24 10:42, 郭纯海 wrote: find_fsync_inode() detect the looped node chain by comparing the loop counter with

Re: [f2fs-dev] [PATCH][next] f2fs: remove redundant assignment to variable err

2023-06-16 Thread patchwork-bot+f2fs
Hello: This patch was applied to jaegeuk/f2fs.git (dev) by Jaegeuk Kim : On Fri, 16 Jun 2023 15:20:09 +0100 you wrote: > The assignment to variable err is redundant since the code jumps to > label next and err is then re-assigned a new value on the call to > sanity_check_node_chain. Remove the

[f2fs-dev] Patchwork summary for: f2fs

2023-06-16 Thread patchwork-bot+f2fs
Hello: The following patches were marked "accepted", because they were applied to jaegeuk/f2fs.git (dev): Patch: [f2fs-dev,next] f2fs: remove redundant assignment to variable err Submitter: Colin Ian King Committer: Jaegeuk Kim Patchwork:

[f2fs-dev] [PATCH][next] f2fs: remove redundant assignment to variable err

2023-06-16 Thread Colin Ian King
The assignment to variable err is redundant since the code jumps to label next and err is then re-assigned a new value on the call to sanity_check_node_chain. Remove the assignment. Cleans up clang scan build warning: fs/f2fs/recovery.c:464:6: warning: Value stored to 'err' is never read

Re: [f2fs-dev] [PATCH v2] f2fs: fix to avoid mmap vs set_compress_option case

2023-06-16 Thread Eric Biggers
Hi Chao, On Mon, May 29, 2023 at 06:47:09PM +0800, Chao Yu wrote: > Compression option in inode should not be changed after they have > been used, however, it may happen in below race case: > > Thread A Thread B > - f2fs_ioc_set_compress_option > - check