(2014/10/01 11:48), Hemant Kumar wrote:
> @@ -35,10 +45,13 @@ int cmd_sdt_cache(int argc, const char **argv, const char 
> *prefix __maybe_unused
>               OPT_CALLBACK('a', "add", NULL, "filename",
>                            "add SDT events from a file.",
>                            opt_add_sdt_events),
> +             OPT_CALLBACK_NOOPT('s', "dump", NULL, "show SDT events",
> +                                "Read SDT events from cache and display.",
> +                                opt_show_sdt_events),

Just one note here. -s and --dump are a bit odd. how about using -D for short 
ops?

Thank you,

-- 
Masami HIRAMATSU
Software Platform Research Dept. Linux Technology Research Center
Hitachi, Ltd., Yokohama Research Laboratory
E-mail: masami.hiramatsu...@hitachi.com


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

Reply via email to