Re: [f2fs-dev] [PATCH v2] f2fs: change i_compr_blocks of inode to atomic value

2020-09-03 Thread Chao Yu
On 2020/9/4 1:48, Jaegeuk Kim wrote: On 09/03, Daeho Jeong wrote: From: Daeho Jeong writepages() can be concurrently invoked for the same file by different threads such as a thread fsyncing the file and a kworker kernel thread. So, changing i_compr_blocks without protection is racy and we

Re: [f2fs-dev] [PATCH v2] f2fs: change i_compr_blocks of inode to atomic value

2020-09-03 Thread Jaegeuk Kim
On 09/03, Daeho Jeong wrote: > From: Daeho Jeong > > writepages() can be concurrently invoked for the same file by different > threads such as a thread fsyncing the file and a kworker kernel thread. > So, changing i_compr_blocks without protection is racy and we need to > protect it by changing

[PATCH v2] f2fs: change i_compr_blocks of inode to atomic value

2020-09-02 Thread Daeho Jeong
From: Daeho Jeong writepages() can be concurrently invoked for the same file by different threads such as a thread fsyncing the file and a kworker kernel thread. So, changing i_compr_blocks without protection is racy and we need to protect it by changing it with atomic type value. Plus, we don't