Use g_build_path for DWARF build_id lookup
[babeltrace.git] / lib / debuginfo.c
index 480d5b29763b79eedd91e2b842c993210eeb50af..7d4a0f95dd69f7eab6079e67330312a2601041de 100644 (file)
@@ -4,6 +4,7 @@
  * Copyright (c) 2015 EfficiOS Inc. and Linux Foundation
  * Copyright (c) 2015 Philippe Proulx <pproulx@efficios.com>
  * Copyright (c) 2015 Antoine Busque <abusque@efficios.com>
+ * Copyright (c) 2016 Jérémie Galarneau <jeremie.galarneau@efficios.com>
  *
  * Permission is hereby granted, free of charge, to any person obtaining a copy
  * of this software and associated documentation files (the "Software"), to deal
 #include <babeltrace/ctf-ir/metadata.h>
 #include <babeltrace/debuginfo.h>
 #include <babeltrace/so-info.h>
+#include <babeltrace/babeltrace-internal.h>
+#include <babeltrace/utils.h>
 
 struct proc_debug_info_sources {
        /*
-        * Hash table: base address to so info; owned by
+        * Hash table: base address (pointer to uint64_t) to so info; owned by
         * proc_debug_info_sources.
         */
        GHashTable *baddr_to_so_info;
 
        /*
-        * Hash table: IP to (struct debug_info_source *); owned by
-        * proc_debug_info_sources.
+        * Hash table: IP (pointer to uint64_t) to (struct debug_info_source *);
+        * owned by proc_debug_info_sources.
         */
        GHashTable *ip_to_debug_info_src;
 };
 
 struct debug_info {
        /*
-        * Hash table of VPIDs (int64_t) to (struct ctf_proc_debug_infos*);
-        * owned by debug_info.
+        * Hash table of VPIDs (pointer to int64_t) to
+        * (struct ctf_proc_debug_infos*); owned by debug_info.
         */
        GHashTable *vpid_to_proc_dbg_info_src;
        GQuark q_statedump_soinfo;
@@ -82,7 +85,9 @@ void debug_info_source_destroy(struct debug_info_source *debug_info_src)
        }
 
        free(debug_info_src->func);
-       free(debug_info_src->filename);
+       free(debug_info_src->src_path);
+       free(debug_info_src->bin_path);
+       free(debug_info_src->bin_loc);
        g_free(debug_info_src);
 }
 
@@ -107,30 +112,45 @@ struct debug_info_source *debug_info_source_create_from_so(struct so_info *so,
        }
 
        /* Can't retrieve src_loc from ELF only, skip it */
