On 2015/06/03 15:57, Dongsheng Yang wrote:
> There are two understanding of the '0' value in btrfs qgroup show.
> (1) is no-limitation on this qgroup. (2) is the max-limitation is 0.
> 
> This patch make it showing in different way.
> 
> (1). max-limitation for 0 is still showing '0'.
> (2). no-limitation will show 'none'.
> 
> qgroupid         rfer         excl     max_rfer     max_excl parent
> --------         ----         ----     --------     -------- ------
> 0/5           2.19GiB      2.19GiB         none         none ---
> 0/257       100.02MiB    100.02MiB         none         none ---
> 
> Signed-off-by: Dongsheng Yang <[email protected]>

Tested-by: Tsutomu Itoh <[email protected]>

> ---
>   qgroup.c | 10 ++++++++--
>   1 file changed, 8 insertions(+), 2 deletions(-)
> 
> diff --git a/qgroup.c b/qgroup.c
> index 53815b5..dc04b03 100644
> --- a/qgroup.c
> +++ b/qgroup.c
> @@ -237,10 +237,16 @@ static void print_qgroup_column(struct btrfs_qgroup 
> *qgroup,
>               print_qgroup_column_add_blank(BTRFS_QGROUP_PARENT, len);
>               break;
>       case BTRFS_QGROUP_MAX_RFER:
> -             len = printf("%*s", max_len, pretty_size_mode(qgroup->max_rfer, 
> unit_mode));
> +             if (qgroup->flags & BTRFS_QGROUP_LIMIT_MAX_RFER)
> +                     len = printf("%*s", max_len, 
> pretty_size_mode(qgroup->max_rfer, unit_mode));
> +             else
> +                     len = printf("%*s", max_len, "none");
>               break;
>       case BTRFS_QGROUP_MAX_EXCL:
> -             len = printf("%*s", max_len, pretty_size_mode(qgroup->max_excl, 
> unit_mode));
> +             if (qgroup->flags & BTRFS_QGROUP_LIMIT_MAX_EXCL)
> +                     len = printf("%*s", max_len, 
> pretty_size_mode(qgroup->max_excl, unit_mode));
> +             else
> +                     len = printf("%*s", max_len, "none");
>               break;
>       case BTRFS_QGROUP_CHILD:
>               len = print_child_column(qgroup);
> 


--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to