f2fs: remove i_sem lock coverage in f2fs_setxattr()
authorChao Yu <yuchao0@huawei.com>
Thu, 27 Feb 2020 11:30:04 +0000 (19:30 +0800)
committerJaegeuk Kim <jaegeuk@kernel.org>
Tue, 10 Mar 2020 16:18:32 +0000 (09:18 -0700)
f2fs_inode.xattr_ver field was gone after commit d260081ccf37
("f2fs: change recovery policy of xattr node block"), remove i_sem
lock coverage in f2fs_setxattr()

Signed-off-by: Chao Yu <yuchao0@huawei.com>
Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
fs/f2fs/xattr.c

index 296b3189448a466c2177ace3d0423f1f31f2162b..6794a13064be38c42317c889301b443ab1911c23 100644 (file)
@@ -758,12 +758,9 @@ int f2fs_setxattr(struct inode *inode, int index, const char *name,
        f2fs_balance_fs(sbi, true);
 
        f2fs_lock_op(sbi);
-       /* protect xattr_ver */
-       down_write(&F2FS_I(inode)->i_sem);
        down_write(&F2FS_I(inode)->i_xattr_sem);
        err = __f2fs_setxattr(inode, index, name, value, size, ipage, flags);
        up_write(&F2FS_I(inode)->i_xattr_sem);
-       up_write(&F2FS_I(inode)->i_sem);
        f2fs_unlock_op(sbi);
 
        f2fs_update_time(sbi, REQ_TIME);