diff options
| author | Chao Yu <yuchao0@huawei.com> | 2017-10-09 11:55:19 +0200 |
|---|---|---|
| committer | Jaegeuk Kim <jaegeuk@kernel.org> | 2017-10-26 10:44:12 +0200 |
| commit | a0d00fad353d4a315f54fb345aa1c65b3771389b (patch) | |
| tree | 65f4c30f4ca73234c64a42ff9a88e02ad0eb45fa /fs/f2fs/data.c | |
| parent | f2fs: Fix bool initialization/comparison (diff) | |
| download | linux-a0d00fad353d4a315f54fb345aa1c65b3771389b.tar.xz linux-a0d00fad353d4a315f54fb345aa1c65b3771389b.zip | |
f2fs: fix to avoid race when accessing last_disk_size
last_disk_size could be wrong due to concurrently updating, so using
i_sem semaphore to make last_disk_size updating exclusive to fix this
issue.
Signed-off-by: Chao Yu <yuchao0@huawei.com>
Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
Diffstat (limited to 'fs/f2fs/data.c')
| -rw-r--r-- | fs/f2fs/data.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/fs/f2fs/data.c b/fs/f2fs/data.c index 60a07fb26475..95fdbe3e6cca 100644 --- a/fs/f2fs/data.c +++ b/fs/f2fs/data.c @@ -1570,8 +1570,11 @@ write: err = do_write_data_page(&fio); } } + + down_write(&F2FS_I(inode)->i_sem); if (F2FS_I(inode)->last_disk_size < psize) F2FS_I(inode)->last_disk_size = psize; + up_write(&F2FS_I(inode)->i_sem); done: if (err && err != -ENOENT) |