Re: [PATCH 2/2] perf sched: Add documentation for timehist options

2013-12-02 Thread David Ahern

On 12/2/13, 12:41 AM, Namhyung Kim wrote:

Hi David,

On Sun,  1 Dec 2013 17:23:15 -0700, David Ahern wrote:

Add entry to perf-sched documentation for timehist command and its
options.


[SNIP]

+--max-stack::
+Set the stack depth limit when showing the callchains. Default: 127


Isn't it 5 instead of 127?


drat. missed that one.

David

--
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/


Re: [PATCH 2/2] perf sched: Add documentation for timehist options

2013-12-02 Thread Ingo Molnar

* Srikar Dronamraju  wrote:

> * David Ahern  [2013-12-01 17:23:15]:
> 
> > Add entry to perf-sched documentation for timehist command and its
> > options.
> > 
> > v2: Addressed comments by Namhyung
> > 
> > Signed-off-by: David Ahern 
> > Cc: Ingo Molnar 
> > Cc: Frederic Weisbecker 
> > Cc: Peter Zijlstra 
> > Cc: Mike Galbraith 
> > Cc: Jiri Olsa 
> > Cc: Namhyung Kim 
> > Cc: Stephane Eranian 
> > Cc: Pekka Enberg 
> > ---
> >  tools/perf/Documentation/perf-sched.txt | 61 
> > -
> >  1 file changed, 60 insertions(+), 1 deletion(-)
> > 
> > diff --git a/tools/perf/Documentation/perf-sched.txt 
> > b/tools/perf/Documentation/perf-sched.txt
> > index 8ff4df956951..e7bb19fcdbff 100644
> > --- a/tools/perf/Documentation/perf-sched.txt
> > +++ b/tools/perf/Documentation/perf-sched.txt
> > @@ -8,7 +8,7 @@ perf-sched - Tool to trace/measure scheduler properties 
> > (latencies)
> >  SYNOPSIS
> >  
> >  [verse]
> > -'perf sched' {record|latency|map|replay|script}
> > +'perf sched' {record|latency|map|replay|script|timehist}
> >  
> >  DESCRIPTION
> >  ---
> > @@ -36,6 +36,11 @@ There are five variants of perf sched:
> 
> Nit: There are six variants now.

Trick: it might make sense to use a more future-proof phrase:

   There are several variants of perf sched:

;-)

Thanks,

Ingo
--
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/


Re: [PATCH 2/2] perf sched: Add documentation for timehist options

2013-12-01 Thread Namhyung Kim
Hi David,

On Sun,  1 Dec 2013 17:23:15 -0700, David Ahern wrote:
> Add entry to perf-sched documentation for timehist command and its
> options.

[SNIP]
> +--max-stack::
> +Set the stack depth limit when showing the callchains. Default: 127

Isn't it 5 instead of 127?

Thanks,
Namhyung
--
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/


Re: [PATCH 2/2] perf sched: Add documentation for timehist options

2013-12-01 Thread Srikar Dronamraju
* David Ahern  [2013-12-01 17:23:15]:

> Add entry to perf-sched documentation for timehist command and its
> options.
> 
> v2: Addressed comments by Namhyung
> 
> Signed-off-by: David Ahern 
> Cc: Ingo Molnar 
> Cc: Frederic Weisbecker 
> Cc: Peter Zijlstra 
> Cc: Mike Galbraith 
> Cc: Jiri Olsa 
> Cc: Namhyung Kim 
> Cc: Stephane Eranian 
> Cc: Pekka Enberg 
> ---
>  tools/perf/Documentation/perf-sched.txt | 61 
> -
>  1 file changed, 60 insertions(+), 1 deletion(-)
> 
> diff --git a/tools/perf/Documentation/perf-sched.txt 
> b/tools/perf/Documentation/perf-sched.txt
> index 8ff4df956951..e7bb19fcdbff 100644
> --- a/tools/perf/Documentation/perf-sched.txt
> +++ b/tools/perf/Documentation/perf-sched.txt
> @@ -8,7 +8,7 @@ perf-sched - Tool to trace/measure scheduler properties 
> (latencies)
>  SYNOPSIS
>  
>  [verse]
> -'perf sched' {record|latency|map|replay|script}
> +'perf sched' {record|latency|map|replay|script|timehist}
>  
>  DESCRIPTION
>  ---
> @@ -36,6 +36,11 @@ There are five variants of perf sched:

