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: Mon, 20 Oct 2014 19:57:15 +0200 From: Jiri Olsa <jolsa@...hat.com> To: Andi Kleen <andi@...stfloor.org> Cc: linux-kernel@...r.kernel.org, namhyung@...nel.org, acme@...nel.org, Andi Kleen <ak@...ux.intel.com> Subject: Re: [PATCH 3/8] perf, tools: Enable printing the srcline in the history On Fri, Sep 26, 2014 at 04:37:11PM -0700, Andi Kleen wrote: > From: Andi Kleen <ak@...ux.intel.com> SNIP > - if (chain->ms.sym) > - ret += fprintf(fp, " %s\n", chain->ms.sym->name); > - else > - ret += fprintf(fp, " %p\n", > - (void *)(long)chain->ip); > + ret += fprintf(fp, " %s\n", callchain_list__sym_name(chain, > + bf, sizeof(bf), false)); > } > > return ret; > diff --git a/tools/perf/util/callchain.c b/tools/perf/util/callchain.c > index 265457c..b7ee210 100644 > --- a/tools/perf/util/callchain.c > +++ b/tools/perf/util/callchain.c > @@ -655,3 +655,32 @@ int fill_callchain_info(struct addr_location *al, struct callchain_cursor_node * > out: > return 1; > } > + > +char *callchain_list__sym_name(struct callchain_list *cl, > + char *bf, size_t bfsize, bool show_dso) > +{ > + int printed; > + > + if (cl->ms.sym) { > + if (callchain_param.key == CCKEY_ADDRESS && > + cl->ms.map && !cl->srcline) > + cl->srcline = get_srcline(cl->ms.map->dso, > + map__rip_2objdump(cl->ms.map, > + cl->ip)); > + if (cl->srcline) > + printed = scnprintf(bf, bfsize, "%s %s", > + cl->ms.sym->name, cl->srcline); > + else > + printed = scnprintf(bf, bfsize, "%s", > + cl->ms.sym->name); > + } else > + printed = scnprintf(bf, bfsize, "%#" PRIx64, cl->ip); > + > + if (show_dso) > + scnprintf(bf + printed, bfsize - printed, " %s", > + cl->ms.map ? > + cl->ms.map->dso->short_name : > + "unknown"); > + > + return bf; could you please get the callchain_list__sym_name factoring code separated of the new changes you introduce? thanks, jirka -- 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