From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754241AbbKBVa0 (ORCPT ); Mon, 2 Nov 2015 16:30:26 -0500 Received: from mail.kernel.org ([198.145.29.136]:33886 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752134AbbKBVaZ (ORCPT ); Mon, 2 Nov 2015 16:30:25 -0500 Date: Mon, 2 Nov 2015 18:30:21 -0300 From: Arnaldo Carvalho de Melo To: Brendan Gregg Cc: Namhyung Kim , Ingo Molnar , Peter Zijlstra , Jiri Olsa , LKML , David Ahern , Frederic Weisbecker , Andi Kleen , Kan Liang Subject: Re: [RFC/PATCH 0/4] perf report: Support folded callchain output (v2) Message-ID: <20151102213021.GA21609@kernel.org> References: <1446469069-16223-1-git-send-email-namhyung@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: 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 Em Mon, Nov 02, 2015 at 12:37:28PM -0800, Brendan Gregg escreveu: > G'Day Namhyung, > > On Mon, Nov 2, 2015 at 4:57 AM, Namhyung Kim wrote: > > Hello, > > > > This is what Brendan requested on the perf-users mailing list [1] to > > support FlameGraphs [2] more efficiently. This patchset adds a few > > more callchain options to adjust the output for it. > > > > At first, 'folded' output mode was added. The folded output puts all > > calchain nodes in a line separated by semicolons, a space and the > > value. Now it only supports --stdio as other UI provides some way of > > folding/expanding callchains dynamically. > > > > The value is now can be one of 'percent', 'period', or 'count'. The > > percent is current default output and the period is the raw number of > > sample periods. The count is the number of samples for each callchain. > > > > Here's an example: > > > > $ perf report --no-children --show-nr-samples --stdio -g folded,count > > ... > > 39.93% 80 swapper [kernel.vmlinux] [k] intel_idel > > intel_idle;cpuidle_enter_state;cpuidle_enter;call_cpuidle;cpu_startup_entry;start_secondary 57 > > intel_idle;cpuidle_enter_state;cpuidle_enter;call_cpuidle;cpu_startup_entry;rest_init;... 23 > > Thanks! > > So for the folded output I don't need the summary line (the row of > columns printed by hist_entry__snprintf()), and don't need anything > except folded stacks and the counts. If working with the existing > stdio interface is making it harder than it needs to be, might it be I don't think it so, just add some flag asking for that hist_entry__snprintf() to be supressed, ideas for a long option name? Having it as Namhyung did may have value for some people as a more compact way to show the callchains together with the hist_entry line. With this in mind, do you have any other issues with Namhyung's patchkit? An acked-by/tested-by you would be nice to have, and then we could work out the new option to suppress that hist_entry__snprintf() in a follow up patch. > easier to make it a separate interface (ui/folded), that just emitted > the folded output? Just an idea. This existing patchset is working for > me, I'd just be filtering the output. > > Having the option for percentages and periods is nice. I can envisage > using periods (for latency flame graphs). You mean in the callchain lines? - Arnaldo