All of lore.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 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.