Hello:

This patch was applied to jaegeuk/f2fs.git (dev)
by Jaegeuk Kim <jaeg...@kernel.org>:

On Thu,  6 Jul 2023 10:06:14 +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
> - f2fs_ioc_set_compress_option
>  - check f2fs_is_mmap_file()
>  - check get_dirty_pages()
>  - check F2FS_HAS_BLOCKS()
>                                       - f2fs_file_mmap
>                                        - set_inode_flag(FI_MMAP_FILE)
>                                       - fault
>                                        - do_page_mkwrite
>                                         - f2fs_vm_page_mkwrite
>                                         - f2fs_get_block_locked
>                                        - fault_dirty_shared_page
>                                         - set_page_dirty
>  - update i_compress_algorithm
>  - update i_log_cluster_size
>  - update i_cluster_size
> 
> [...]

Here is the summary with links:
  - [f2fs-dev,v3] f2fs: fix to avoid mmap vs set_compress_option case
    https://git.kernel.org/jaegeuk/f2fs/c/3174a6dbb945

You are awesome, thank you!
-- 
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html




_______________________________________________
Linux-f2fs-devel mailing list
Linux-f2fs-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel

Reply via email to