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

[Arnaldo Carvalho de Melo]

On March 20, 2015 2:15:53 PM GMT-03:00, Stephane Eranian <eranian@xxxxxxxxxx> 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@xxxxxxxxxx>
>>> > 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@xxxxxxxxx>
>>> > Cc: Borislav Petkov <bp@xxxxxxx>
>>> > Cc: David Ahern <dsahern@xxxxxxxxx>
>>> > Cc: Don Zickus <dzickus@xxxxxxxxxx>
>>> > Cc: Frederic Weisbecker <fweisbec@xxxxxxxxx>
>>> > Cc: Jiri Olsa <jolsa@xxxxxxxxxx>
>>> > Cc: Mike Galbraith <efault@xxxxxx>
>>> > Cc: Namhyung Kim <namhyung@xxxxxxxxxx>
>>> > Cc: Peter Zijlstra <peterz@xxxxxxxxxxxxx>
>>> > Cc: Stephane Eranian <eranian@xxxxxxxxxx>
>>> > Link:
>>> >
>http://lkml.kernel.org/n/tip-d2rf1p3h5gdp7hdl2gf2bozl@xxxxxxxxxxxxxx
>>> > Signed-off-by: Arnaldo Carvalho de Melo <acme@xxxxxxxxxx>
>>> > ---
>>> >  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@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/