Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752001AbaAKPhA (ORCPT ); Sat, 11 Jan 2014 10:37:00 -0500 Received: from mx1.redhat.com ([209.132.183.28]:12950 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751099AbaAKPg4 (ORCPT ); Sat, 11 Jan 2014 10:36:56 -0500 Date: Sat, 11 Jan 2014 16:36:14 +0100 From: Jiri Olsa To: Andi Kleen Cc: acme@infradead.org, linux-kernel@vger.kernel.org, namhyung@kernel.org, eranian@google.com, fweisbec@gmail.com, mingo@kernel.org, adrian.hunter@intel.com, dsahern@gmail.com, Andi Kleen Subject: Re: [PATCH 1/4] perf, tools: Add support for prepending LBRs to the callstack Message-ID: <20140111153614.GC1131@krava.brq.redhat.com> References: <1389357126-3003-1-git-send-email-andi@firstfloor.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1389357126-3003-1-git-send-email-andi@firstfloor.org> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Jan 10, 2014 at 04:32:03AM -0800, Andi Kleen wrote: > From: Andi Kleen > > I never found the default LBR display mode which generates histograms > of individual branches particularly useful. > > This implements an alternative mode that creates histograms over complete > branch traces, instead of individual branches, similar to how normal > callgraphs are handled. This is done by putting it in > front of the normal callgraph and then using the normal callgraph > histogram infrastructure to unify them. > > This way in complex functions we can understand the control flow > that lead to a particular sample. > > The default output is unchanged. > > This is only implemented in perf report, no change to record > or anywhere else. > > This adds the basic code to report: > - add a new "branch" option to the -g option parser to enable this mode > - when the flag is set include the LBR into the callstack in machine.c. > The rest of the history code is unchanged and doesn't know the difference > between LBR entry and normal call entry. sounds like nice idea, but I could not get the patchset applied on acme's perf/core jirka -- 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/