| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Date: Sat, 16 May 2015 21:09:51 +0900
From: Masami Hiramatsu <masami.hiramatsu.pt@...achi.com>
To: Arnaldo Carvalho de Melo <acme@...nel.org>
CC: Peter Zijlstra <peterz@...radead.org>,
Richard Weinberger <richard@....at>,
Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
David Ahern <dsahern@...il.com>, namhyung@...nel.org,
Jiri Olsa <jolsa@...hat.com>, Ingo Molnar <mingo@...nel.org>
Subject: Re: [PATCH perf/core ] perf probe: Show the error reason comes from
invalid DSO
Arnaldo, could you also apply this?
This will helps users to find the reason why perf probe failed
to open the debuginfo.
On 2015/05/12 12:40, Masami Hiramatsu wrote:
> Show the reason of error when dso__load* failed. This shows
> when user gives wrong kernel image or wrong path.
>
> Without this, perf probe shows an obscure message.
> ----
> $ perf probe -k ~/kbin/linux-3.x86_64/vmlinux -L vfs_read
> Failed to find path of kernel module.
> Error: Failed to show lines.
> ----
>
> With this, perf shows appropriate error message.
> ----
> $ perf probe -k ~/kbin/linux-3.x86_64/vmlinux -L vfs_read
> Failed to find the path for kernel: Mismatching build id
> Error: Failed to show lines.
> ----
> And
> ----
> $ perf probe -k /non-exist/kernel/vmlinux -L vfs_read
> Failed to find the path for kernel: No such file or directory
> Error: Failed to show lines.
> ----
>
> Signed-off-by: Masami Hiramatsu <masami.hiramatsu.pt@...achi.com>
> ---
> tools/perf/util/probe-event.c | 47 +++++++++++++++++++++--------------------
> tools/perf/util/probe-event.h | 3 ---
> 2 files changed, 24 insertions(+), 26 deletions(-)
>
> diff --git a/tools/perf/util/probe-event.c b/tools/perf/util/probe-event.c
> index a2d8cef..a4830a6 100644
> --- a/tools/perf/util/probe-event.c
> +++ b/tools/perf/util/probe-event.c
> @@ -201,11 +201,12 @@ static void put_target_map(struct map *map, bool user)
> }
>
>
> -static struct dso *kernel_get_module_dso(const char *module)
> +static int kernel_get_module_dso(const char *module, struct dso **pdso)
> {
> struct dso *dso;
> struct map *map;
> const char *vmlinux_name;
> + int ret = 0;
>
> if (module) {
> list_for_each_entry(dso, &host_machine->kernel_dsos.head,
> @@ -215,30 +216,21 @@ static struct dso *kernel_get_module_dso(const char *module)
> goto found;
> }
> pr_debug("Failed to find module %s.\n", module);
> - return NULL;
> + return -ENOENT;
> }
>
> map = host_machine->vmlinux_maps[MAP__FUNCTION];
> dso = map->dso;
>
> vmlinux_name = symbol_conf.vmlinux_name;
> - if (vmlinux_name) {
> - if (dso__load_vmlinux(dso, map, vmlinux_name, false, NULL) <= 0)
> - return NULL;
> - } else {
> - if (dso__load_vmlinux_path(dso, map, NULL) <= 0) {
> - pr_debug("Failed to load kernel map.\n");
> - return NULL;
> - }
> - }
> + dso->load_errno = 0;
> + if (vmlinux_name)
> + ret = dso__load_vmlinux(dso, map, vmlinux_name, false, NULL);
> + else
> + ret = dso__load_vmlinux_path(dso, map, NULL);
> found:
> - return dso;
> -}
> -
> -const char *kernel_get_module_path(const char *module)
> -{
> - struct dso *dso = kernel_get_module_dso(module);
> - return (dso) ? dso->long_name : NULL;
> + *pdso = dso;
> + return ret;
> }
>
> static int convert_exec_to_group(const char *exec, char **result)
> @@ -390,16 +382,25 @@ static int get_alternative_line_range(struct debuginfo *dinfo,
> static struct debuginfo *open_debuginfo(const char *module, bool silent)
> {
> const char *path = module;
> - struct debuginfo *ret;
> + char reason[STRERR_BUFSIZE];
> + struct debuginfo *ret = NULL;
> + struct dso *dso = NULL;
> + int err;
>
> if (!module || !strchr(module, '/')) {
> - path = kernel_get_module_path(module);
> - if (!path) {
> + err = kernel_get_module_dso(module, &dso);
> + if (err < 0) {
> + if (!dso || dso->load_errno == 0) {
> + if (!strerror_r(-err, reason, STRERR_BUFSIZE))
> + strcpy(reason, "(unknown)");
> + } else
> + dso__strerror_load(dso, reason, STRERR_BUFSIZE);
> if (!silent)
> - pr_err("Failed to find path of %s module.\n",
> - module ?: "kernel");
> + pr_err("Failed to find the path for %s: %s\n",
> + module ?: "kernel", reason);
> return NULL;
> }
> + path = dso->long_name;
> }
> ret = debuginfo__new(path);
> if (!ret && !silent) {
> diff --git a/tools/perf/util/probe-event.h b/tools/perf/util/probe-event.h
> index 537eb32..31db6ee 100644
> --- a/tools/perf/util/probe-event.h
> +++ b/tools/perf/util/probe-event.h
> @@ -131,9 +131,6 @@ extern void line_range__clear(struct line_range *lr);
> /* Initialize line range */
> extern int line_range__init(struct line_range *lr);
>
> -/* Internal use: Return kernel/module path */
> -extern const char *kernel_get_module_path(const char *module);
> -
> extern int add_perf_probe_events(struct perf_probe_event *pevs, int npevs);
> extern int del_perf_probe_events(struct strfilter *filter);
> extern int show_perf_probe_events(struct strfilter *filter);
>
>
--
Masami HIRAMATSU
Linux Technology Research Center, System Productivity Research Dept.
Center for Technology Innovation - Systems Engineering
Hitachi, Ltd., Research & Development Group
E-mail: masami.hiramatsu.pt@...achi.com
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists