f2fs: use shared inode lock during f2fs_fiemap()
authorChao Yu <chao@kernel.org>
Thu, 16 Nov 2023 06:25:55 +0000 (14:25 +0800)
committerJaegeuk Kim <jaegeuk@kernel.org>
Fri, 17 Nov 2023 17:31:18 +0000 (09:31 -0800)
f2fs_fiemap() will only traverse metadata of inode, let's use shared
inode lock for it to avoid unnecessary race on inode lock.

Signed-off-by: Chao Yu <chao@kernel.org>
Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
fs/f2fs/data.c

index 4e42b5f24debec2c1c940b034f622671adb4994c..42f0f6184f736405bd0ac3c95ed2cfd0a25d212e 100644 (file)
@@ -1992,7 +1992,7 @@ int f2fs_fiemap(struct inode *inode, struct fiemap_extent_info *fieinfo,
        if (ret)
                return ret;
 
-       inode_lock(inode);
+       inode_lock_shared(inode);
 
        maxbytes = max_file_blocks(inode) << F2FS_BLKSIZE_BITS;
        if (start > maxbytes) {
@@ -2112,7 +2112,7 @@ out:
        if (ret == 1)
                ret = 0;
 
-       inode_unlock(inode);
+       inode_unlock_shared(inode);
        return ret;
 }