Merge branch 'perf-fixes-for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/linux-2.6-tip

* 'perf-fixes-for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/linux-2.6-tip:
  perf sched: Fix build failure on sparc
  perf bench: Add "all" pseudo subsystem and "all" pseudo suite
  perf tools: Introduce perf_session class
  perf symbols: Ditch dso->find_symbol
  perf symbols: Allow lookups by symbol name too
  perf symbols: Add missing "Variables" entry to map_type__name
  perf symbols: Add support for 'variable' symtabs
  perf symbols: Introduce ELF counterparts to symbol_type__is_a
  perf symbols: Introduce symbol_type__is_a
  perf symbols: Rename kthreads to kmaps, using another abstraction for it
  perf tools: Allow building for ARM
  hw-breakpoints: Handle bad modify_user_hw_breakpoint off-case return value
  perf tools: Allow cross compiling
  tracing, slab: Fix no callsite ifndef CONFIG_KMEMTRACE
  tracing, slab: Define kmem_cache_alloc_notrace ifdef CONFIG_TRACING

Trivial conflict due to different fixes to modify_user_hw_breakpoint()
in include/linux/hw_breakpoint.h

1 files changed, 54 insertions, 33 deletions

diff --git a/tools/perf/util/map.c b/tools/perf/util/map.c
index 69f94fe9db20..76bdca640a9b 100644
--- a/tools/perf/util/map.c
+++ b/tools/perf/util/map.c
@@ -104,43 +104,64 @@ void map__fixup_end(struct map *self)
 
 #define DSO__DELETED "(deleted)"
 
-struct symbol *map__find_symbol(struct map *self, u64 addr,
-                                symbol_filter_t filter)
+static int map__load(struct map *self, symbol_filter_t filter)
 {
-        if (!dso__loaded(self->dso, self->type)) {
-                int nr = dso__load(self->dso, self, filter);
-
-                if (nr < 0) {
-                        if (self->dso->has_build_id) {
-                                char sbuild_id[BUILD_ID_SIZE * 2 + 1];
-
-                                build_id__sprintf(self->dso->build_id,
-                                                  sizeof(self->dso->build_id),
-                                                  sbuild_id);
-                                pr_warning("%s with build id %s not found",
-                                           self->dso->long_name, sbuild_id);
-                        } else
-                                pr_warning("Failed to open %s",
-                                           self->dso->long_name);
-                        pr_warning(", continuing without symbols\n");
-                        return NULL;
-                } else if (nr == 0) {
-                        const char *name = self->dso->long_name;
-                        const size_t len = strlen(name);
-                        const size_t real_len = len - sizeof(DSO__DELETED);
-
-                        if (len > sizeof(DSO__DELETED) &&
-                            strcmp(name + real_len + 1, DSO__DELETED) == 0) {
-                                pr_warning("%.*s was updated, restart the long running apps that use it!\n",
-                                           (int)real_len, name);
-                        } else {
-                                pr_warning("no symbols found in %s, maybe install a debug package?\n", name);
-                        }
-                        return NULL;
+        const char *name = self->dso->long_name;
+        int nr = dso__load(self->dso, self, filter);
+
+        if (nr < 0) {
+                if (self->dso->has_build_id) {
+                        char sbuild_id[BUILD_ID_SIZE * 2 + 1];
+
+                        build_id__sprintf(self->dso->build_id,
+                                          sizeof(self->dso->build_id),
+                                          sbuild_id);
+                        pr_warning("%s with build id %s not found",
+                                   name, sbuild_id);
+                } else
+                        pr_warning("Failed to open %s", name);
+
+                pr_warning(", continuing without symbols\n");
+                return -1;
+        } else if (nr == 0) {
+                const size_t len = strlen(name);
+                const size_t real_len = len - sizeof(DSO__DELETED);
+
+                if (len > sizeof(DSO__DELETED) &&
+                    strcmp(name + real_len + 1, DSO__DELETED) == 0) {
+                        pr_warning("%.*s was updated, restart the long "
+                                   "running apps that use it!\n",
+                                   (int)real_len, name);
+                } else {
+                        pr_warning("no symbols found in %s, maybe install "
+                                   "a debug package?\n", name);
                 }
+
+                return -1;
         }
 
-        return self->dso->find_symbol(self->dso, self->type, addr);
+        return 0;
+}
+
+struct symbol *map__find_symbol(struct map *self, u64 addr,
+                                symbol_filter_t filter)
+{
+        if (!dso__loaded(self->dso, self->type) && map__load(self, filter) < 0)
+                return NULL;
+
+        return dso__find_symbol(self->dso, self->type, addr);
+}
+
+struct symbol *map__find_symbol_by_name(struct map *self, const char *name,
+                                        symbol_filter_t filter)
+{
+        if (!dso__loaded(self->dso, self->type) && map__load(self, filter) < 0)
+                return NULL;
+
+        if (!dso__sorted_by_name(self->dso, self->type))
+                dso__sort_by_name(self->dso, self->type);
+
+        return dso__find_symbol_by_name(self->dso, self->type, name);
 }
 
 struct map *map__clone(struct map *self)