Commit 21916c38 authored by Dave Martin's avatar Dave Martin Committed by Arnaldo Carvalho de Melo

perf tools: Factor out buildid reading and make it implicit in dso__load

If we have a buildid, then we never want to load an image which has no buildid,
or which has a different buildid, so it makes sense for the check to be built
into dso__load and not done separately.  This is fine for old distros which
don't use buildid at all since we do no check in that case.

This refactoring also alleviates some subtle race condition issues by not
opening ELF images twice to check the buildid and then load the symbols, which
could lead to weirdness if an image is replaced under our feet.
Signed-off-by: default avatarDave Martin <dave.martin@linaro.org>
LKML-Reference: <new-submission>
Signed-off-by: default avatarArnaldo Carvalho de Melo <acme@redhat.com>
parent 6e406257
...@@ -26,6 +26,8 @@ ...@@ -26,6 +26,8 @@
#define NT_GNU_BUILD_ID 3 #define NT_GNU_BUILD_ID 3
#endif #endif
static bool dso__build_id_equal(const struct dso *self, u8 *build_id);
static int elf_read_build_id(Elf *elf, void *bf, size_t size);
static void dsos__add(struct list_head *head, struct dso *dso); static void dsos__add(struct list_head *head, struct dso *dso);
static struct map *map__new2(u64 start, struct dso *dso, enum map_type type); static struct map *map__new2(u64 start, struct dso *dso, enum map_type type);
static int dso__load_kernel_sym(struct dso *self, struct map *map, static int dso__load_kernel_sym(struct dso *self, struct map *map,
...@@ -993,6 +995,17 @@ static int dso__load_sym(struct dso *self, struct map *map, const char *name, ...@@ -993,6 +995,17 @@ static int dso__load_sym(struct dso *self, struct map *map, const char *name,
goto out_elf_end; goto out_elf_end;
} }
if (self->has_build_id) {
u8 build_id[BUILD_ID_SIZE];
if (elf_read_build_id(elf, build_id,
BUILD_ID_SIZE) != BUILD_ID_SIZE)
goto out_elf_end;
if (!dso__build_id_equal(self, build_id))
goto out_elf_end;
}
sec = elf_section_by_name(elf, &ehdr, &shdr, ".symtab", NULL); sec = elf_section_by_name(elf, &ehdr, &shdr, ".symtab", NULL);
if (sec == NULL) { if (sec == NULL) {
sec = elf_section_by_name(elf, &ehdr, &shdr, ".dynsym", NULL); sec = elf_section_by_name(elf, &ehdr, &shdr, ".dynsym", NULL);
...@@ -1193,37 +1206,26 @@ bool __dsos__read_build_ids(struct list_head *head, bool with_hits) ...@@ -1193,37 +1206,26 @@ bool __dsos__read_build_ids(struct list_head *head, bool with_hits)
*/ */
#define NOTE_ALIGN(n) (((n) + 3) & -4U) #define NOTE_ALIGN(n) (((n) + 3) & -4U)
int filename__read_build_id(const char *filename, void *bf, size_t size) static int elf_read_build_id(Elf *elf, void *bf, size_t size)
{ {
int fd, err = -1; int err = -1;
GElf_Ehdr ehdr; GElf_Ehdr ehdr;
GElf_Shdr shdr; GElf_Shdr shdr;
Elf_Data *data; Elf_Data *data;
Elf_Scn *sec; Elf_Scn *sec;
Elf_Kind ek; Elf_Kind ek;
void *ptr; void *ptr;
Elf *elf;
if (size < BUILD_ID_SIZE) if (size < BUILD_ID_SIZE)
goto out; goto out;
fd = open(filename, O_RDONLY);
if (fd < 0)
goto out;
elf = elf_begin(fd, PERF_ELF_C_READ_MMAP, NULL);
if (elf == NULL) {
pr_debug2("%s: cannot read %s ELF file.\n", __func__, filename);
goto out_close;
}
ek = elf_kind(elf); ek = elf_kind(elf);
if (ek != ELF_K_ELF) if (ek != ELF_K_ELF)
goto out_elf_end; goto out;
if (gelf_getehdr(elf, &ehdr) == NULL) { if (gelf_getehdr(elf, &ehdr) == NULL) {
pr_err("%s: cannot get elf header.\n", __func__); pr_err("%s: cannot get elf header.\n", __func__);
goto out_elf_end; goto out;
} }
sec = elf_section_by_name(elf, &ehdr, &shdr, sec = elf_section_by_name(elf, &ehdr, &shdr,
...@@ -1232,12 +1234,12 @@ int filename__read_build_id(const char *filename, void *bf, size_t size) ...@@ -1232,12 +1234,12 @@ int filename__read_build_id(const char *filename, void *bf, size_t size)
sec = elf_section_by_name(elf, &ehdr, &shdr, sec = elf_section_by_name(elf, &ehdr, &shdr,
".notes", NULL); ".notes", NULL);
if (sec == NULL) if (sec == NULL)
goto out_elf_end; goto out;
} }
data = elf_getdata(sec, NULL); data = elf_getdata(sec, NULL);
if (data == NULL) if (data == NULL)
goto out_elf_end; goto out;
ptr = data->d_buf; ptr = data->d_buf;
while (ptr < (data->d_buf + data->d_size)) { while (ptr < (data->d_buf + data->d_size)) {
...@@ -1259,7 +1261,31 @@ int filename__read_build_id(const char *filename, void *bf, size_t size) ...@@ -1259,7 +1261,31 @@ int filename__read_build_id(const char *filename, void *bf, size_t size)
} }
ptr += descsz; ptr += descsz;
} }
out_elf_end:
out:
return err;
}
int filename__read_build_id(const char *filename, void *bf, size_t size)
{
int fd, err = -1;
Elf *elf;
if (size < BUILD_ID_SIZE)
goto out;
fd = open(filename, O_RDONLY);
if (fd < 0)
goto out;
elf = elf_begin(fd, PERF_ELF_C_READ_MMAP, NULL);
if (elf == NULL) {
pr_debug2("%s: cannot read %s ELF file.\n", __func__, filename);
goto out_close;
}
err = elf_read_build_id(elf, bf, size);
elf_end(elf); elf_end(elf);
out_close: out_close:
close(fd); close(fd);
...@@ -1335,7 +1361,6 @@ int dso__load(struct dso *self, struct map *map, symbol_filter_t filter) ...@@ -1335,7 +1361,6 @@ int dso__load(struct dso *self, struct map *map, symbol_filter_t filter)
{ {
int size = PATH_MAX; int size = PATH_MAX;
char *name; char *name;
u8 build_id[BUILD_ID_SIZE];
int ret = -1; int ret = -1;
int fd; int fd;
struct machine *machine; struct machine *machine;
...@@ -1382,16 +1407,14 @@ int dso__load(struct dso *self, struct map *map, symbol_filter_t filter) ...@@ -1382,16 +1407,14 @@ int dso__load(struct dso *self, struct map *map, symbol_filter_t filter)
self->long_name); self->long_name);
break; break;
case DSO__ORIG_BUILDID: case DSO__ORIG_BUILDID:
if (filename__read_build_id(self->long_name, build_id, if (self->has_build_id) {
sizeof(build_id))) {
char build_id_hex[BUILD_ID_SIZE * 2 + 1]; char build_id_hex[BUILD_ID_SIZE * 2 + 1];
build_id__sprintf(build_id, sizeof(build_id), build_id__sprintf(self->build_id,
sizeof(self->build_id),
build_id_hex); build_id_hex);
snprintf(name, size, snprintf(name, size,
"/usr/lib/debug/.build-id/%.2s/%s.debug", "/usr/lib/debug/.build-id/%.2s/%s.debug",
build_id_hex, build_id_hex + 2); build_id_hex, build_id_hex + 2);
if (self->has_build_id)
goto compare_build_id;
break; break;
} }
self->origin++; self->origin++;
...@@ -1410,15 +1433,6 @@ int dso__load(struct dso *self, struct map *map, symbol_filter_t filter) ...@@ -1410,15 +1433,6 @@ int dso__load(struct dso *self, struct map *map, symbol_filter_t filter)
default: default:
goto out; goto out;
} }
if (self->has_build_id) {
if (filename__read_build_id(name, build_id,
sizeof(build_id)) < 0)
goto more;
compare_build_id:
if (!dso__build_id_equal(self, build_id))
goto more;
}
open_file: open_file:
fd = open(name, O_RDONLY); fd = open(name, O_RDONLY);
} while (fd < 0); } while (fd < 0);
......
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