From: Chao Yu <yuch...@huawei.com>

In order to avoid racing problem, make largest extent cache being updated
under lock.

Signed-off-by: Chao Yu <yuch...@huawei.com>
---
 fs/f2fs/inode.c | 11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)

diff --git a/fs/f2fs/inode.c b/fs/f2fs/inode.c
index 34ae03c..d9cd458 100644
--- a/fs/f2fs/inode.c
+++ b/fs/f2fs/inode.c
@@ -252,6 +252,7 @@ retry:
 int update_inode(struct inode *inode, struct page *node_page)
 {
        struct f2fs_inode *ri;
+       struct extent_tree *et = F2FS_I(inode)->extent_tree;
 
        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_blocks = cpu_to_le64(inode->i_blocks);
 
-       if (F2FS_I(inode)->extent_tree)
-               set_raw_extent(&F2FS_I(inode)->extent_tree->largest,
-                                                       &ri->i_ext);
-       else
+       if (et) {
+               read_lock(&et->lock);
+               set_raw_extent(&et->largest, &ri->i_ext);
+               read_unlock(&et->lock);
+       } else {
                memset(&ri->i_ext, 0, sizeof(ri->i_ext));
+       }
        set_raw_inline(inode, ri);
 
        ri->i_atime = cpu_to_le64(inode->i_atime.tv_sec);
-- 
2.10.1

Reply via email to