diff options
| author | Arnaldo Carvalho de Melo <acme@redhat.com> | 2018-03-15 21:52:33 +0300 |
|---|---|---|
| committer | Arnaldo Carvalho de Melo <acme@redhat.com> | 2018-03-20 19:19:29 +0300 |
| commit | 1cf5f98a5edbb4b628349bea37da0aa259ded07c (patch) | |
| tree | fe35bf958df850b2b6eb2a752361fb026ad33401 | |
| parent | 0ca693b315aa6893c9d3552fa6a5d536c38b6c4a (diff) | |
| download | linux-1cf5f98a5edbb4b628349bea37da0aa259ded07c.tar.xz | |
perf annotate: Move nr_{asm_}entries to struct annotation
More non-TUI stuff.
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-yd4g6q0rngq4i49hz6iymtta@git.kernel.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
| -rw-r--r-- | tools/perf/ui/browsers/annotate.c | 23 | ||||
| -rw-r--r-- | tools/perf/util/annotate.h | 2 |
2 files changed, 13 insertions, 12 deletions
diff --git a/tools/perf/ui/browsers/annotate.c b/tools/perf/ui/browsers/annotate.c index 7ec441f93f7e..00b88349a3c2 100644 --- a/tools/perf/ui/browsers/annotate.c +++ b/tools/perf/ui/browsers/annotate.c @@ -35,8 +35,6 @@ struct annotate_browser { struct rb_node *curr_hot; struct annotation_line *selection; struct arch *arch; - int nr_asm_entries; - int nr_entries; bool searching_backwards; u8 addr_width; u8 jumps_width; @@ -470,7 +468,7 @@ static bool annotate_browser__toggle_source(struct annotate_browser *browser) if (al->idx_asm < offset) offset = al->idx; - browser->b.nr_entries = browser->nr_entries; + browser->b.nr_entries = notes->nr_entries; notes->options->hide_src_code = false; browser->b.seek(&browser->b, -offset, SEEK_CUR); browser->b.top_idx = al->idx - offset; @@ -485,7 +483,7 @@ static bool annotate_browser__toggle_source(struct annotate_browser *browser) if (al->idx_asm < offset) offset = al->idx_asm; - browser->b.nr_entries = browser->nr_asm_entries; + browser->b.nr_entries = notes->nr_asm_entries; notes->options->hide_src_code = true; browser->b.seek(&browser->b, -offset, SEEK_CUR); browser->b.top_idx = al->idx_asm - offset; @@ -495,10 +493,11 @@ static bool annotate_browser__toggle_source(struct annotate_browser *browser) return true; } -static void annotate_browser__init_asm_mode(struct annotate_browser *browser) +static void ui_browser__init_asm_mode(struct ui_browser *browser) { - ui_browser__reset_index(&browser->b); - browser->b.nr_entries = browser->nr_asm_entries; + struct annotation *notes = browser__annotation(browser); + ui_browser__reset_index(browser); + browser->nr_entries = notes->nr_asm_entries; } #define SYM_TITLE_MAX_SIZE (PATH_MAX + 64) @@ -854,7 +853,7 @@ show_help: browser->b.height, browser->b.index, browser->b.top_idx, - browser->nr_asm_entries); + notes->nr_asm_entries); } continue; case K_ENTER: @@ -997,9 +996,9 @@ int symbol__tui_annotate(struct symbol *sym, struct map *map, if (browser.b.width < line_len) browser.b.width = line_len; - al->idx = browser.nr_entries++; + al->idx = notes->nr_entries++; if (al->offset != -1) { - al->idx_asm = browser.nr_asm_entries++; + al->idx_asm = notes->nr_asm_entries++; /* * FIXME: short term bandaid to cope with assembly * routines that comes with labels in the same column @@ -1020,12 +1019,12 @@ int symbol__tui_annotate(struct symbol *sym, struct map *map, browser.max_addr_width = hex_width(sym->end); browser.jumps_width = width_jumps(notes->max_jump_sources); notes->nr_events = nr_pcnt; - browser.b.nr_entries = browser.nr_entries; + browser.b.nr_entries = notes->nr_entries; browser.b.entries = ¬es->src->source, browser.b.width += 18; /* Percentage */ if (notes->options->hide_src_code) - annotate_browser__init_asm_mode(&browser); + ui_browser__init_asm_mode(&browser.b); annotate_browser__update_addr_width(&browser); diff --git a/tools/perf/util/annotate.h b/tools/perf/util/annotate.h index 0424c127b004..0c34eb0bd7c8 100644 --- a/tools/perf/util/annotate.h +++ b/tools/perf/util/annotate.h @@ -175,6 +175,8 @@ struct annotation { int nr_events; int nr_jumps; int max_jump_sources; + int nr_entries; + int nr_asm_entries; bool have_cycles; struct annotated_source *src; }; |