Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753098AbbG2TUs (ORCPT ); Wed, 29 Jul 2015 15:20:48 -0400 Received: from mail.kernel.org ([198.145.29.136]:60442 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751820AbbG2TUr (ORCPT ); Wed, 29 Jul 2015 15:20:47 -0400 Date: Wed, 29 Jul 2015 16:20:38 -0300 From: Arnaldo Carvalho de Melo To: Kan Liang Cc: jolsa@kernel.org, namhyung@kernel.org, ak@linux.intel.com, linux-kernel@vger.kernel.org Subject: Re: [PATCH RFC V7 0/7] per event callgraph and time support Message-ID: <20150729192038.GK575@kernel.org> References: <1438162936-59698-1-git-send-email-kan.liang@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1438162936-59698-1-git-send-email-kan.liang@intel.com> X-Url: http://acmel.wordpress.com 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 Content-Length: 2825 Lines: 72 Em Wed, Jul 29, 2015 at 05:42:09AM -0400, Kan Liang escreveu: > This patchkit adds the ability to turn off callgraphs and time stamps > per event. This in term can reduce sampling overhead and the size of > the perf.data. Applied the first three patches, will wait for Jiri comments on the remaining. - Arnaldo > Changes since V1: > - Break up V1 patches into three patches(parse option changes, > partial time support and partial callgraph support). > - Use strings 'fp,dwarf,lbr,no' to identify callchains > - Add test case in parse-events.c > > Changes since V2: > - Rebase on 60cd37eb10 > > Changes since V3: > - Replace OPT_CALLBACK_SET by current existing callback mechanism. > - Using perf_evsel__set_sample_bit if possible > - Change the expression "partial" to "per event" > - Using global variable to indicate if 'time' is set per event. > If 'time' is not set, enable it by default for perf record. > > Changes since V4: > - Fix issue of setting callgraph_set > > Changes since V5: > - per-event settings over global settings in general > - support for event post configuration structure > > Changes since V6: > - Add error infor for per-event time > - rename the option to call-graph and stacl-size > - refine parse/config callchain functions, > and reuse them for per-event callgraph setting > - Modify perf test > > Jiri Olsa (2): > perf tools: Add support for event post configuration > perf tools: Force period term to overload global settings > > Kan Liang (5): > perf,tools: introduce callgraph_set for callgraph option > perf,tools: per-event time support > perf,tools: refine parse/config callchain functions > perf,tools: per-event callgraph support > perf,tests: Add tests to callgraph and time parse > > tools/perf/Documentation/perf-record.txt | 10 ++- > tools/perf/builtin-record.c | 11 +++- > tools/perf/perf.h | 1 + > tools/perf/tests/parse-events.c | 50 ++++++++++++++- > tools/perf/util/callchain.c | 14 ++--- > tools/perf/util/callchain.h | 2 +- > tools/perf/util/evsel.c | 105 +++++++++++++++++++++++++++++-- > tools/perf/util/evsel.h | 26 ++++++++ > tools/perf/util/parse-events.c | 94 +++++++++++++++++++++++---- > tools/perf/util/parse-events.h | 3 + > tools/perf/util/parse-events.l | 3 + > tools/perf/util/pmu.c | 3 +- > 12 files changed, 289 insertions(+), 33 deletions(-) > > -- > 1.8.3.1 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/