-       if (so->is_elf_only) {
-               goto end;
-       }
-
-       /* Lookup source location */
-       ret = so_info_lookup_source_location(so, ip, &src_loc);
-       if (ret) {
-               goto error;
+       if (!so->is_elf_only) {
+               /* Lookup source location */
+               ret = so_info_lookup_source_location(so, ip, &src_loc);
+               if (ret) {
+                       goto error;
+               }
        }
 
        if (src_loc) {
                debug_info_src->line_no = src_loc->line_no;
 
                if (src_loc->filename) {
-                       debug_info_src->filename = strdup(src_loc->filename);
-
-                       if (!debug_info_src->filename) {
+                       debug_info_src->src_path = strdup(src_loc->filename);
+                       if (!debug_info_src->src_path) {
                                goto error;
                        }
+
+                       debug_info_src->short_src_path = get_filename_from_path(
+                                       debug_info_src->src_path);
                }
 
                source_location_destroy(src_loc);
        }
 
+       if (so->elf_path) {
+               debug_info_src->bin_path = strdup(so->elf_path);
+               if (!debug_info_src->bin_path) {
+                       goto error;
+               }
+
+               debug_info_src->short_bin_path = get_filename_from_path(
+                               debug_info_src->bin_path);
+
+               ret = so_info_get_bin_loc(so, ip, &(debug_info_src->bin_loc));
+               if (ret) {
+                       goto error;
+               }
+       }
+
 end:
        return debug_info_src;
 
@@ -169,13 +189,14 @@ struct proc_debug_info_sources *proc_debug_info_sources_create(void)
        }
 
        proc_dbg_info_src->baddr_to_so_info = g_hash_table_new_full(
-                       NULL, NULL, NULL, (GDestroyNotify) so_info_destroy);
+                       g_int64_hash, g_int64_equal, (GDestroyNotify) g_free,
+                       (GDestroyNotify) so_info_destroy);
        if (!proc_dbg_info_src->baddr_to_so_info) {
                goto error;
        }
 
        proc_dbg_info_src->ip_to_debug_info_src = g_hash_table_new_full(
-                       NULL, NULL, NULL,
+                       g_int64_hash, g_int64_equal, (GDestroyNotify) g_free,
                        (GDestroyNotify) debug_info_source_destroy);
        if (!proc_dbg_info_src->ip_to_debug_info_src) {
                goto error;
@@ -193,9 +214,15 @@ static
 struct proc_debug_info_sources *proc_debug_info_sources_ht_get_entry(
                GHashTable *ht, int64_t vpid)
 {
-       gpointer key = (gpointer) vpid;
+       gpointer key = g_new0(int64_t, 1);
        struct proc_debug_info_sources *proc_dbg_info_src = NULL;
 
+       if (!key) {
+               goto end;
+       }
+
+       *((int64_t *) key) = vpid;
+
        /* Exists? Return it */
        proc_dbg_info_src = g_hash_table_lookup(ht, key);
        if (proc_dbg_info_src) {
@@ -209,7 +236,10 @@ struct proc_debug_info_sources *proc_debug_info_sources_ht_get_entry(
        }
 
        g_hash_table_insert(ht, key, proc_dbg_info_src);
+       /* Ownership passed to ht */
+       key = NULL;
 end:
+       g_free(key);
        return proc_dbg_info_src;
 }
 
@@ -218,14 +248,20 @@ struct debug_info_source *proc_debug_info_sources_get_entry(
                struct proc_debug_info_sources *proc_dbg_info_src, uint64_t ip)
 {
        struct debug_info_source *debug_info_src = NULL;
-       gint64 key = (gint64) ip;
+       gpointer key = g_new0(uint64_t, 1);
        GHashTableIter iter;
        gpointer baddr, value;
 
+       if (!key) {
+               goto end;
+       }
+
+       *((uint64_t *) key) = ip;
+
        /* Look in IP to debug infos hash table first. */
        debug_info_src = g_hash_table_lookup(
                        proc_dbg_info_src->ip_to_debug_info_src,
-                       (gpointer) key);
+                       key);
        if (debug_info_src) {
                goto end;
        }
@@ -241,17 +277,26 @@ struct debug_info_source *proc_debug_info_sources_get_entry(
                        continue;
                }
 
-               /* Found; add it to cache. */
+               /*
+                * Found; add it to cache.
+                *
+                * FIXME: this should be bounded in size (and implement
+                * a caching policy), and entries should be prunned when
+                * libraries are unmapped.
+                */
                debug_info_src = debug_info_source_create_from_so(so, ip);
                if (debug_info_src) {
                        g_hash_table_insert(
                                        proc_dbg_info_src->ip_to_debug_info_src,
-                                       (gpointer) key, debug_info_src);
+                                       key, debug_info_src);
+                       /* Ownership passed to ht. */
+                       key = NULL;
                }
                break;
        }
 
 end:
+       free(key);
        return debug_info_src;
 }
 
@@ -289,8 +334,8 @@ struct debug_info *debug_info_create(void)
                goto end;
        }
 
