lkml.org 
[lkml]   [2013]   [Dec]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
From
Subject[PATCH 6/7] perf tools: Do not need to read symbols for source line lookup
Date
When using libbfd to lookup source filename and line number,
libbfd does not need the symbol table. Removing the symbol
table reading, doubles the speed of 'perf script' with the
'srcline' option.

Signed-off-by: Adrian Hunter <adrian.hunter@intel.com>
---
tools/perf/util/srcline.c | 53 +----------------------------------------------
1 file changed, 1 insertion(+), 52 deletions(-)

diff --git a/tools/perf/util/srcline.c b/tools/perf/util/srcline.c
index 93795f9..e09c9e0 100644
--- a/tools/perf/util/srcline.c
+++ b/tools/perf/util/srcline.c
@@ -29,54 +29,6 @@ struct a2l_data {
asymbol **syms;
};

-static int bfd_error(const char *string)
-{
- const char *errmsg;
-
- errmsg = bfd_errmsg(bfd_get_error());
- fflush(stdout);
-
- if (string)
- pr_debug("%s: %s\n", string, errmsg);
- else
- pr_debug("%s\n", errmsg);
-
- return -1;
-}
-
-static int slurp_symtab(bfd *abfd, struct a2l_data *a2l)
-{
- long storage;
- long symcount;
- asymbol **syms;
- bfd_boolean dynamic = FALSE;
-
- if ((bfd_get_file_flags(abfd) & HAS_SYMS) == 0)
- return bfd_error(bfd_get_filename(abfd));
-
- storage = bfd_get_symtab_upper_bound(abfd);
- if (storage == 0L) {
- storage = bfd_get_dynamic_symtab_upper_bound(abfd);
- dynamic = TRUE;
- }
- if (storage < 0L)
- return bfd_error(bfd_get_filename(abfd));
-
- syms = malloc(storage);
- if (dynamic)
- symcount = bfd_canonicalize_dynamic_symtab(abfd, syms);
- else
- symcount = bfd_canonicalize_symtab(abfd, syms);
-
- if (symcount < 0) {
- free(syms);
- return bfd_error(bfd_get_filename(abfd));
- }
-
- a2l->syms = syms;
- return 0;
-}
-
static void find_address_in_section(bfd *abfd, asection *section, void *data)
{
bfd_vma pc, vma;
@@ -96,7 +48,7 @@ static void find_address_in_section(bfd *abfd, asection *section, void *data)
if (pc < vma || pc >= vma + size)
return;

- a2l->found = bfd_find_nearest_line(abfd, section, a2l->syms, pc - vma,
+ a2l->found = bfd_find_nearest_line(abfd, section, NULL, pc - vma,
&a2l->filename, &a2l->funcname,
&a2l->line);
}
@@ -122,9 +74,6 @@ static struct a2l_data *addr2line_init(const char *path)
if (a2l->input == NULL)
goto out;

- if (slurp_symtab(abfd, a2l))
- goto out;
-
return a2l;

out:
--
1.7.11.7


\
 
 \ /
  Last update: 2013-12-03 08:21    [W:0.127 / U:0.136 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site