linux-kernel - Re: [PATCH] perf: change event inheritance logic in stat and record

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20100512152447.GF5405@nowhere>
Date:	Wed, 12 May 2010 17:24:52 +0200
From:	Frederic Weisbecker <fweisbec@...il.com>
To:	Stephane Eranian <eranian@...gle.com>
Cc:	linux-kernel@...r.kernel.org, peterz@...radead.org, mingo@...e.hu,
	paulus@...ba.org, davem@...emloft.net, acme@...radead.org,
	perfmon2-devel@...ts.sf.net, eranian@...il.com
Subject: Re: [PATCH] perf: change event inheritance logic in stat and record

On Wed, May 12, 2010 at 10:40:01AM +0200, Stephane Eranian wrote:
>   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.
> 
>   Signed-off-by: Stephane Eranian <eranian@...gle.com>


Acked-by: Frederic Weisbecker <fweisbec@...il.com>

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