-       debug_info->vpid_to_proc_dbg_info_src = g_hash_table_new_full(NULL,
-                       NULL, NULL,
+       debug_info->vpid_to_proc_dbg_info_src = g_hash_table_new_full(
+                       g_int64_hash, g_int64_equal, (GDestroyNotify) g_free,
                        (GDestroyNotify) proc_debug_info_sources_destroy);
        if (!debug_info->vpid_to_proc_dbg_info_src) {
                goto error;
@@ -509,12 +554,13 @@ end:
 }
 
 static
-void handle_statedump_soinfo_event(struct debug_info *debug_info,
-               struct ctf_event_definition *event_def)
+void handle_bin_info_event(struct debug_info *debug_info,
+               struct ctf_event_definition *event_def, bool has_pic_field)
 {
        struct bt_definition *baddr_def = NULL;
        struct bt_definition *memsz_def = NULL;
        struct bt_definition *sopath_def = NULL;
+       struct bt_definition *is_pic_def = NULL;
        struct bt_definition *vpid_def = NULL;
        struct bt_definition *event_fields_def = NULL;
        struct bt_definition *sec_def = NULL;
@@ -523,6 +569,8 @@ void handle_statedump_soinfo_event(struct debug_info *debug_info,
        uint64_t baddr, memsz;
        int64_t vpid;
        const char *sopath;
+       gpointer key = NULL;
+       bool is_pic;
 
        event_fields_def = (struct bt_definition *) event_def->event_fields;
        sec_def = (struct bt_definition *)
@@ -547,6 +595,25 @@ void handle_statedump_soinfo_event(struct debug_info *debug_info,
                goto end;
        }
 
+       if (has_pic_field) {
+               is_pic_def = bt_lookup_definition(event_fields_def, "_is_pic");
+               if (!is_pic_def) {
+                       goto end;
+               }
+
+               if (is_pic_def->declaration->id != CTF_TYPE_INTEGER) {
+                       goto end;
+               }
+
+               is_pic = (bt_get_unsigned_int(is_pic_def) == 1);
+       } else {
+               /*
+                * dlopen has no is_pic field, because the shared
+                * object is always PIC.
+                */
+               is_pic = true;
+       }
+
        vpid_def = bt_lookup_definition(sec_def, "_vpid");
        if (!vpid_def) {
                goto end;
@@ -588,24 +655,49 @@ void handle_statedump_soinfo_event(struct debug_info *debug_info,
                goto end;
        }
 
+       key = g_new0(uint64_t, 1);
+       if (!key) {
+               goto end;
+       }
+
+       *((uint64_t *) key) = baddr;
+
        so = g_hash_table_lookup(proc_dbg_info_src->baddr_to_so_info,
-                       (gpointer) baddr);
+                       key);
        if (so) {
                goto end;
        }
 
-       so = so_info_create(sopath, baddr, memsz);
+       so = so_info_create(sopath, baddr, memsz, is_pic);
        if (!so) {
                goto end;
        }
 
        g_hash_table_insert(proc_dbg_info_src->baddr_to_so_info,
-                       (gpointer) baddr, so);
+                       key, so);
+       /* Ownership passed to ht. */
+       key = NULL;
 
 end:
+       g_free(key);
        return;
 }
 
+static inline
+void handle_soinfo_event(struct debug_info *debug_info,
+               struct ctf_event_definition *event_def)
+{
+       handle_bin_info_event(debug_info, event_def, true);
+}
+
+static inline
+void handle_dlopen_event(struct debug_info *debug_info,
+               struct ctf_event_definition *event_def)
+{
+       handle_bin_info_event(debug_info, event_def, false);
+}
+
+
 static
 void handle_statedump_start(struct debug_info *debug_info,
                struct ctf_event_definition *event_def)
@@ -691,10 +783,11 @@ void debug_info_handle_event(struct debug_info *debug_info,
        event_class = g_ptr_array_index(stream_class->events_by_id,
                        event->stream->event_id);
 
-       if (event_class->name == debug_info->q_statedump_soinfo ||
-                       event_class->name == debug_info->q_dl_open) {
-               /* State dump/dlopen() */
-               handle_statedump_soinfo_event(debug_info, event);
+       if (event_class->name == debug_info->q_statedump_soinfo) {
+               /* State dump */
+               handle_soinfo_event(debug_info, event);
+       } else if (event_class->name == debug_info->q_dl_open) {
+               handle_dlopen_event(debug_info, event);
        } else if (event_class->name == debug_info->q_statedump_start) {
                /* Start state dump */
                handle_statedump_start(debug_info, event);
This page took 0.027336 seconds and 4 git commands to generate.