Nit: There are six variants now.

-- 
Thanks and Regards
Srikar Dronamraju

--
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/


[PATCH 2/2] perf sched: Add documentation for timehist options

2013-12-01 Thread David Ahern
Add entry to perf-sched documentation for timehist command and its
options.

v2: Addressed comments by Namhyung

Signed-off-by: David Ahern 
Cc: Ingo Molnar 
Cc: Frederic Weisbecker 
Cc: Peter Zijlstra 
Cc: Mike Galbraith 
Cc: Jiri Olsa 
Cc: Namhyung Kim 
Cc: Stephane Eranian 
Cc: Pekka Enberg 
---
 tools/perf/Documentation/perf-sched.txt | 61 -
 1 file changed, 60 insertions(+), 1 deletion(-)

diff --git a/tools/perf/Documentation/perf-sched.txt 
b/tools/perf/Documentation/perf-sched.txt
index 8ff4df956951..e7bb19fcdbff 100644
--- a/tools/perf/Documentation/perf-sched.txt
+++ b/tools/perf/Documentation/perf-sched.txt
@@ -8,7 +8,7 @@ perf-sched - Tool to trace/measure scheduler properties 
(latencies)
 SYNOPSIS
 
 [verse]
-'perf sched' {record|latency|map|replay|script}
+'perf sched' {record|latency|map|replay|script|timehist}
 
 DESCRIPTION
 ---
@@ -36,6 +36,11 @@ There are five variants of perf sched:
   are running on a CPU. A '*' denotes the CPU that had the event, and
   a dot signals an idle CPU.
 
+  'perf sched timehist' to show context-switching analysis with times
+  between schedule-in, schedule delay, and run time. If callchains are
+  present stack trace is dumped at the end of the line. A summary of
+  run times can be shown as well.
+
 OPTIONS
 ---
 -i::
@@ -50,6 +55,60 @@ OPTIONS
 --dump-raw-trace=::
 Display verbose dump of the sched data.
 
+TIMEHIST OPTIONS
+
+-k::
+--vmlinux=::
+vmlinux pathname
+
+--kallsyms=::
+kallsyms pathname
+
+-s::
+--summary::
+Show only a summary of scheduling by thread with min, max, and average
+run times (in sec) and relative stddev.
+
+-S::
+--with-summary::
+Show all scheduling events followed by a summary by thread with min,
+max, and average run times (in sec) and relative stddev.
+
+-w::
+--wakeups::
+Show wakeup events.
+
+-c::
+--comms=::
+Only show events for given comms (comma separated list)
+
+-p::
+--pid=::
+Only show events for given process ID (comma separated list).
+
+-t::
+--tid=::
+Only show events for given thread ID (comma separated list).
+
+--call-graph::
+Display call chains. Default is on. Use --no-call-graph to suppress
+
+--max-stack::
+Set the stack depth limit when showing the callchains. Default: 127
+
+-x::
+--exclude-sym::
+Symbols to skip in backtrace. Useful to compact repetitive or
+uninteresting frames to pull more useful frames into view.
+
+--symfs=::
+Look for files with symbols relative to this directory.
+
+-V::
+--cpu-visual::
+Add a visual that better emphasizes activity by cpu. Idle times
+are denoted with 'i'; schedule events with an 's'.
+
 SEE ALSO
 
 linkperf:perf-record[1]
-- 
1.8.3.4 (Apple Git-47)

--
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/