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

2023-06-25 Thread Chao Yu
Compression option in inode should not be changed after they have been used, however, it may happen in below race case: Thread AThread B - f2fs_ioc_set_compress_option - check f2fs_is_mmap_file() - check get_dirty_pages() - check F2FS_HAS_BLOCKS()

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

2023-06-19 Thread Chao Yu
On 2023/6/16 14:03, Eric Biggers wrote: Hi Chao, Eric, Thanks for the report, let me check this. Thanks, 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

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

2023-06-15 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 f2fs_is_mmap

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

2023-06-12 Thread patchwork-bot+f2fs
Hello: This patch was applied to jaegeuk/f2fs.git (dev) by Jaegeuk Kim : On Mon, 29 May 2023 18:47:09 +0800 you 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 >

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

2023-06-05 Thread Chao Yu
Jaegeuk, Any comments on this patch? On 2023/5/29 18:47, 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 AThread B - f2fs_ioc_set_compress_option - check f2fs_is_mmap

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

2023-05-29 Thread Chao Yu
Compression option in inode should not be changed after they have been used, however, it may happen in below race case: Thread AThread B - f2fs_ioc_set_compress_option - check f2fs_is_mmap_file() - check get_dirty_pages() - check F2FS_HAS_BLOCKS()