Re: [PATCH 3/8] Add verbose option to btrfs_add_to_fsid()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 2014/12/18 5:14, Goffredo Baroncelli wrote:
> Add verbose option to btrfs_add_to_fsid() in order to avoid to print
> the information to console when not needed.
> The same information is print in the summary of the mkfs.btrfs command.
> 
> Signed-off-by: Goffredo Baroncelli <kreijack@xxxxxxxxx>

Reviewed-by: Satoru Takeuchi <takeuchi_satoru@xxxxxxxxxxxxxx>

> ---
>   mkfs.c  | 3 ++-
>   utils.c | 7 ++++---
>   utils.h | 2 +-
>   3 files changed, 7 insertions(+), 5 deletions(-)
> 
> diff --git a/mkfs.c b/mkfs.c
> index a5f19cc..042d12e 100644
> --- a/mkfs.c
> +++ b/mkfs.c
> @@ -1638,7 +1638,8 @@ int main(int ac, char **av)
>   		mixed = old_mixed;
>   
>   		ret = btrfs_add_to_fsid(trans, root, fd, file, dev_block_count,
> -					sectorsize, sectorsize, sectorsize);
> +					sectorsize, sectorsize, sectorsize,
> +					verbose);
>   		BUG_ON(ret);
>   		btrfs_register_one_device(file);
>   	}
> diff --git a/utils.c b/utils.c
> index 19049bb..dcb4f74 100644
> --- a/utils.c
> +++ b/utils.c
> @@ -610,7 +610,7 @@ static int zero_dev_clamped(int fd, off_t start, ssize_t len, u64 dev_size)
>   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)
> +		      u32 sectorsize, int verbose)
>   {
>   	struct btrfs_super_block *disk_super;
>   	struct btrfs_super_block *super = root->fs_info->super_copy;
> @@ -659,8 +659,9 @@ int btrfs_add_to_fsid(struct btrfs_trans_handle *trans,
>   
>   	memcpy(disk_super, super, sizeof(*disk_super));
>   
> -	printf("adding device %s id %llu\n", path,
> -	       (unsigned long long)device->devid);
> +	if (verbose)
> +		printf("adding device %s id %llu\n", path,
> +		       (unsigned long long)device->devid);
>   
>   	btrfs_set_super_bytenr(disk_super, BTRFS_SUPER_INFO_OFFSET);
>   	btrfs_set_stack_device_id(dev_item, device->devid);
> diff --git a/utils.h b/utils.h
> index bb8a449..9218199 100644
> --- a/utils.h
> +++ b/utils.h
> @@ -79,7 +79,7 @@ int btrfs_prepare_device(int fd, char *file, int zero_end, u64 *block_count_ret,
>   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);
> +		      u32 sectorsize, int verbose);
>   int btrfs_scan_for_fsid(int run_ioctls);
>   int btrfs_register_one_device(const char *fname);
>   int btrfs_register_all_devices(void);
> 

--
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



[Index of Archives]     [Linux Filesystem Development]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux