diff options
| author | Arnaldo Carvalho de Melo <acme@redhat.com> | 2018-03-15 12:41:39 -0300 |
|---|---|---|
| committer | Arnaldo Carvalho de Melo <acme@redhat.com> | 2018-03-20 13:19:28 -0300 |
| commit | 6af612d2b10593cdd8bba4427741a981bdd57c86 (patch) | |
| tree | 3b51019d57d237a6c196c59c66647176d7772b4d /tools/perf/ui | |
| parent | 16932d77050fb3d76bc265c21c53eeec14639d5e (diff) | |
| download | linux-6af612d2b10593cdd8bba4427741a981bdd57c86.tar.bz2 | |
perf annotate: Move pcnt_with() to the annotation library
Out of the TUI code, since now all it touches is what is in 'struct
annotation'.
Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: Andi Kleen <ak@linux.intel.com>
Cc: David Ahern <dsahern@gmail.com>
Cc: Jin Yao <yao.jin@linux.intel.com>
Cc: Jiri Olsa <jolsa@kernel.org>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Wang Nan <wangnan0@huawei.com>
Link: https://lkml.kernel.org/n/tip-kh5bbbgd7l4agv9oc5hnw0ui@git.kernel.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Diffstat (limited to 'tools/perf/ui')
| -rw-r--r-- | tools/perf/ui/browsers/annotate.c | 14 |
1 files changed, 3 insertions, 11 deletions
diff --git a/tools/perf/ui/browsers/annotate.c b/tools/perf/ui/browsers/annotate.c index b20e0ae72466..3b030ee4505f 100644 --- a/tools/perf/ui/browsers/annotate.c +++ b/tools/perf/ui/browsers/annotate.c @@ -96,13 +96,6 @@ static int annotate_browser__set_jumps_percent_color(struct annotate_browser *br return ui_browser__set_color(&browser->b, color); } -static int annotate_browser__pcnt_width(struct annotate_browser *ab) -{ - struct map_symbol *ms = ab->b.priv; - struct annotation *notes = symbol__annotation(ms->sym); - return (notes->options->show_total_period ? 12 : 7) * notes->nr_events; -} - static void disasm_line__write(struct disasm_line *dl, struct ui_browser *browser, char *bf, size_t size) { @@ -145,7 +138,7 @@ static void annotate_browser__write(struct ui_browser *browser, void *entry, int (!current_entry || (browser->use_navkeypressed && !browser->navkeypressed))); int width = browser->width, printed; - int i, pcnt_width = annotate_browser__pcnt_width(ab), + int i, pcnt_width = annotation__pcnt_width(notes), cycles_width = annotation__cycles_width(notes); double percent_max = 0.0; char bf[256]; @@ -310,7 +303,7 @@ static void annotate_browser__draw_current_jump(struct ui_browser *browser) struct map_symbol *ms = ab->b.priv; struct symbol *sym = ms->sym; struct annotation *notes = symbol__annotation(sym); - u8 pcnt_width = annotate_browser__pcnt_width(ab); + u8 pcnt_width = annotation__pcnt_width(notes); int width; /* PLT symbols contain external offsets */ @@ -375,11 +368,10 @@ static void annotate_browser__draw_current_jump(struct ui_browser *browser) static unsigned int annotate_browser__refresh(struct ui_browser *browser) { - struct annotate_browser *ab = container_of(browser, struct annotate_browser, b); struct map_symbol *ms = browser->priv; struct annotation *notes = symbol__annotation(ms->sym); int ret = ui_browser__list_head_refresh(browser); - int pcnt_width = annotate_browser__pcnt_width(ab); + int pcnt_width = annotation__pcnt_width(notes); if (notes->options->jump_arrows) annotate_browser__draw_current_jump(browser); |