All of lore.kernel.org
 help / color / mirror / Atom feed
* perf report broken for branch stack samples
@ 2015-04-01  9:02 ` Anshuman Khandual
  0 siblings, 0 replies; 4+ messages in thread
From: Anshuman Khandual @ 2015-04-01  9:02 UTC (permalink / raw)
  To: Linux PPC dev, Linux Kernel Mailing List
  Cc: Sukadev Bhattiprolu, Michael Ellerman, Arnaldo Carvalho de Melo, ak

Hello,

perf report is not showing up the branch stack sample results in the
from_symbol ---> to_symbol format even if the perf.data file has got
the samples (through 'perf record -b <workload>' session). Perf report
still displays the generic event based relative symbol classification
as usual. This problem can also be seen on X86. I wonder whether there
is any semantics change in the way perf report auto detects branch stack
samples in the perf.data file and reports in the intended format. After
doing a git bisect session, this following commit seems to have changed
the  perf report behavior.

commit fa94c36c29ed8bb4749b5fd7ea51a593f673dcef
Author: Andi Kleen <ak@linux.intel.com>
Date:   Wed Nov 12 18:05:22 2014 -0800

    perf report: Add --branch-history option
    
    Add a --branch-history option to perf report that changes all the
    settings necessary for using the branches in callstacks.
    
    This is just a short cut to make this nicer to use, it does not enable
    any functionality by itself.
    
    v2: Change sort order. Rename option to --branch-history to
        be less confusing.
    v3: Updates
    v4: Fix conflict with newer perf base
    v5: Port to latest tip
    v6: Add more comments. Remove CCKEY_ADDRESS setting. Remove
        unnecessary branch_mode setting. Use a boolean.
    
    Signed-off-by: Andi Kleen <ak@linux.intel.com>
    Cc: Jiri Olsa <jolsa@redhat.com>
    Cc: Namhyung Kim <namhyung@kernel.org>
    Link: http://lkml.kernel.org/r/1415844328-4884-5-git-send-email-andi@firstfloor.org
    Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>

I dont have much context of this change but it seems to be using branch
records along with call chain data (provided you have used both -b/-j
and -g option during perf record) to display better control flow graph ?
But then if we have just used -b and not -g then it should still give us
the previous output what we used to get.

Regards
Anshuman


^ permalink raw reply	[flat|nested] 4+ messages in thread

* perf report broken for branch stack samples
@ 2015-04-01  9:02 ` Anshuman Khandual
  0 siblings, 0 replies; 4+ messages in thread
From: Anshuman Khandual @ 2015-04-01  9:02 UTC (permalink / raw)
  To: Linux PPC dev, Linux Kernel Mailing List
  Cc: ak, Sukadev Bhattiprolu, Arnaldo Carvalho de Melo, Michael Ellerman

Hello,

perf report is not showing up the branch stack sample results in the
from_symbol ---> to_symbol format even if the perf.data file has got
the samples (through 'perf record -b <workload>' session). Perf report
still displays the generic event based relative symbol classification
as usual. This problem can also be seen on X86. I wonder whether there
is any semantics change in the way perf report auto detects branch stack
samples in the perf.data file and reports in the intended format. After
doing a git bisect session, this following commit seems to have changed
the  perf report behavior.

commit fa94c36c29ed8bb4749b5fd7ea51a593f673dcef
Author: Andi Kleen <ak@linux.intel.com>
Date:   Wed Nov 12 18:05:22 2014 -0800

    perf report: Add --branch-history option
    
    Add a --branch-history option to perf report that changes all the
    settings necessary for using the branches in callstacks.
    
    This is just a short cut to make this nicer to use, it does not enable
    any functionality by itself.
    
    v2: Change sort order. Rename option to --branch-history to
        be less confusing.
    v3: Updates
    v4: Fix conflict with newer perf base
    v5: Port to latest tip
    v6: Add more comments. Remove CCKEY_ADDRESS setting. Remove
        unnecessary branch_mode setting. Use a boolean.
    
    Signed-off-by: Andi Kleen <ak@linux.intel.com>
    Cc: Jiri Olsa <jolsa@redhat.com>
    Cc: Namhyung Kim <namhyung@kernel.org>
    Link: http://lkml.kernel.org/r/1415844328-4884-5-git-send-email-andi@firstfloor.org
    Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>

I dont have much context of this change but it seems to be using branch
records along with call chain data (provided you have used both -b/-j
and -g option during perf record) to display better control flow graph ?
But then if we have just used -b and not -g then it should still give us
the previous output what we used to get.

Regards
Anshuman

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: perf report broken for branch stack samples
  2015-04-01  9:02 ` Anshuman Khandual
@ 2015-04-06 23:35   ` Andi Kleen
  -1 siblings, 0 replies; 4+ messages in thread
From: Andi Kleen @ 2015-04-06 23:35 UTC (permalink / raw)
  To: Anshuman Khandual
  Cc: Linux PPC dev, Linux Kernel Mailing List, Sukadev Bhattiprolu,
	Michael Ellerman, Arnaldo Carvalho de Melo

On Wed, Apr 01, 2015 at 02:32:54PM +0530, Anshuman Khandual wrote:
> Hello,
> 
> perf report is not showing up the branch stack sample results in the
> from_symbol ---> to_symbol format even if the perf.data file has got
> the samples (through 'perf record -b <workload>' session). Perf report

Sorry for the late answer. This was already fixed in tip by 

commit fefd2d9619de3bf0bf02a8622e9f445c3d19cc3f
Author: He Kuang <hekuang@huawei.com>
Date:   Sun Feb 15 10:33:37 2015 +0800

    perf report: Fix branch stack mode cannot be set

-Andi

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: perf report broken for branch stack samples
@ 2015-04-06 23:35   ` Andi Kleen
  0 siblings, 0 replies; 4+ messages in thread
From: Andi Kleen @ 2015-04-06 23:35 UTC (permalink / raw)
  To: Anshuman Khandual
  Cc: Linux PPC dev, Sukadev Bhattiprolu, Arnaldo Carvalho de Melo,
	Linux Kernel Mailing List, Michael Ellerman

On Wed, Apr 01, 2015 at 02:32:54PM +0530, Anshuman Khandual wrote:
> Hello,
> 
> perf report is not showing up the branch stack sample results in the
> from_symbol ---> to_symbol format even if the perf.data file has got
> the samples (through 'perf record -b <workload>' session). Perf report

Sorry for the late answer. This was already fixed in tip by 

commit fefd2d9619de3bf0bf02a8622e9f445c3d19cc3f
Author: He Kuang <hekuang@huawei.com>
Date:   Sun Feb 15 10:33:37 2015 +0800

    perf report: Fix branch stack mode cannot be set

-Andi

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2015-04-06 23:35 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-04-01  9:02 perf report broken for branch stack samples Anshuman Khandual
2015-04-01  9:02 ` Anshuman Khandual
2015-04-06 23:35 ` Andi Kleen
2015-04-06 23:35   ` Andi Kleen

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.