diff options
| author | Nikolay Borisov <nborisov@suse.com> | 2017-06-27 10:02:24 +0300 |
|---|---|---|
| committer | David Sterba <dsterba@suse.com> | 2017-08-16 14:19:52 +0200 |
| commit | ebcc9301eacb99350ef7e1e2492e5d18f7986ef8 (patch) | |
| tree | 0c2fd2319da6f7a36d5ba3e0c7ca5d62aac3f94f | |
| parent | ef954844c7ace62f773f4f23e28d2d915adc419f (diff) | |
| download | linux-ebcc9301eacb99350ef7e1e2492e5d18f7986ef8.tar.gz linux-ebcc9301eacb99350ef7e1e2492e5d18f7986ef8.tar.bz2 linux-ebcc9301eacb99350ef7e1e2492e5d18f7986ef8.zip | |
btrfs: convert while loop to list_for_each_entry
No functional changes, just make the loop a bit more readable
Signed-off-by: Nikolay Borisov <nborisov@suse.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
| -rw-r--r-- | fs/btrfs/volumes.c | 11 |
1 files changed, 2 insertions, 9 deletions
diff --git a/fs/btrfs/volumes.c b/fs/btrfs/volumes.c index e8b9a269fdde..c665af0727e6 100644 --- a/fs/btrfs/volumes.c +++ b/fs/btrfs/volumes.c @@ -4629,7 +4629,7 @@ static int __btrfs_alloc_chunk(struct btrfs_trans_handle *trans, { struct btrfs_fs_info *info = trans->fs_info; struct btrfs_fs_devices *fs_devices = info->fs_devices; - struct list_head *cur; + struct btrfs_device *device; struct map_lookup *map = NULL; struct extent_map_tree *em_tree; struct extent_map *em; @@ -4703,22 +4703,15 @@ static int __btrfs_alloc_chunk(struct btrfs_trans_handle *trans, if (!devices_info) return -ENOMEM; - cur = fs_devices->alloc_list.next; - /* * in the first pass through the devices list, we gather information * about the available holes on each device. */ ndevs = 0; - while (cur != &fs_devices->alloc_list) { - struct btrfs_device *device; + list_for_each_entry(device, &fs_devices->alloc_list, dev_alloc_list) { u64 max_avail; u64 dev_offset; - device = list_entry(cur, struct btrfs_device, dev_alloc_list); - - cur = cur->next; - if (!device->writeable) { WARN(1, KERN_ERR "BTRFS: read-only device in alloc_list\n"); |