perf tools: Add the bash completion for listing subsubcommands of perf timechart
authorYunlong Song <yunlong.song@huawei.com>
Wed, 18 Mar 2015 13:35:56 +0000 (21:35 +0800)
committerArnaldo Carvalho de Melo <acme@redhat.com>
Thu, 19 Mar 2015 16:53:25 +0000 (13:53 -0300)
The bash completion does not support listing subsubcommands for 'perf
timechart <TAB>', so fix it.

Example:

Before this patch:

 $ perf timechart <TAB>
 $

As shown above, the subsubcommands of perf timechart does not come out.

After this patch:

 $ perf timechart <TAB>
 record

As shown above, the subsubcommands of perf timechart can come out now.

Signed-off-by: Yunlong Song <yunlong.song@huawei.com>
Tested-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Cc: Wang Nan <wangnan0@huawei.com>
Link: http://lkml.kernel.org/r/1426685758-25488-12-git-send-email-yunlong.song@huawei.com
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
tools/perf/builtin-timechart.c
tools/perf/perf-completion.sh

index 51440d1fc722d9a9475e37de5df036c469da1c2b..494b3bbe5ea430c3711f0e4ec00a8353c61260a1 100644 (file)
@@ -1958,7 +1958,8 @@ int cmd_timechart(int argc, const char **argv,
                     parse_time),
        OPT_END()
        };
-       const char * const timechart_usage[] = {
+       const char * const timechart_subcommands[] = { "record", NULL };
+       const char *timechart_usage[] = {
                "perf timechart [<options>] {record}",
                NULL
        };
@@ -1976,8 +1977,8 @@ int cmd_timechart(int argc, const char **argv,
                "perf timechart record [<options>]",
                NULL
        };
-       argc = parse_options(argc, argv, timechart_options, timechart_usage,
-                       PARSE_OPT_STOP_AT_NON_OPTION);
+       argc = parse_options_subcommand(argc, argv, timechart_options, timechart_subcommands,
+                       timechart_usage, PARSE_OPT_STOP_AT_NON_OPTION);
 
        if (tchart.power_only && tchart.tasks_only) {
                pr_err("-P and -T options cannot be used at the same time.\n");
index 35f43fd7a38b0128ebbf673bc89b6ed669a3bc8e..a33d2ef53ae4ddb03cb5caebd9337dcaf2c2e4da 100644 (file)
@@ -143,7 +143,7 @@ __perf_main ()
                __perfcomp_colon "$evts" "$cur"
        else
                # List subcommands for perf commands
-               if [[ $prev_skip_opts == @(kvm|kmem|mem|lock|sched|data|help|script|test) ]]; then
+               if [[ $prev_skip_opts == @(kvm|kmem|mem|lock|sched|data|help|script|test|timechart) ]]; then
                        subcmds=$($cmd $prev_skip_opts --list-cmds)
                        __perfcomp_colon "$subcmds" "$cur"
                fi