f2fs: fix to update largest extent under lock
commit b691d98fdd4cc2514c60fd6975e6016da203e64f upstream. In order to avoid racing problem, make largest extent cache being updated under lock. Signed-off-by: Chao Yu <yuchao0@huawei.com> Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
This commit is contained in:
parent
9e3d0bf6d3
commit
1158df42b2
1 changed files with 7 additions and 4 deletions
|
@ -252,6 +252,7 @@ retry:
|
||||||
int update_inode(struct inode *inode, struct page *node_page)
|
int update_inode(struct inode *inode, struct page *node_page)
|
||||||
{
|
{
|
||||||
struct f2fs_inode *ri;
|
struct f2fs_inode *ri;
|
||||||
|
struct extent_tree *et = F2FS_I(inode)->extent_tree;
|
||||||
|
|
||||||
f2fs_inode_synced(inode);
|
f2fs_inode_synced(inode);
|
||||||
|
|
||||||
|
@ -267,11 +268,13 @@ int update_inode(struct inode *inode, struct page *node_page)
|
||||||
ri->i_size = cpu_to_le64(i_size_read(inode));
|
ri->i_size = cpu_to_le64(i_size_read(inode));
|
||||||
ri->i_blocks = cpu_to_le64(inode->i_blocks);
|
ri->i_blocks = cpu_to_le64(inode->i_blocks);
|
||||||
|
|
||||||
if (F2FS_I(inode)->extent_tree)
|
if (et) {
|
||||||
set_raw_extent(&F2FS_I(inode)->extent_tree->largest,
|
read_lock(&et->lock);
|
||||||
&ri->i_ext);
|
set_raw_extent(&et->largest, &ri->i_ext);
|
||||||
else
|
read_unlock(&et->lock);
|
||||||
|
} else {
|
||||||
memset(&ri->i_ext, 0, sizeof(ri->i_ext));
|
memset(&ri->i_ext, 0, sizeof(ri->i_ext));
|
||||||
|
}
|
||||||
set_raw_inline(inode, ri);
|
set_raw_inline(inode, ri);
|
||||||
|
|
||||||
ri->i_atime = cpu_to_le64(inode->i_atime.tv_sec);
|
ri->i_atime = cpu_to_le64(inode->i_atime.tv_sec);
|
||||||
|
|
Loading…
Add table
Reference in a new issue