Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1351919 > unrolled thread

[PATCH 08/15] perf stat: Document --detailed option

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  [PATCH 08/15] perf stat: Document --detailed option Arnaldo Carvalho de Melo <acme@kernel.org> - 2016-03-07 20:50 +0100
    Re: [PATCH 08/15] perf stat: Document --detailed option Ingo Molnar <mingo@kernel.org> - 2016-03-08 10:20 +0100

#1351919 — [PATCH 08/15] perf stat: Document --detailed option

From: Borislav Petkov <bp@suse.de>

I'm surprised this remained undocumented since at least 2011. And it is
actually a very useful switch, as Steve and I came to realize recently.

Add the text from

  2cba3ffb9a9d ("perf stat: Add -d -d and -d -d -d options to show more CPU events")

which added the incrementing aspect to -d.

Signed-off-by: Borislav Petkov <bp@suse.de>
Tested-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Cc: Davidlohr Bueso <dbueso@suse.com>
Cc: Jiri Olsa <jolsa@redhat.com>
Cc: Mel Gorman <mgorman@suse.com>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Cc: Steven Rostedt <rostedt@goodmis.org>
Fixes: 2cba3ffb9a9d ("perf stat: Add -d -d and -d -d -d options to show more CPU events")
Link: http://lkml.kernel.org/r/1457347294-32546-1-git-send-email-bp@alien8.de
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
---
 tools/perf/Documentation/perf-stat.txt | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/tools/perf/Documentation/perf-stat.txt b/tools/perf/Documentation/perf-stat.txt
index 52ef7a9d50aa..14d9e8ffaff7 100644
--- a/tools/perf/Documentation/perf-stat.txt
+++ b/tools/perf/Documentation/perf-stat.txt
@@ -69,6 +69,14 @@ report::
 --scale::
 	scale/normalize counter values
 
+-d::
+--detailed::
+	print more detailed statistics, can be specified up to 3 times
+
+	   -d:          detailed events, L1 and LLC data cache
+        -d -d:     more detailed events, dTLB and iTLB events
+     -d -d -d:     very detailed events, adding prefetch events
+
 -r::
 --repeat=<n>::
 	repeat command and print average + stddev (max: 100). 0 means forever.
-- 
2.5.0

[toc] | [next] | [standalone]

#1352773

* Arnaldo Carvalho de Melo <acme@kernel.org> wrote:

> From: Borislav Petkov <bp@suse.de>
> 
> I'm surprised this remained undocumented since at least 2011. And it is
> actually a very useful switch, as Steve and I came to realize recently.
> 
> Add the text from
> 
>   2cba3ffb9a9d ("perf stat: Add -d -d and -d -d -d options to show more CPU events")
> 
> which added the incrementing aspect to -d.

Ah yes, my fault ...

> Signed-off-by: Borislav Petkov <bp@suse.de>
> Tested-by: Arnaldo Carvalho de Melo <acme@redhat.com>
> Cc: Davidlohr Bueso <dbueso@suse.com>
> Cc: Jiri Olsa <jolsa@redhat.com>
> Cc: Mel Gorman <mgorman@suse.com>
> Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
> Cc: Steven Rostedt <rostedt@goodmis.org>
> Fixes: 2cba3ffb9a9d ("perf stat: Add -d -d and -d -d -d options to show more CPU events")
> Link: http://lkml.kernel.org/r/1457347294-32546-1-git-send-email-bp@alien8.de
> Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
> ---
>  tools/perf/Documentation/perf-stat.txt | 8 ++++++++
>  1 file changed, 8 insertions(+)
> 
> diff --git a/tools/perf/Documentation/perf-stat.txt b/tools/perf/Documentation/perf-stat.txt
> index 52ef7a9d50aa..14d9e8ffaff7 100644
> --- a/tools/perf/Documentation/perf-stat.txt
> +++ b/tools/perf/Documentation/perf-stat.txt
> @@ -69,6 +69,14 @@ report::
>  --scale::
>  	scale/normalize counter values
>  
> +-d::
> +--detailed::
> +	print more detailed statistics, can be specified up to 3 times
> +
> +	   -d:          detailed events, L1 and LLC data cache
> +        -d -d:     more detailed events, dTLB and iTLB events
> +     -d -d -d:     very detailed events, adding prefetch events

Btw., something I noticed: it would be really nice if 'perf stat -h -d' printed 
the most detailed documentation available for the option. Right now it prints:

 triton:~/tip/tools/perf> perf stat -h -d

  Usage: perf stat [<options>] [<command>]

    -d, --detailed        detailed run - start a lot of events

The only way to get to the most detailed documentation is the not very obvious 
path of:

 perf help stat

and then searching for '-d'.

Ideally there should only be a single place to document options - but I guess 
unifying the in-code option descriptions and more verbose description in the man 
pages is very non-trivial ...

Thanks,

	Ingo

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web