Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1495121

Re: [PATCH 1/2] perf intel-pt-decoder: Report instruction bytes and length in sample

From Andi Kleen <andi@firstfloor.org>
Newsgroups linux.kernel
Subject Re: [PATCH 1/2] perf intel-pt-decoder: Report instruction bytes and length in sample
Date 2016-10-04 02:40 +0200
Message-ID <solW2-3vV-9@gated-at.bofh.it> (permalink)
References <smX9n-4UQ-3@gated-at.bofh.it> <sn358-vu-17@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


On Fri, Sep 30, 2016 at 01:07:17PM +0300, Adrian Hunter wrote:
> On 30/09/16 06:49, Andi Kleen wrote:
> > From: Andi Kleen <ak@linux.intel.com>
> > 
> > Change the Intel PT decoder to pass up the length and the instruction
> > bytes of the decoded or sampled instruction in the perf sample.
> > 
> > The decoder already knows this information, we just need to pass it
> > up. Since it is only a couple of movs it is not very expensive.
> 
> The decoder doesn't always fill the instruction buffer because it caches the
> results from walking basic blocks.  That means the bytes from the last
> instruction in the basic block would need to be added to the cache.

I fixed this and posted a new version.
> 
> I wonder if we shouldn't look at something more sophisticated e.g. a python
> script that can do full disassembly.

I had an earlier version that used a disassembler library, but that
wasn't appreciated, so now switched to this simpler method.
Assembler output can be done with a sed + objdump hack now.

-Andi

Back to linux.kernel | Previous | NextPrevious in thread | Find similar | Unroll thread


Thread

[PATCH 1/2] perf intel-pt-decoder: Report instruction bytes and length in sample Andi Kleen <andi@firstfloor.org> - 2016-09-30 06:00 +0200
  Re: [PATCH 1/2] perf intel-pt-decoder: Report instruction bytes and  length in sample Adrian Hunter <adrian.hunter@intel.com> - 2016-09-30 12:20 +0200
    Re: [PATCH 1/2] perf intel-pt-decoder: Report instruction bytes and  length in sample Andi Kleen <andi@firstfloor.org> - 2016-10-04 02:40 +0200

csiph-web