On 6/10/13 9:56 AM, Anand Jain wrote:
> btrfs_scan_for_fsid uses only one argument run_ioctl out of 3
> so remove the rest two of them
and run_ioctl is only ever '1' (and it's completely unobvious
at the call point what '1' means).
Why not just go with 0 args?
Then only btrfs_scan_one_dir and btrfs_scan_block_devices
will have a non-obvious 0/1 arg.... :(
-Eric
>
> Signed-off-by: Anand Jain <anand.jain@xxxxxxxxxx>
> ---
> btrfs-find-root.c | 2 +-
> disk-io.c | 2 +-
> utils.c | 5 ++---
> utils.h | 3 +--
> 4 files changed, 5 insertions(+), 7 deletions(-)
>
> diff --git a/btrfs-find-root.c b/btrfs-find-root.c
> index 810d835..e736cb5 100644
> --- a/btrfs-find-root.c
> +++ b/btrfs-find-root.c
> @@ -110,7 +110,7 @@ static struct btrfs_root *open_ctree_broken(int fd, const char *device)
> }
>
> if (total_devs != 1) {
> - ret = btrfs_scan_for_fsid(fs_devices, total_devs, 1);
> + ret = btrfs_scan_for_fsid(1);
> if (ret)
> goto out;
> }
> diff --git a/disk-io.c b/disk-io.c
> index 9ffe6e4..acd5480 100644
> --- a/disk-io.c
> +++ b/disk-io.c
> @@ -838,7 +838,7 @@ static struct btrfs_fs_info *__open_ctree_fd(int fp, const char *path,
> }
>
> if (total_devs != 1) {
> - ret = btrfs_scan_for_fsid(fs_devices, total_devs, 1);
> + ret = btrfs_scan_for_fsid(1);
> if (ret)
> goto out;
> }
> diff --git a/utils.c b/utils.c
> index 7b4cd74..25f3cb4 100644
> --- a/utils.c
> +++ b/utils.c
> @@ -928,7 +928,7 @@ int check_mounted_where(int fd, const char *file, char *where, int size,
>
> /* scan other devices */
> if (is_btrfs && total_devs > 1) {
> - if((ret = btrfs_scan_for_fsid(fs_devices_mnt, total_devs, 1)))
> + if((ret = btrfs_scan_for_fsid(1)))
> return ret;
> }
>
> @@ -1110,8 +1110,7 @@ fail:
> return ret;
> }
>
> -int btrfs_scan_for_fsid(struct btrfs_fs_devices *fs_devices, u64 total_devs,
> - int run_ioctls)
> +int btrfs_scan_for_fsid(int run_ioctls)
> {
> int ret;
>
> diff --git a/utils.h b/utils.h
> index 3c17e14..dba37e8 100644
> --- a/utils.h
> +++ b/utils.h
> @@ -35,8 +35,7 @@ int btrfs_add_to_fsid(struct btrfs_trans_handle *trans,
> struct btrfs_root *root, int fd, char *path,
> u64 block_count, u32 io_width, u32 io_align,
> u32 sectorsize);
> -int btrfs_scan_for_fsid(struct btrfs_fs_devices *fs_devices, u64 total_devs,
> - int run_ioctls);
> +int btrfs_scan_for_fsid(int run_ioctls);
> void btrfs_register_one_device(char *fname);
> int btrfs_scan_one_dir(char *dirname, int run_ioctl);
> int check_mounted(const char *devicename);
>
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
