iv/linux
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

perf annotate-data: Collect global variables in advance

Browse Source 
Currently it looks up global variables from the current CU using address
and name.  But it sometimes fails to find a variable as the variable can
come from a different CU - but it's still strange it failed to find a
declaration for some reason.

Anyway, it can collect all global variables from all CU once and then
lookup them later on.  This slightly improves the success rate of my
test data set.

Signed-off-by: Namhyung Kim <namhyung@kernel.org>
Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: Ian Rogers <irogers@google.com>
Cc: Ingo Molnar <mingo@kernel.org>
Cc: Jiri Olsa <jolsa@kernel.org>
Cc: Kan Liang <kan.liang@linux.intel.com>
Cc: Peter Zijlstra <peterz@infradead.org>
Link: https://lore.kernel.org/r/20240502060011.1838090-3-namhyung@kernel.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
This commit is contained in:
Namhyung Kim 2024-05-01 23:00:07 -07:00 committed by Arnaldo Carvalho de Melo
parent d7b60803a7
commit c1da8411e4
1 changed files with 55 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

57
tools/perf/util/annotate-data.c
View File

@ -28,6 +28,8 @@
/* register number of the stack pointer */
#define X86_REG_SP 7
static void delete_var_types(struct die_var_type *var_types);
enum type_state_kind {
TSR_KIND_INVALID = 0,
TSR_KIND_TYPE,
@ -557,8 +559,8 @@ static bool global_var__add(struct data_loc_info *dloc, u64 addr,
if (gvar == NULL)
return false;
gvar->name = strdup(name);
if (gvar->name == NULL) {
gvar->name = name ? strdup(name) : NULL;
if (name && gvar->name == NULL) {
free(gvar);
return false;
}
@ -612,6 +614,53 @@ static bool get_global_var_info(struct data_loc_info *dloc, u64 addr,
return true;
}
static void global_var__collect(struct data_loc_info *dloc)
{
Dwarf *dwarf = dloc->di->dbg;
Dwarf_Off off, next_off;
Dwarf_Die cu_die, type_die;
size_t header_size;
/* Iterate all CU and collect global variables that have no location in a register. */
off = 0;
while (dwarf_nextcu(dwarf, off, &next_off, &header_size,
NULL, NULL, NULL) == 0) {
struct die_var_type *var_types = NULL;
struct die_var_type *pos;
if (dwarf_offdie(dwarf, off + header_size, &cu_die) == NULL) {
off = next_off;
continue;
}
die_collect_global_vars(&cu_die, &var_types);
for (pos = var_types; pos; pos = pos->next) {
const char *var_name = NULL;
int var_offset = 0;
if (pos->reg != -1)
continue;
if (!dwarf_offdie(dwarf, pos->die_off, &type_die))
continue;
if (!get_global_var_info(dloc, pos->addr, &var_name,
&var_offset))
continue;
if (var_offset != 0)
continue;
global_var__add(dloc, pos->addr, var_name, &type_die);
}
delete_var_types(var_types);
off = next_off;
}
}
static bool get_global_var_type(Dwarf_Die *cu_die, struct data_loc_info *dloc,
u64 ip, u64 var_addr, int *var_offset,
Dwarf_Die *type_die)
@ -620,8 +669,12 @@ static bool get_global_var_type(Dwarf_Die *cu_die, struct data_loc_info *dloc,
int offset;
const char *var_name = NULL;
struct global_var_entry *gvar;
struct dso *dso = map__dso(dloc->ms->map);
Dwarf_Die var_die;
if (RB_EMPTY_ROOT(&dso->global_vars))
global_var__collect(dloc);
gvar = global_var__find(dloc, var_addr);
if (gvar) {
if (!dwarf_offdie(dloc->di->dbg, gvar->die_offset, type_die))