perf tools: Reference count struct map

We have pointers to struct map instances in several places, like in the
hist_entry instances, so we need a way to know when we can destroy them,
otherwise we may either keep leaking them or end up referencing deleted
instances.

Start fixing it by reference counting them.

This patch puts the reference count for struct map in place, replacing
direct map__delete() calls with map__put() ones and then grabbing a
reference count when adding it to the maps struct where maps for a
struct thread are kept.

Next we'll grab reference counts when setting pointers to struct map
instances, in places like in the hist_entry code.

Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: Borislav Petkov <bp@suse.de>
Cc: David Ahern <dsahern@gmail.com>
Cc: Don Zickus <dzickus@redhat.com>
Cc: Frederic Weisbecker <fweisbec@gmail.com>
Cc: Jiri Olsa <jolsa@redhat.com>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Stephane Eranian <eranian@google.com>
Link: http://lkml.kernel.org/n/tip-wi19xczk0t2a41r1i2chuio5@git.kernel.orgSigned-off-by: default avatarArnaldo Carvalho de Melo <acme@redhat.com>
parent facf3f06
...@@ -759,7 +759,6 @@ void machine__destroy_kernel_maps(struct machine *machine) ...@@ -759,7 +759,6 @@ void machine__destroy_kernel_maps(struct machine *machine)
kmap->ref_reloc_sym = NULL; kmap->ref_reloc_sym = NULL;
} }
map__delete(machine->vmlinux_maps[type]);
machine->vmlinux_maps[type] = NULL; machine->vmlinux_maps[type] = NULL;
} }
} }
...@@ -1247,6 +1246,7 @@ int machine__process_mmap2_event(struct machine *machine, ...@@ -1247,6 +1246,7 @@ int machine__process_mmap2_event(struct machine *machine,
thread__insert_map(thread, map); thread__insert_map(thread, map);
thread__put(thread); thread__put(thread);
map__put(map);
return 0; return 0;
out_problem_map: out_problem_map:
...@@ -1297,6 +1297,7 @@ int machine__process_mmap_event(struct machine *machine, union perf_event *event ...@@ -1297,6 +1297,7 @@ int machine__process_mmap_event(struct machine *machine, union perf_event *event
thread__insert_map(thread, map); thread__insert_map(thread, map);
thread__put(thread); thread__put(thread);
map__put(map);
return 0; return 0;
out_problem_map: out_problem_map:
......
...@@ -139,6 +139,7 @@ void map__init(struct map *map, enum map_type type, ...@@ -139,6 +139,7 @@ void map__init(struct map *map, enum map_type type,
map->groups = NULL; map->groups = NULL;
map->referenced = false; map->referenced = false;
map->erange_warned = false; map->erange_warned = false;
atomic_set(&map->refcnt, 1);
} }
struct map *map__new(struct machine *machine, u64 start, u64 len, struct map *map__new(struct machine *machine, u64 start, u64 len,
...@@ -229,6 +230,12 @@ void map__delete(struct map *map) ...@@ -229,6 +230,12 @@ void map__delete(struct map *map)
free(map); free(map);
} }
void map__put(struct map *map)
{
if (map && atomic_dec_and_test(&map->refcnt))
map__delete(map);
}
void map__fixup_start(struct map *map) void map__fixup_start(struct map *map)
{ {
struct rb_root *symbols = &map->dso->symbols[map->type]; struct rb_root *symbols = &map->dso->symbols[map->type];
...@@ -448,7 +455,7 @@ static void __maps__purge(struct maps *maps) ...@@ -448,7 +455,7 @@ static void __maps__purge(struct maps *maps)
next = rb_next(&pos->rb_node); next = rb_next(&pos->rb_node);
rb_erase_init(&pos->rb_node, root); rb_erase_init(&pos->rb_node, root);
map__delete(pos); map__put(pos);
} }
} }
...@@ -458,7 +465,7 @@ static void __maps__purge_removed_maps(struct maps *maps) ...@@ -458,7 +465,7 @@ static void __maps__purge_removed_maps(struct maps *maps)
list_for_each_entry_safe(pos, n, &maps->removed_maps, node) { list_for_each_entry_safe(pos, n, &maps->removed_maps, node) {
list_del_init(&pos->node); list_del_init(&pos->node);
map__delete(pos); map__put(pos);
} }
} }
...@@ -682,7 +689,7 @@ static int maps__fixup_overlappings(struct maps *maps, struct map *map, FILE *fp ...@@ -682,7 +689,7 @@ static int maps__fixup_overlappings(struct maps *maps, struct map *map, FILE *fp
if (before == NULL) { if (before == NULL) {
err = -ENOMEM; err = -ENOMEM;
goto move_map; goto put_map;
} }
before->end = map->start; before->end = map->start;
...@@ -696,7 +703,7 @@ static int maps__fixup_overlappings(struct maps *maps, struct map *map, FILE *fp ...@@ -696,7 +703,7 @@ static int maps__fixup_overlappings(struct maps *maps, struct map *map, FILE *fp
if (after == NULL) { if (after == NULL) {
err = -ENOMEM; err = -ENOMEM;
goto move_map; goto put_map;
} }
after->start = map->end; after->start = map->end;
...@@ -704,14 +711,14 @@ static int maps__fixup_overlappings(struct maps *maps, struct map *map, FILE *fp ...@@ -704,14 +711,14 @@ static int maps__fixup_overlappings(struct maps *maps, struct map *map, FILE *fp
if (verbose >= 2) if (verbose >= 2)
map__fprintf(after, fp); map__fprintf(after, fp);
} }
move_map: put_map:
/* /*
* If we have references, just move them to a separate list. * If we have references, just move them to a separate list.
*/ */
if (pos->referenced) if (pos->referenced)
list_add_tail(&pos->node, &maps->removed_maps); list_add_tail(&pos->node, &maps->removed_maps);
else else
map__delete(pos); map__put(pos);
if (err) if (err)
goto out; goto out;
...@@ -772,6 +779,7 @@ static void __maps__insert(struct maps *maps, struct map *map) ...@@ -772,6 +779,7 @@ static void __maps__insert(struct maps *maps, struct map *map)
rb_link_node(&map->rb_node, parent, p); rb_link_node(&map->rb_node, parent, p);
rb_insert_color(&map->rb_node, &maps->entries); rb_insert_color(&map->rb_node, &maps->entries);
map__get(map);
} }
void maps__insert(struct maps *maps, struct map *map) void maps__insert(struct maps *maps, struct map *map)
...@@ -784,6 +792,7 @@ void maps__insert(struct maps *maps, struct map *map) ...@@ -784,6 +792,7 @@ void maps__insert(struct maps *maps, struct map *map)
static void __maps__remove(struct maps *maps, struct map *map) static void __maps__remove(struct maps *maps, struct map *map)
{ {
rb_erase_init(&map->rb_node, &maps->entries); rb_erase_init(&map->rb_node, &maps->entries);
map__put(map);
} }
void maps__remove(struct maps *maps, struct map *map) void maps__remove(struct maps *maps, struct map *map)
......
...@@ -52,6 +52,7 @@ struct map { ...@@ -52,6 +52,7 @@ struct map {
struct dso *dso; struct dso *dso;
struct map_groups *groups; struct map_groups *groups;
atomic_t refcnt;
}; };
struct kmap { struct kmap {
...@@ -150,6 +151,16 @@ struct map *map__new(struct machine *machine, u64 start, u64 len, ...@@ -150,6 +151,16 @@ struct map *map__new(struct machine *machine, u64 start, u64 len,
struct map *map__new2(u64 start, struct dso *dso, enum map_type type); struct map *map__new2(u64 start, struct dso *dso, enum map_type type);
void map__delete(struct map *map); void map__delete(struct map *map);
struct map *map__clone(struct map *map); struct map *map__clone(struct map *map);
static inline struct map *map__get(struct map *map)
{
if (map)
atomic_inc(&map->refcnt);
return map;
}
void map__put(struct map *map);
int map__overlap(struct map *l, struct map *r); int map__overlap(struct map *l, struct map *r);
size_t map__fprintf(struct map *map, FILE *fp); size_t map__fprintf(struct map *map, FILE *fp);
size_t map__fprintf_dsoname(struct map *map, FILE *fp); size_t map__fprintf_dsoname(struct map *map, FILE *fp);
......
...@@ -195,7 +195,7 @@ static void put_target_map(struct map *map, bool user) ...@@ -195,7 +195,7 @@ static void put_target_map(struct map *map, bool user)
{ {
if (map && user) { if (map && user) {
/* Only the user map needs to be released */ /* Only the user map needs to be released */
map__delete(map); map__put(map);
} }
} }
...@@ -1791,7 +1791,7 @@ static int find_perf_probe_point_from_map(struct probe_trace_point *tp, ...@@ -1791,7 +1791,7 @@ static int find_perf_probe_point_from_map(struct probe_trace_point *tp,
out: out:
if (map && !is_kprobe) { if (map && !is_kprobe) {
map__delete(map); map__put(map);
} }
return ret; return ret;
...@@ -2884,7 +2884,7 @@ int show_available_funcs(const char *target, struct strfilter *_filter, ...@@ -2884,7 +2884,7 @@ int show_available_funcs(const char *target, struct strfilter *_filter,
dso__fprintf_symbols_by_name(map->dso, map->type, stdout); dso__fprintf_symbols_by_name(map->dso, map->type, stdout);
end: end:
if (user) { if (user) {
map__delete(map); map__put(map);
} }
exit_symbol_maps(); exit_symbol_maps();
......
...@@ -972,8 +972,10 @@ int dso__load_sym(struct dso *dso, struct map *map, ...@@ -972,8 +972,10 @@ int dso__load_sym(struct dso *dso, struct map *map,
map->unmap_ip = map__unmap_ip; map->unmap_ip = map__unmap_ip;
/* Ensure maps are correctly ordered */ /* Ensure maps are correctly ordered */
if (kmaps) { if (kmaps) {
map__get(map);
map_groups__remove(kmaps, map); map_groups__remove(kmaps, map);
map_groups__insert(kmaps, map); map_groups__insert(kmaps, map);
map__put(map);
} }
} }
......
...@@ -1180,13 +1180,16 @@ static int dso__load_kcore(struct dso *dso, struct map *map, ...@@ -1180,13 +1180,16 @@ static int dso__load_kcore(struct dso *dso, struct map *map,
map->pgoff = new_map->pgoff; map->pgoff = new_map->pgoff;
map->map_ip = new_map->map_ip; map->map_ip = new_map->map_ip;
map->unmap_ip = new_map->unmap_ip; map->unmap_ip = new_map->unmap_ip;
map__delete(new_map);
/* Ensure maps are correctly ordered */ /* Ensure maps are correctly ordered */
map__get(map);
map_groups__remove(kmaps, map); map_groups__remove(kmaps, map);
map_groups__insert(kmaps, map); map_groups__insert(kmaps, map);
map__put(map);
} else { } else {
map_groups__insert(kmaps, new_map); map_groups__insert(kmaps, new_map);
} }
map__put(new_map);
} }
/* /*
...@@ -1212,7 +1215,7 @@ static int dso__load_kcore(struct dso *dso, struct map *map, ...@@ -1212,7 +1215,7 @@ static int dso__load_kcore(struct dso *dso, struct map *map,
while (!list_empty(&md.maps)) { while (!list_empty(&md.maps)) {
map = list_entry(md.maps.next, struct map, node); map = list_entry(md.maps.next, struct map, node);
list_del_init(&map->node); list_del_init(&map->node);
map__delete(map); map__put(map);
} }
close(fd); close(fd);
return -EINVAL; return -EINVAL;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment