From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from cn.fujitsu.com ([59.151.112.132]:42897 "EHLO heian.cn.fujitsu.com" rhost-flags-OK-FAIL-OK-FAIL) by vger.kernel.org with ESMTP id S1750841AbbAOBR4 convert rfc822-to-8bit (ORCPT ); Wed, 14 Jan 2015 20:17:56 -0500 Message-ID: <54B7150D.20903@cn.fujitsu.com> Date: Thu, 15 Jan 2015 09:17:01 +0800 From: =?UTF-8?B?RmFuIENoZW5nbmlhbmcv5qiK5oiQ6YW/?= MIME-Version: 1.0 To: CC: Subject: Re: [PATCH v2 RESEND] btrfs-progs: make btrfs qgroups show human readable sizes References: <1421128419-17169-1-git-send-email-fancn.fnst@cn.fujitsu.com> <20150114154633.GU3685@twin.jikos.cz> In-Reply-To: <20150114154633.GU3685@twin.jikos.cz> Content-Type: text/plain; charset="UTF-8"; format=flowed Sender: linux-btrfs-owner@vger.kernel.org List-ID: 在 2015年01月14日 23:46, David Sterba 写道: > On Tue, Jan 13, 2015 at 01:53:39PM +0800, Fan Chengniang wrote: >> make btrfs qgroups show human readable sizes >> using --human-readable option, example: > That's too long to type and the idea was to add all the long options > that force the specific unit base, ie. --kbytes/--mbytes/..., --raw, > --si and --iec. We can possibly make the human readable the default > because that's what I'd expect to see to have a quick overview and can > use the other options otherwise. > > The geopt parser accepts short options if they're unique, so --kb or > even --k works as a very convenient shorcut for frequent commandline > use. I have sent a mail for your advise of adding options. In that mail, I asked whether I should use --human-readable and add --kbytes --mbytes ... But you have not reply to me. So, your advise is add --kbytes --mbytes ... and make human-readable default behaviour? >> qgroupid rfer excl max_rfer max_excl parent child >> -------- ---- ---- -------- -------- ------ ----- >> 0/5 299.58MiB 299.58MiB 400.00MiB 0.00B 1/1 --- >> 0/265 299.58MiB 16.00KiB 0.00B 320.00MiB 1/1 --- >> 0/266 299.58MiB 16.00KiB 350.00MiB 0.00B --- --- >> 1/1 599.16MiB 299.59MiB 800.00MiB 0.00B --- 0/5,0/265 > The values should be also aligned to the right. It is aligned to left before my patch. I just keep it.