diff options
author | Arnaldo Carvalho de Melo <acme@redhat.com> | 2018-03-15 21:04:33 +0300 |
---|---|---|
committer | Arnaldo Carvalho de Melo <acme@redhat.com> | 2018-03-20 19:19:28 +0300 |
commit | 95aa89d92de8b8da6dc1469bfc560784ab177aba (patch) | |
tree | e827e4b0e4a28d7dc3bb4cef59451da0c61bdb83 /tools/perf | |
parent | 6af612d2b10593cdd8bba4427741a981bdd57c86 (diff) | |
download | linux-95aa89d92de8b8da6dc1469bfc560784ab177aba.tar.xz |
perf annotate tui: Add browser__annotation() helper
To reduce the boilerplate to get to the symbol being annotated from the
struct browser ->priv area.
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-ficdyqhe9esjseflvkriskwn@git.kernel.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Diffstat (limited to 'tools/perf')
-rw-r--r-- | tools/perf/ui/browsers/annotate.c | 40 |
1 files changed, 16 insertions, 24 deletions
diff --git a/tools/perf/ui/browsers/annotate.c b/tools/perf/ui/browsers/annotate.c index 3b030ee4505f..c995d28d1a58 100644 --- a/tools/perf/ui/browsers/annotate.c +++ b/tools/perf/ui/browsers/annotate.c @@ -55,6 +55,12 @@ struct annotate_browser { char search_bf[128]; }; +static inline struct annotation *browser__annotation(struct ui_browser *browser) +{ + struct map_symbol *ms = browser->priv; + return symbol__annotation(ms->sym); +} + static inline struct browser_line *browser_line(struct annotation_line *al) { void *ptr = al; @@ -65,8 +71,7 @@ static inline struct browser_line *browser_line(struct annotation_line *al) static bool disasm_line__filter(struct ui_browser *browser, void *entry) { - struct map_symbol *ms = browser->priv; - struct annotation *notes = symbol__annotation(ms->sym); + struct annotation *notes = browser__annotation(browser); if (notes->options->hide_src_code) { struct annotation_line *al = list_entry(entry, struct annotation_line, node); @@ -99,8 +104,7 @@ static int annotate_browser__set_jumps_percent_color(struct annotate_browser *br static void disasm_line__write(struct disasm_line *dl, struct ui_browser *browser, char *bf, size_t size) { - struct map_symbol *ms = browser->priv; - struct annotation *notes = symbol__annotation(ms->sym); + struct annotation *notes = browser__annotation(browser); if (dl->ins.ops && dl->ins.ops->scnprintf) { if (ins__is_jump(&dl->ins)) { @@ -128,9 +132,7 @@ static void disasm_line__write(struct disasm_line *dl, struct ui_browser *browse static void annotate_browser__write(struct ui_browser *browser, void *entry, int row) { struct annotate_browser *ab = container_of(browser, struct annotate_browser, b); - struct map_symbol *ms = browser->priv; - struct symbol *sym = ms->sym; - struct annotation *notes = symbol__annotation(sym); + struct annotation *notes = browser__annotation(browser); struct annotation_line *al = list_entry(entry, struct annotation_line, node); struct browser_line *bl = browser_line(al); bool current_entry = ui_browser__is_current_entry(browser, row); @@ -368,8 +370,7 @@ static void annotate_browser__draw_current_jump(struct ui_browser *browser) static unsigned int annotate_browser__refresh(struct ui_browser *browser) { - struct map_symbol *ms = browser->priv; - struct annotation *notes = symbol__annotation(ms->sym); + struct annotation *notes = browser__annotation(browser); int ret = ui_browser__list_head_refresh(browser); int pcnt_width = annotation__pcnt_width(notes); @@ -438,8 +439,7 @@ static void annotate_browser__set_top(struct annotate_browser *browser, static void annotate_browser__set_rb_top(struct annotate_browser *browser, struct rb_node *nd) { - struct map_symbol *ms = browser->b.priv; - struct annotation *notes = symbol__annotation(ms->sym); + struct annotation *notes = browser__annotation(&browser->b); struct browser_line *bpos; struct annotation_line *pos; u32 idx; @@ -497,8 +497,7 @@ static void annotate_browser__calc_percent(struct annotate_browser *browser, static bool annotate_browser__toggle_source(struct annotate_browser *browser) { - struct map_symbol *ms = browser->b.priv; - struct annotation *notes = symbol__annotation(ms->sym); + struct annotation *notes = browser__annotation(&browser->b); struct annotation_line *al; struct browser_line *bl; off_t offset = browser->b.index - browser->b.top_idx; @@ -588,9 +587,7 @@ static struct disasm_line *annotate_browser__find_offset(struct annotate_browser *browser, s64 offset, s64 *idx) { - struct map_symbol *ms = browser->b.priv; - struct symbol *sym = ms->sym; - struct annotation *notes = symbol__annotation(sym); + struct annotation *notes = browser__annotation(&browser->b); struct disasm_line *pos; *idx = 0; @@ -629,9 +626,7 @@ static struct annotation_line *annotate_browser__find_string(struct annotate_browser *browser, char *s, s64 *idx) { - struct map_symbol *ms = browser->b.priv; - struct symbol *sym = ms->sym; - struct annotation *notes = symbol__annotation(sym); + struct annotation *notes = browser__annotation(&browser->b); struct annotation_line *al = browser->selection; *idx = browser->b.index; @@ -668,9 +663,7 @@ static struct annotation_line *annotate_browser__find_string_reverse(struct annotate_browser *browser, char *s, s64 *idx) { - struct map_symbol *ms = browser->b.priv; - struct symbol *sym = ms->sym; - struct annotation *notes = symbol__annotation(sym); + struct annotation *notes = browser__annotation(&browser->b); struct annotation_line *al = browser->selection; *idx = browser->b.index; @@ -753,8 +746,7 @@ bool annotate_browser__continue_search_reverse(struct annotate_browser *browser, static void annotate_browser__update_addr_width(struct annotate_browser *browser) { - struct map_symbol *ms = browser->b.priv; - struct annotation *notes = symbol__annotation(ms->sym); + struct annotation *notes = browser__annotation(&browser->b); if (notes->options->use_offset) browser->target_width = browser->min_addr_width; |