public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Arnaldo Carvalho de Melo <acme@infradead.org>
To: Jamie Iles <jamie.iles@picochip.com>
Cc: linux-kernel@vger.kernel.org, Jamie Iles <jamie@jamieiles.com>,
	Peter Zijlstra <a.p.zijlstra@chello.nl>,
	Paul Mackerras <paulus@samba.org>
Subject: Re: [PATCH] perf symbols: don't use modules or try vmlinux unless needed
Date: Fri, 8 Jan 2010 09:36:57 -0200	[thread overview]
Message-ID: <20100108113657.GE12660@ghostprotocols.net> (raw)
In-Reply-To: <1262600634-6628-1-git-send-email-jamie.iles@picochip.com>

Em Mon, Jan 04, 2010 at 10:23:54AM +0000, Jamie Iles escreveu:
> Commit 75be6cf (perf symbols: Make symbol_conf global) does what it says
> on the tin, but also initialises the member fields use_modules and
> try_vmlinux_path to true rather than the 'false' value they would have
> had when symbol_conf was static.
> 
> When there is no vmlinux on the system and modules loaded, perf top will
> not show any kernel symbols and perf report will only list raw IP's
> rather than symbol names. Unloading the modules will allow perf to give
> normal output.
> 
> Restore previous behaviour by initialising these fields to false.

James,

	I just tried using perf top here in such conditions:

[root@ana ~]# strace -e open -o /tmp/perf.open.strace perf top
------------------------------------------------------------------------
   PerfTop:      65 irqs/sec kernel:64.6% [1000Hz cycles], (all, 2 CPUs)
------------------------------------------------------------------------

             samples  pcnt function           DSO
             _______ _____ __________________ _________________

               12.00  7.0% format_decode      [kernel.kallsyms]
               10.00  5.8% __strstr_ia32      /lib/libc-2.11.so
                9.00  5.3% read_hpet          [kernel.kallsyms]
                7.00  4.1% module_get_kallsym [kernel.kallsyms]
                5.00  2.9% vsnprintf          [kernel.kallsyms]
                5.00  2.9% __memchr           /lib/libc-2.11.so
/usr/lib/libgtk-x11-2.0.so.0.1800.5.#prelink#.aAFN89 was updated,
restart the long running apps that use it!

[root@ana ~]# grep vmlinux /tmp/perf.open.strace 
open("vmlinux", O_RDONLY|O_LARGEFILE)   = -1 ENOENT (No such file or directory)
open("/boot/vmlinux", O_RDONLY|O_LARGEFILE) = -1 ENOENT (No such file or directory)
open("/boot/vmlinux-2.6.31.9-174.fc12.i686", O_RDONLY|O_LARGEFILE) = -1 ENOENT (No such file or directory)
open("/lib/modules/2.6.31.9-174.fc12.i686/build/vmlinux", O_RDONLY|O_LARGEFILE) = -1 ENOENT (No such file or directory)
open("/usr/lib/debug/lib/modules/2.6.31.9-174.fc12.i686/vmlinux", O_RDONLY|O_LARGEFILE) = -1 ENOENT (No such file or directory)

And it is resolving the kernel symbols.

Maybe some changes I made that aren't yet in Ingo's tree or upstream
fixed it, can you please try after those patches are merged?

- Arnaldo

  reply	other threads:[~2010-01-08 11:37 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-01-04 10:23 [PATCH] perf symbols: don't use modules or try vmlinux unless needed Jamie Iles
2010-01-08 11:36 ` Arnaldo Carvalho de Melo [this message]
2010-01-08 11:54   ` Jamie Iles
     [not found]     ` <20100108123035.GA7485@ghostprotocols.net>
     [not found]       ` <20100108124244.GO4179@wear.picochip.com>
     [not found]         ` <20100108125521.GB7485@ghostprotocols.net>
     [not found]           ` <20100118163332.GA5789@wear.picochip.com>
     [not found]             ` <20100118171035.GD14636@ghostprotocols.net>
     [not found]               ` <20100118202451.GA4167@wear.picochip.com>
     [not found]                 ` <20100119000134.GE14636@ghostprotocols.net>
     [not found]                   ` <20100119000345.GF14636@ghostprotocols.net>
     [not found]                     ` <20100119091231.GC4167@wear.picochip.com>
2010-01-19 13:09                       ` Analysing an ARM perf.data file on a x86-64 workstation Arnaldo Carvalho de Melo
2010-01-19 13:51                         ` Jamie Iles
2010-01-19 14:30                           ` Arnaldo Carvalho de Melo

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=20100108113657.GE12660@ghostprotocols.net \
    --to=acme@infradead.org \
    --cc=a.p.zijlstra@chello.nl \
    --cc=jamie.iles@picochip.com \
    --cc=jamie@jamieiles.com \
    --cc=linux-kernel@vger.kernel.org \
    --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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox