Re: [PATCH v1 4/4] perf script: enable printing of branch stack

2015-08-19 Thread Andi Kleen
> > Also it would be good to resolve the addresses to symbol + offset > > But then, that becomes harder to post-process or even read with long > C++ signatures. Ok. Maybe make it optional? For many usages resolved addresses will be much more useful than raw. Normall call stacks also do that. -

Re: [PATCH v1 4/4] perf script: enable printing of branch stack

2015-08-19 Thread Stephane Eranian
On Wed, Aug 19, 2015 at 3:11 PM, Andi Kleen wrote: > On Wed, Aug 19, 2015 at 10:38:26AM +0200, Stephane Eranian wrote: >> This patch improves perf script by enabling printing of the >> branch stack via the 'brstack' argument to the field selection >> option -F. The option is off by default and ope

Re: [PATCH v1 4/4] perf script: enable printing of branch stack

2015-08-19 Thread Andi Kleen
On Wed, Aug 19, 2015 at 10:38:26AM +0200, Stephane Eranian wrote: > This patch improves perf script by enabling printing of the > branch stack via the 'brstack' argument to the field selection > option -F. The option is off by default and operates only if the > perf.data file has branch stack conte

[PATCH v1 4/4] perf script: enable printing of branch stack

2015-08-19 Thread Stephane Eranian
This patch improves perf script by enabling printing of the branch stack via the 'brstack' argument to the field selection option -F. The option is off by default and operates only if the perf.data file has branch stack content. The branches are printed in to/from pairs. the most recent branch is