On 10/17/2016 10:24 PM, Roman Mamedov wrote:
On Tue, 18 Oct 2016 09:39:32 +0800
Qu Wenruo <quwen...@cn.fujitsu.com> wrote:

  static const char * const cmd_inspect_inode_resolve_usage[] = {
        "btrfs inspect-internal inode-resolve [-v] <inode> <path>",
        "Get file system paths for the given inode",
@@ -702,6 +814,8 @@ const struct cmd_group inspect_cmd_group = {
                        0 },
                { "min-dev-size", cmd_inspect_min_dev_size,
                        cmd_inspect_min_dev_size_usage, NULL, 0 },
+               { "bg_analysis", cmd_inspect_bg_analysis,
+                       cmd_inspect_bg_analysis_usage, NULL, 0 },
Just naming preference, IMHO show-block-groups or dump-block-groups
seems better for me.
And in any case please don't mix separation by "-" and "_" in the same command
string. In btrfs tool the convention is to separate words in subcommand names
using "-".

Noted, thanks! Will update the patch to correct this.

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

Reply via email to