linux-kernel - Re: [RFC PATCH] perf tui: Annotate entries in callchains

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <64EFCD56-C036-4236-B338-3947B99DA862@gmail.com>
Date:	Fri, 20 Mar 2015 14:21:15 -0300
From:	Arnaldo Carvalho de Melo <arnaldo.melo@...il.com>
To:	Stephane Eranian <eranian@...gle.com>
CC:	Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
	Jiri Olsa <jolsa@...nel.org>,
	Peter Zijlstra <peterz@...radead.org>,
	Frederic Weisbecker <fweisbec@...il.com>,
	linux-perf-users@...r.kernel.org, David Ahern <dsahern@...il.com>,
	Namhyung Kim <namhyung@...nel.org>,
	Arnaldo Carvalho de Melo <acme@...nel.org>,
	Ingo Molnar <mingo@...nel.org>
Subject: Re: [RFC PATCH] perf tui: Annotate entries in callchains



On March 20, 2015 2:15:53 PM GMT-03:00, Stephane Eranian <eranian@...gle.com> wrote:
>> Thanks for testing, please let us know if you have further
>suggestions,
>>
>Ok, it does not work.
>I think it works as long as the caller you want to annotate is in the
>same module.
>But suppose, I am on malloc() (libc) and I want to see a caller of
>malloc(), it will
>propose 'annotate bar()', but will still show me the code of
>libc:malloc.
>
>In my earlier test, everything worked because the callee and caller
>were in the
>same module.
>
>Could you fix this?

I'll try.


>
>
>> - Arnaldo
>>
>> Sent from smartphone
>>
>>
>>>
>>> >
>>> > - Arnaldo
>>> >
>>> >
>>> > From: Arnaldo Carvalho de Melo <acme@...hat.com>
>>> > Subject: [PATCH 1/1] perf hists browser: Indicate which callchain
>>> > entries are annotated
>>> >
>>> > Now that we can annotate entries in a callchain, show which ones
>have an
>>> > associated symbol and samples, by adding a right arrow just before
>the
>>> > symbol name when in verbose mode.
>>> >
>>> > To toggle verbose mode press 'V'.
>>> >
>>> > Cc: Adrian Hunter <adrian.hunter@...el.com>
>>> > Cc: Borislav Petkov <bp@...e.de>
>>> > Cc: David Ahern <dsahern@...il.com>
>>> > Cc: Don Zickus <dzickus@...hat.com>
>>> > Cc: Frederic Weisbecker <fweisbec@...il.com>
>>> > Cc: Jiri Olsa <jolsa@...hat.com>
>>> > Cc: Mike Galbraith <efault@....de>
>>> > Cc: Namhyung Kim <namhyung@...nel.org>
>>> > Cc: Peter Zijlstra <peterz@...radead.org>
>>> > Cc: Stephane Eranian <eranian@...gle.com>
>>> > Link:
>>> >
>http://lkml.kernel.org/n/tip-d2rf1p3h5gdp7hdl2gf2bozl@git.kernel.org
>>> > Signed-off-by: Arnaldo Carvalho de Melo <acme@...hat.com>
>>> > ---
>>> >  tools/perf/ui/browsers/hists.c | 4 +++-
>>> >  1 file changed, 3 insertions(+), 1 deletion(-)
>>> >
>>> > diff --git a/tools/perf/ui/browsers/hists.c
>>> > b/tools/perf/ui/browsers/hists.c
>>> > index cd7350aeb8e7..995b7a8596b1 100644
>>> > --- a/tools/perf/ui/browsers/hists.c
>>> > +++ b/tools/perf/ui/browsers/hists.c
>>> > @@ -511,6 +511,7 @@ static void
>>> > hist_browser__show_callchain_entry(struct hist_browser *browser,
>>> >  {
>>> >         int color, width;
>>> >         char folded_sign = callchain_list__folded(chain);
>>> > +       bool show_annotated = browser->show_dso && chain->ms.sym
>&&
>>> > symbol__annotation(chain->ms.sym)->src;
>>> >
>>> >         color = HE_COLORSET_NORMAL;
>>> >         width = browser->b.width - (offset + 2);
>>> > @@ -523,7 +524,8 @@ static void
>>> > hist_browser__show_callchain_entry(struct hist_browser *browser,
>>> >         ui_browser__set_color(&browser->b, color);
>>> >         hist_browser__gotorc(browser, row, 0);
>>> >         slsmg_write_nstring(" ", offset);
>>> > -       slsmg_printf("%c ", folded_sign);
>>> > +       slsmg_printf("%c", folded_sign);
>>> > +       ui_browser__write_graph(&browser->b, show_annotated ?
>>> > SLSMG_RARROW_CHAR : ' ');
>>> >         slsmg_write_nstring(str, width);
>>> >  }
>>> >
>>> > --
>>> > 1.9.3
>>> >

-- Arnaldo

Sent from smartphone.
--
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/