From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756380AbaHVNb2 (ORCPT ); Fri, 22 Aug 2014 09:31:28 -0400 Received: from mx1.redhat.com ([209.132.183.28]:48656 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756216AbaHVNb1 (ORCPT ); Fri, 22 Aug 2014 09:31:27 -0400 Date: Fri, 22 Aug 2014 15:30:56 +0200 From: Jiri Olsa To: Jiri Olsa Cc: linux-kernel@vger.kernel.org, Andi Kleen , Arnaldo Carvalho de Melo , Corey Ashford , David Ahern , Frederic Weisbecker , Ingo Molnar , "Jen-Cheng(Tommy) Huang" , Namhyung Kim , Paul Mackerras , Peter Zijlstra , Stephane Eranian Subject: Re: [RFC 0/5] perf: Allow leader sampling on inherited events Message-ID: <20140822133056.GA22315@krava.brq.redhat.com> References: <1408712718-19656-1-git-send-email-jolsa@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1408712718-19656-1-git-send-email-jolsa@kernel.org> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Aug 22, 2014 at 03:05:13PM +0200, Jiri Olsa wrote: > hi, > Jen-Cheng(Tommy) Huang reported the leader sampling not working > on children processes: > http://www.mail-archive.com/linux-perf-users@vger.kernel.org/msg01644.html > > The leader sampling (example below) lets the group leader event (cycles) > do the sampling and reads the rest of the group (cache-misses) via > PERF_FORMAT_GROUP format. > > Example: > $ perf record -e '{cycles,cache-misses}:S' > $ perf report --group > > The perf report --group allows to see all events group > data in single view. > > The reason for leader sampling being switched off for inherited > events, is that the kernel does no allow PERF_FORMAT_GROUP format > on inherited events (which is used by leader sampling). > > I switched on the PERF_FORMAT_GROUP format for inherited events > with few other fixies in patches: > perf: Deny optimized switch for events read by PERF_SAMPLE_READ > perf: Allow PERF_FORMAT_GROUP format on inherited events > > And I fixed perf tool code to be able to process data from > children processes. > > Anyway, I might have missed some other reason why this was > never switched on in kernel, so sending this as RFC. Also reachable in here: git://git.kernel.org/pub/scm/linux/kernel/git/jolsa/perf.git perf/core_format_group jirka