Re: [PATCHSET 0/5] perf tools: Enhance option parsing error message

From: Ingo Molnar
Date: Fri Nov 01 2013 - 03:38:19 EST



* Namhyung Kim <namhyung@xxxxxxxxxx> wrote:

> Hi,
>
> This patchset tries to enhance error message when perf failed to parse
> option argument. Currently it just shows entire usage and options
> which may fill two pages of screen. It's hard for me to say it's
> really helpful to users. ;-)
>
> With this patch series, it changed to show only related option(s).
> There're also more points to improve yet - like auto-breaking long
> lines, dealing with UIs and so on. But I believe this will be a good
> start.
>
> $ perf report -g help
>
> usage: perf report [<options>]
>
> -g, --call-graph <output_type,min_percent[,print_limit],call_order>
> Display callchains using output_type (graph, flat, fractal, or none) , min percent threshold, optional print limit, callchain order, key (function or address). Default: fractal,0.5,callee,function
>
> $ perf stat -Bx, ls
> -B option not supported with -x
>
> usage: perf stat [<options>] [<command>]
>
> -B, --big-num print large numbers with thousands' separators
> -x, --field-separator <separator>
> print counts with custom separator
>
>
> You can get this on 'perf/option-v1' branch in my tree
>
> git://git.kernel.org/pub/scm/linux/kernel/git/namhyung/linux-perf.git

Cool!

Acked-by: Ingo Molnar <mingo@xxxxxxxxxx>
Reviewed-by: Ingo Molnar <mingo@xxxxxxxxxx>
Enthusiastically-Supported-by: Ingo Molnar <mingo@xxxxxxxxxx>
/me runs out of tags

Thanks!

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