marko10_000
/
btrfs-progs
forked from Mirrors/btrfs-progs
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

btrfs-progs: Dont' stop scanning of devices at first failed device 

When 'btrfs device scan' command is invoked, it scans all devices,
check them for btrfs superblock and add devices with btrfs to a list.

Next, each device from the list is passed to kernel where it is handled
in the btrfs_scan_one_device() function. This function can, for example,
return -EBUSY when device contains superblock matched to existing and
mounted filesystem (if this device was pulled out from RAID and
connected again after some time).

btrfs tool stops device scan if any device has been failed to add, so
other existing devices with (possibly) valid FS will never be reached.

Fix this by remove stopping at any failure in the btrfs_register_all_devices(),
just return error count. btrfs_scan_one_device() reports any kind of
error already.

Signed-off-by: Yauhen Kharuzhy <yauhen.kharuzhy@zavadatar.com>
[ initialize err to 0 ]
Signed-off-by: David Sterba <dsterba@suse.com>
master
Yauhen Kharuzhy 2016-03-10 16:04:35 -08:00 committed by David Sterba
parent 83ad1dcbb0
commit 3ee935e4dd
2 changed files with 9 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
cmds-device.c
View File

@ -254,7 +254,7 @@ static int cmd_device_scan(int argc, char **argv)
ret = btrfs_scan_lblkid();
error_on(ret, "error %d while scanning", ret);
ret = btrfs_register_all_devices();
error_on(ret, "error %d while registering devices", ret);
error_on(ret, "there are %d errors while registering devices", ret);
goto out;
}

16
utils.c
View File

@ -1545,7 +1545,8 @@ int btrfs_register_one_device(const char *fname)
*/
int btrfs_register_all_devices(void)
{
int err;
int err = 0;
int ret = 0;
struct btrfs_fs_devices *fs_devices;
struct btrfs_device *device;
struct list_head *all_uuids;
@ -1554,16 +1555,15 @@ int btrfs_register_all_devices(void)
list_for_each_entry(fs_devices, all_uuids, list) {
list_for_each_entry(device, &fs_devices->devices, dev_list) {
if (*device->name) {
if (*device->name)
err = btrfs_register_one_device(device->name);
if (err < 0)
return err;
if (err > 0)
return -err;
}
if (err)
ret++;
}
}
return 0;
return ret;
}
int btrfs_device_already_in_root(struct btrfs_root *root, int fd,