btrfs: drop unnecessary variable in btrfs_init_new_device
authorAnand Jain <anand.jain@oracle.com>
Tue, 3 Jul 2018 05:14:51 +0000 (13:14 +0800)
committerDavid Sterba <dsterba@suse.com>
Mon, 6 Aug 2018 11:12:42 +0000 (13:12 +0200)
There is only usage of the declared devices variable, instead use its
value directly.

Signed-off-by: Anand Jain <anand.jain@oracle.com>
Reviewed-by: Nikolay Borisov <nborisov@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
fs/btrfs/volumes.c

index adce1c2e9b7b1c98daca44eeb530daea26d36ba0..1ffc49c9c02fe16c3d7ad78899009043e9300bb8 100644 (file)
@@ -2407,7 +2407,6 @@ int btrfs_init_new_device(struct btrfs_fs_info *fs_info, const char *device_path
        struct btrfs_trans_handle *trans;
        struct btrfs_device *device;
        struct block_device *bdev;
        struct btrfs_trans_handle *trans;
        struct btrfs_device *device;
        struct block_device *bdev;
-       struct list_head *devices;
        struct super_block *sb = fs_info->sb;
        struct rcu_string *name;
        struct btrfs_fs_devices *fs_devices = fs_info->fs_devices;
        struct super_block *sb = fs_info->sb;
        struct rcu_string *name;
        struct btrfs_fs_devices *fs_devices = fs_info->fs_devices;
@@ -2432,10 +2431,8 @@ int btrfs_init_new_device(struct btrfs_fs_info *fs_info, const char *device_path
 
        filemap_write_and_wait(bdev->bd_inode->i_mapping);
 
 
        filemap_write_and_wait(bdev->bd_inode->i_mapping);
 
-       devices = &fs_devices->devices;
-
        mutex_lock(&fs_devices->device_list_mutex);
        mutex_lock(&fs_devices->device_list_mutex);
-       list_for_each_entry(device, devices, dev_list) {
+       list_for_each_entry(device, &fs_devices->devices, dev_list) {
                if (device->bdev == bdev) {
                        ret = -EEXIST;
                        mutex_unlock(
                if (device->bdev == bdev) {
                        ret = -EEXIST;
                        mutex_unlock(