Re: [PATCH v11 08/24] perf config: Document 'hist.percentage' variable in man page

[Namhyung Kim <namhyung@kernel.org>]

On Tue, Nov 17, 2015 at 10:53:28PM +0900, Taeung Song wrote:
> Explain 'hist.percentage' variable.
> 
> Cc: Jiri Olsa <jolsa@redhat.com>
> Signed-off-by: Taeung Song <treeze.taeung@gmail.com>

Acked-by: Namhyung Kim <namhyung@kernel.org>

One nitpick below..


> ---
>  tools/perf/Documentation/perf-config.txt | 17 +++++++++++++++++
>  1 file changed, 17 insertions(+)
> 
> diff --git a/tools/perf/Documentation/perf-config.txt b/tools/perf/Documentation/perf-config.txt
> index 88e9b88..b166b81 100644
> --- a/tools/perf/Documentation/perf-config.txt
> +++ b/tools/perf/Documentation/perf-config.txt
> @@ -263,6 +263,23 @@ help.*::
>  		<perf sub-command> can be record, script, report or inject
>  		    or: perf-with-kcore fix_buildid_cache_permissions
>  
> +hist.*::
> +	hist.percentage::
> +		This option control a way to calcurate overhead of filtered entries -

s/control a way/controls the way/,  s/calcurate/calculate/

Thanks,
Namhyung


> +		that means the value of this option is effective only if there's a
> +		filter (by comm, dso or symbol name). Suppose a following example:
> +
> +		       Overhead  Symbols
> +		       ........  .......
> +		        33.33%     foo
> +		        33.33%     bar
> +		        33.33%     baz
> +
> +	       This is an original overhead and we'll filter out the first 'foo'
> +	       entry. The value of 'relative' would increase the overhead of 'bar'
> +	       and 'baz' to 50.00% for each, while 'absolute' would show their
> +	       current overhead (33.33%).
> +
>  SEE ALSO
>  --------
>  linkperf:perf[1]
> -- 
> 1.9.1
>