Re: [PATCH] libceph: separate multiple ops with commas in debugfs output

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

 



Reviewed-by: Sage Weil <sage@xxxxxxxxxx>

On Wed, 8 Oct 2014, Ilya Dryomov wrote:

> For requests with multiple ops, separate ops with commas instead of \t,
> which is a field separator here.
> 
> Signed-off-by: Ilya Dryomov <idryomov@xxxxxxxxxx>
> ---
>  net/ceph/debugfs.c |    3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/net/ceph/debugfs.c b/net/ceph/debugfs.c
> index d1a62c69a9f4..d2d525529f87 100644
> --- a/net/ceph/debugfs.c
> +++ b/net/ceph/debugfs.c
> @@ -169,7 +169,8 @@ static int osdc_show(struct seq_file *s, void *pp)
>  
>  		for (i = 0; i < req->r_num_ops; i++) {
>  			opcode = req->r_ops[i].op;
> -			seq_printf(s, "\t%s", ceph_osd_op_name(opcode));
> +			seq_printf(s, "%s%s", (i == 0 ? "\t" : ","),
> +				   ceph_osd_op_name(opcode));
>  		}
>  
>  		seq_printf(s, "\n");
> -- 
> 1.7.10.4
> 
> --
> To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 
> 
--
To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [CEPH Users]     [Ceph Large]     [Information on CEPH]     [Linux BTRFS]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux