diff options
| author | Anand Jain | 2018-01-22 23:49:36 +0100 |
|---|---|---|
| committer | David Sterba | 2018-03-26 15:09:36 +0200 |
| commit | f8e10cd3f8dd3b15ea49f820a0d2decf3c654b13 (patch) | |
| tree | c81c02d063c231d74f167b994504ba7e0b331389 /fs/btrfs/volumes.c | |
| parent | Btrfs: do not check inode's runtime flags under root->orphan_lock (diff) | |
| download | kernel-qcow2-linux-f8e10cd3f8dd3b15ea49f820a0d2decf3c654b13.tar.gz kernel-qcow2-linux-f8e10cd3f8dd3b15ea49f820a0d2decf3c654b13.tar.xz kernel-qcow2-linux-f8e10cd3f8dd3b15ea49f820a0d2decf3c654b13.zip | |
btrfs: keep device list sorted
By maintaining the device list sorted lets us reproduce the problems
related to missing chunk in the degraded mode much more consistent. So
fix this by sorting the devices by devid within the kernel. So that we
know which device is assigned to the struct fs_info::latest_bdev when
all the devices are having and same SB generation.
Signed-off-by: Anand Jain <anand.jain@oracle.com>
Reviewed-by: Nikolay Borisov <nborisov@suse.com>
[ update changelog ]
Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'fs/btrfs/volumes.c')
| -rw-r--r-- | fs/btrfs/volumes.c | 16 |
1 files changed, 16 insertions, 0 deletions
diff --git a/fs/btrfs/volumes.c b/fs/btrfs/volumes.c index 77c7a0c18190..abc6555105bb 100644 --- a/fs/btrfs/volumes.c +++ b/fs/btrfs/volumes.c @@ -27,6 +27,7 @@ #include <linux/raid/pq.h> #include <linux/semaphore.h> #include <linux/uuid.h> +#include <linux/list_sort.h> #include <asm/div64.h> #include "ctree.h" #include "extent_map.h" @@ -1103,6 +1104,20 @@ out: return ret; } +static int devid_cmp(void *priv, struct list_head *a, struct list_head *b) +{ + struct btrfs_device *dev1, *dev2; + + dev1 = list_entry(a, struct btrfs_device, dev_list); + dev2 = list_entry(b, struct btrfs_device, dev_list); + + if (dev1->devid < dev2->devid) + return -1; + else if (dev1->devid > dev2->devid) + return 1; + return 0; +} + int btrfs_open_devices(struct btrfs_fs_devices *fs_devices, fmode_t flags, void *holder) { @@ -1113,6 +1128,7 @@ int btrfs_open_devices(struct btrfs_fs_devices *fs_devices, fs_devices->opened++; ret = 0; } else { + list_sort(NULL, &fs_devices->devices, devid_cmp); ret = __btrfs_open_devices(fs_devices, flags, holder); } mutex_unlock(&uuid_mutex); |