mirror of
https://github.com/kdave/btrfs-progs
synced 2025-01-11 16:29:42 +00:00
btrfs-progs: use canonical name for device in btrfs fi show when mounted
When using lvm volumes to check fstests: btrfs/006, it fails like: Label: 'TestLabel.006' uuid: <UUID> Total devices <EXACTNUM> FS bytes used <SIZE> devid <DEVID> size <SIZE> used <SIZE> path SCRATCH_DEV + devid <DEVID> size <SIZE> used <SIZE> path /dev/dm-4 + devid <DEVID> size <SIZE> used <SIZE> path /dev/dm-5 + devid <DEVID> size <SIZE> used <SIZE> path /dev/dm-6 The /dev/dm-* points to lvm volumes, use @canonicalize_path() to convert them and we will make it through. Of course we should do the same thing for dev stat. Signed-off-by: Gui Hecheng <guihc.fnst@cn.fujitsu.com> Signed-off-by: David Sterba <dsterba@suse.cz>
This commit is contained in:
parent
0c44c195fe
commit
a1c3bcf1c2
@ -408,31 +408,37 @@ static int cmd_dev_stats(int argc, char **argv)
|
||||
path, strerror(errno));
|
||||
err = 1;
|
||||
} else {
|
||||
char *canonical_path;
|
||||
|
||||
canonical_path = canonicalize_path((char *)path);
|
||||
|
||||
if (args.nr_items >= BTRFS_DEV_STAT_WRITE_ERRS + 1)
|
||||
printf("[%s].write_io_errs %llu\n",
|
||||
path,
|
||||
canonical_path,
|
||||
(unsigned long long) args.values[
|
||||
BTRFS_DEV_STAT_WRITE_ERRS]);
|
||||
if (args.nr_items >= BTRFS_DEV_STAT_READ_ERRS + 1)
|
||||
printf("[%s].read_io_errs %llu\n",
|
||||
path,
|
||||
canonical_path,
|
||||
(unsigned long long) args.values[
|
||||
BTRFS_DEV_STAT_READ_ERRS]);
|
||||
if (args.nr_items >= BTRFS_DEV_STAT_FLUSH_ERRS + 1)
|
||||
printf("[%s].flush_io_errs %llu\n",
|
||||
path,
|
||||
canonical_path,
|
||||
(unsigned long long) args.values[
|
||||
BTRFS_DEV_STAT_FLUSH_ERRS]);
|
||||
if (args.nr_items >= BTRFS_DEV_STAT_CORRUPTION_ERRS + 1)
|
||||
printf("[%s].corruption_errs %llu\n",
|
||||
path,
|
||||
canonical_path,
|
||||
(unsigned long long) args.values[
|
||||
BTRFS_DEV_STAT_CORRUPTION_ERRS]);
|
||||
if (args.nr_items >= BTRFS_DEV_STAT_GENERATION_ERRS + 1)
|
||||
printf("[%s].generation_errs %llu\n",
|
||||
path,
|
||||
canonical_path,
|
||||
(unsigned long long) args.values[
|
||||
BTRFS_DEV_STAT_GENERATION_ERRS]);
|
||||
|
||||
free(canonical_path);
|
||||
}
|
||||
}
|
||||
|
||||
|
@ -510,7 +510,10 @@ static int print_one_fs(struct btrfs_ioctl_fs_info_args *fs_info,
|
||||
pretty_size(calc_used_bytes(space_info)));
|
||||
|
||||
for (i = 0; i < fs_info->num_devices; i++) {
|
||||
char *canonical_path;
|
||||
|
||||
tmp_dev_info = (struct btrfs_ioctl_dev_info_args *)&dev_info[i];
|
||||
canonical_path = canonicalize_path((char *)tmp_dev_info->path);
|
||||
|
||||
/* Add check for missing devices even mounted */
|
||||
fd = open((char *)tmp_dev_info->path, O_RDONLY);
|
||||
@ -523,7 +526,9 @@ static int print_one_fs(struct btrfs_ioctl_fs_info_args *fs_info,
|
||||
tmp_dev_info->devid,
|
||||
pretty_size(tmp_dev_info->total_bytes),
|
||||
pretty_size(tmp_dev_info->bytes_used),
|
||||
tmp_dev_info->path);
|
||||
canonical_path);
|
||||
|
||||
free(canonical_path);
|
||||
}
|
||||
|
||||
if (missing)
|
||||
|
Loading…
Reference in New Issue
Block a user