Commit-ID:  f1c5fd4d0bb944da2138338dd361e06ce6c8385e
Gitweb:     http://git.kernel.org/tip/f1c5fd4d0bb944da2138338dd361e06ce6c8385e
Author:     Jiri Olsa <jo...@kernel.org>
AuthorDate: Mon, 2 May 2016 18:30:44 +0200
Committer:  Arnaldo Carvalho de Melo <a...@redhat.com>
CommitDate: Fri, 21 Oct 2016 10:31:55 -0300

perf c2c report: Add TUI cacheline browser

Adding simple TUI cacheline browser. It triggers when you press 'd' in
the main browser on the specific cacheline.

It allows to navigate through cacheline's offsets and display callchains
(implemented in following patches).

Signed-off-by: Jiri Olsa <jo...@kernel.org>
Tested-by: Arnaldo Carvalho de Melo <a...@redhat.com>
Cc: Andi Kleen <a...@firstfloor.org>
Cc: David Ahern <dsah...@gmail.com>
Cc: Don Zickus <dzic...@redhat.com>
Cc: Joe Mario <jma...@redhat.com>
Cc: Namhyung Kim <namhy...@kernel.org>
Cc: Peter Zijlstra <a.p.zijls...@chello.nl>
Link: http://lkml.kernel.org/n/tip-fovjwgyusv3rz5qxk3hna...@git.kernel.org
Signed-off-by: Arnaldo Carvalho de Melo <a...@redhat.com>
---
 tools/perf/builtin-c2c.c | 81 ++++++++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 81 insertions(+)

diff --git a/tools/perf/builtin-c2c.c b/tools/perf/builtin-c2c.c
index 34da2a3..1415640 100644
--- a/tools/perf/builtin-c2c.c
+++ b/tools/perf/builtin-c2c.c
@@ -1829,6 +1829,84 @@ static void c2c_browser__update_nr_entries(struct 
hist_browser *hb)
        hb->nr_non_filtered_entries = nr_entries;
 }
 
+struct c2c_cacheline_browser {
+       struct hist_browser      hb;
+       struct hist_entry       *he;
+};
+
+static int
+perf_c2c_cacheline_browser__title(struct hist_browser *browser,
+                                 char *bf, size_t size)
+{
+       struct c2c_cacheline_browser *cl_browser;
+       struct hist_entry *he;
+       uint64_t addr = 0;
+
+       cl_browser = container_of(browser, struct c2c_cacheline_browser, hb);
+       he = cl_browser->he;
+
+       if (he->mem_info)
+               addr = cl_address(he->mem_info->daddr.addr);
+
+       scnprintf(bf, size, "Cacheline 0x%lx", addr);
+       return 0;
+}
+
+static struct c2c_cacheline_browser*
+c2c_cacheline_browser__new(struct hists *hists, struct hist_entry *he)
+{
+       struct c2c_cacheline_browser *browser;
+
+       browser = zalloc(sizeof(*browser));
+       if (browser) {
+               hist_browser__init(&browser->hb, hists);
+               browser->hb.c2c_filter  = true;
+               browser->hb.title       = perf_c2c_cacheline_browser__title;
+               browser->he             = he;
+       }
+
+       return browser;
+}
+
+static int perf_c2c__browse_cacheline(struct hist_entry *he)
+{
+       struct c2c_hist_entry *c2c_he;
+       struct c2c_hists *c2c_hists;
+       struct c2c_cacheline_browser *cl_browser;
+       struct hist_browser *browser;
+       int key = -1;
+
+       c2c_he = container_of(he, struct c2c_hist_entry, he);
+       c2c_hists = c2c_he->hists;
+
+       cl_browser = c2c_cacheline_browser__new(&c2c_hists->hists, he);
+       if (cl_browser == NULL)
+               return -1;
+
+       browser = &cl_browser->hb;
+
+       /* reset abort key so that it can get Ctrl-C as a key */
+       SLang_reset_tty();
+       SLang_init_tty(0, 0, 0);
+
+       c2c_browser__update_nr_entries(browser);
+
+       while (1) {
+               key = hist_browser__run(browser, "help");
+
+               switch (key) {
+               case 'q':
+                       goto out;
+               default:
+                       break;
+               }
+       }
+
+out:
+       free(cl_browser);
+       return 0;
+}
+
 static int perf_c2c_browser__title(struct hist_browser *browser,
                                   char *bf, size_t size)
 {
@@ -1872,6 +1950,9 @@ static int perf_c2c__hists_browse(struct hists *hists)
                switch (key) {
                case 'q':
                        goto out;
+               case 'd':
+                       perf_c2c__browse_cacheline(browser->he_selection);
+                       break;
                default:
                        break;
                }

Reply via email to