block/loop: Don't grab "struct file" for vfs_getattr() operation.
authorTetsuo Handa <penguin-kernel@I-love.SAKURA.ne.jp>
Thu, 8 Nov 2018 13:01:01 +0000 (14:01 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 23 Jan 2019 07:09:51 +0000 (08:09 +0100)
commit b1ab5fa309e6c49e4e06270ec67dd7b3e9971d04 upstream.

vfs_getattr() needs "struct path" rather than "struct file".
Let's use path_get()/path_put() rather than get_file()/fput().

Signed-off-by: Tetsuo Handa <penguin-kernel@I-love.SAKURA.ne.jp>
Reviewed-by: Jan Kara <jack@suse.cz>
Signed-off-by: Jan Kara <jack@suse.cz>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/block/loop.c

index 6d61633a7f899f0bc2d98eaffc018c46d954205e..58164bc29da23d45bc82021bf479ddaa9049e975 100644 (file)
@@ -1175,7 +1175,7 @@ loop_set_status(struct loop_device *lo, const struct loop_info64 *info)
 static int
 loop_get_status(struct loop_device *lo, struct loop_info64 *info)
 {
-       struct file *file;
+       struct path path;
        struct kstat stat;
        int ret;
 
@@ -1200,16 +1200,16 @@ loop_get_status(struct loop_device *lo, struct loop_info64 *info)
        }
 
        /* Drop lo_ctl_mutex while we call into the filesystem. */
-       file = get_file(lo->lo_backing_file);
+       path = lo->lo_backing_file->f_path;
+       path_get(&path);
        mutex_unlock(&lo->lo_ctl_mutex);
-       ret = vfs_getattr(&file->f_path, &stat, STATX_INO,
-                         AT_STATX_SYNC_AS_STAT);
+       ret = vfs_getattr(&path, &stat, STATX_INO, AT_STATX_SYNC_AS_STAT);
        if (!ret) {
                info->lo_device = huge_encode_dev(stat.dev);
                info->lo_inode = stat.ino;
                info->lo_rdevice = huge_encode_dev(stat.rdev);
        }
-       fput(file);
+       path_put(&path);
        return ret;
 }