All of lore.kernel.org
 help / color / mirror / Atom feed
From: Frederic Weisbecker <fweisbec@gmail.com>
To: Ingo Molnar <mingo@elte.hu>
Cc: LKML <linux-kernel@vger.kernel.org>,
	Peter Zijlstra <a.p.zijlstra@chello.nl>,
	Mike Galbraith <efault@gmx.de>, Paul Mackerras <paulus@samba.org>
Subject: Re: [PATCH 1/2] perfcounters: Print the filename:line for annotated colored lines
Date: Sat, 13 Jun 2009 17:26:46 +0200	[thread overview]
Message-ID: <20090613152644.GA4911@nowhere> (raw)
In-Reply-To: <1244906732-7521-1-git-send-email-fweisbec@gmail.com>

On Sat, Jun 13, 2009 at 05:25:31PM +0200, Frederic Weisbecker wrote:
> When we have a colored line in perf annotate, ie a middle/high
> overhead one, it's sometimes useful to get the matching line
> and filename from the source file, especially this path prepares
> to another subsequent one which will print a sorted summary of
> midle/high overhead lines in the beginning of the output.
> 
> Filename:Lines have the same color than the concerned ip lines.
> 
> It can be slow because it relies on addr2line. We could also
> use objdump with -l but that implies we would have to bufferize
> objdump output and parse it to filter the relevant lines since
> we want to print a sorted summary in the beginning.
> 
> Changes in v2:
> - fix addr2line on userspace binary
> - fix string allocation size (missing ending null char room)
> - fix overflow in symbol extra info
> 
> Signed-off-by: Frederic Weisbecker <fweisbec@gmail.com>


Forgot to put "v2" in the [PATCH] token. It fixes what you reported
Ingo, and also some other things.

Thanks.


  parent reply	other threads:[~2009-06-13 15:26 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-06-13 15:25 [PATCH 1/2] perfcounters: Print the filename:line for annotated colored lines Frederic Weisbecker
2009-06-13 15:25 ` [PATCH 2/2] perfcounters: print a sorted summary of annotated overhead lines Frederic Weisbecker
2009-06-13 15:26 ` Frederic Weisbecker [this message]
  -- strict thread matches above, loose matches on Subject: below --
2009-06-12 22:11 [PATCH 1/2] perfcounters: Print the filename:line for annotated colored lines Frederic Weisbecker
2009-06-13 11:12 ` Ingo Molnar
2009-06-13 12:35   ` Frederic Weisbecker

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20090613152644.GA4911@nowhere \
    --to=fweisbec@gmail.com \
    --cc=a.p.zijlstra@chello.nl \
    --cc=efault@gmx.de \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@elte.hu \
    --cc=paulus@samba.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.