perf tools: change event inheritance logic in stat and record
authorStephane Eranian <eranian@google.com>
Wed, 12 May 2010 08:40:01 +0000 (10:40 +0200)
committerArnaldo Carvalho de Melo <acme@redhat.com>
Thu, 13 May 2010 19:39:12 +0000 (16:39 -0300)
commit2e6cdf996ba43ce0b090ffbf754f83e17362cd69
treebb471a4ae13fa3941612c30c75bead7417084667
parent8a0ecfb8b47dc765fdf460913231876bbc95385e
perf tools: change event inheritance logic in stat and record

By default, event inheritance across fork and pthread_create was on but the -i
option of stat and record, which enabled inheritance, led to believe it was off
by default.

This patch fixes this logic by inverting the meaning of the -i option.  By
default inheritance is on whether you attach to a process (-p), a thread (-t)
or start a process. If you pass -i, then you turn off inheritance. Turning off
inheritance if you don't need it, helps limit perf resource usage as well.

The patch also fixes perf stat -t xxxx and perf record -t xxxx which did not
start the counters.

Acked-by: Frederic Weisbecker <fweisbec@gmail.com>
Cc: David S. Miller <davem@davemloft.net>
Cc: Frédéric Weisbecker <fweisbec@gmail.com>
Cc: Ingo Molnar <mingo@elte.hu>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Peter Zijlstra <peterz@infradead.org>
LKML-Reference: <4bea9d2f.d60ce30a.0b5b.08e1@mx.google.com>
Signed-off-by: Stephane Eranian <eranian@google.com>
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
tools/perf/Documentation/perf-record.txt
tools/perf/Documentation/perf-stat.txt
tools/perf/builtin-record.c
tools/perf/builtin-stat.c