perf script: Fix symbol insertion behavior in db-export
authorChris Phlipot <cphlipot0@gmail.com>
Wed, 11 May 2016 03:26:47 +0000 (20:26 -0700)
committerArnaldo Carvalho de Melo <acme@redhat.com>
Wed, 11 May 2016 15:24:57 +0000 (12:24 -0300)
Use the dso__insert_symbol function instead of symbols__insert() in
order to properly update the dso symbol cache.

If the cache is not updated, then duplicate symbols can be
unintentionally created, inserted, and exported.

This change prevents duplicate symbols from being exported due to
dso__find_symbol() using a stale symbol cache.

Signed-off-by: Chris Phlipot <cphlipot0@gmail.com>
Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: Peter Zijlstra <peterz@infradead.org>
Link: http://lkml.kernel.org/r/1462937209-6032-3-git-send-email-cphlipot0@gmail.com
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
tools/perf/util/db-export.c

index f8e3057ae3b1903ff7c15f605d0319486c957b30..2ef1f692c35ba8a7070b4b12f7b8187cdd310af4 100644 (file)
@@ -260,8 +260,7 @@ static int db_ids_from_al(struct db_export *dbe, struct addr_location *al,
                if (!al->sym) {
                        al->sym = symbol__new(al->addr, 0, 0, "unknown");
                        if (al->sym)
-                               symbols__insert(&dso->symbols[al->map->type],
-                                               al->sym);
+                               dso__insert_symbol(dso, al->map->type, al->sym);
                }
 
                if (al->sym) {