diff options
| author |   Nikolay Borisov <nborisov@suse.com> | 2020-07-16 10:25:33 +0300 |
|---|---|---|
| committer |   David Sterba <dsterba@suse.com> | 2020-10-07 12:06:58 +0200 |
| commit | 944d3f9fac61e24e13a056b25974df3831994f29 (patch) | |
| tree | 4c0d57b0ee08af868fa53ef46977a4d24919e2fe /fs/btrfs/reada.c | |
| parent | btrfs: simplify setting/clearing fs_info to btrfs_fs_devices (diff) | |
| download | linux-dev-944d3f9fac61e24e13a056b25974df3831994f29.tar.xz linux-dev-944d3f9fac61e24e13a056b25974df3831994f29.zip | |
btrfs: switch seed device to list api
While this patch touches a bunch of files the conversion is
straighforward. Instead of using the implicit linked list anchored at
btrfs_fs_devices::seed the code is switched to using
list_for_each_entry.
Previous patches in the series already factored out code that processed
both main and seed devices so in those cases the factored out functions
are called on the main fs_devices and then on every seed dev inside
list_for_each_entry.
Using list api also allows to simplify deletion from the seed dev list
performed in btrfs_rm_device and btrfs_rm_dev_replace_free_srcdev by
substituting a while() loop with a simple list_del_init.
Reviewed-by: Josef Bacik <josef@toxicpanda.com>
Reviewed-by: Anand Jain <anand.jain@oracle.com>
Signed-off-by: Nikolay Borisov <nborisov@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'fs/btrfs/reada.c')
| -rw-r--r-- | fs/btrfs/reada.c | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/fs/btrfs/reada.c b/fs/btrfs/reada.c index a6bfe7ec14cb..e20972230823 100644 --- a/fs/btrfs/reada.c +++ b/fs/btrfs/reada.c @@ -791,16 +791,13 @@ static int reada_start_for_fsdevs(struct btrfs_fs_devices *fs_devices) static void __reada_start_machine(struct btrfs_fs_info *fs_info) { - struct btrfs_fs_devices *fs_devices = fs_info->fs_devices; + struct btrfs_fs_devices *fs_devices = fs_info->fs_devices, *seed_devs; int i; u64 enqueued = 0; -again: enqueued += reada_start_for_fsdevs(fs_devices); - if (fs_devices->seed) { - fs_devices = fs_devices->seed; - goto again; - } + list_for_each_entry(seed_devs, &fs_devices->seed_list, seed_list) + enqueued += reada_start_for_fsdevs(seed_devs); if (enqueued == 0) return; |