Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1239823
| From | Jiri Olsa <jolsa@kernel.org> |
|---|---|
| Newsgroups | linux.kernel |
| Subject | [PATCH 7/9] perf tools: Add support for sorting on the iaddr |
| Date | 2015-10-05 20:10 +0200 |
| Message-ID | <qgin0-6vn-41@gated-at.bofh.it> (permalink) |
| References | <qgimZ-6vn-11@gated-at.bofh.it> |
| Organization | linux.* mail to news gateway |
From: Don Zickus <dzickus@redhat.com>
Sorting on 'symbol' gives to broad a resolution as it can cover a
range of IP address. Use the iaddr instead to get proper sorting on
IP addresses. Need to use the 'mem_sort' feature of perf record.
New sort option is: symbol_iaddr, header label is 'Code Symbol'.
$ perf mem report --stdio -F +symbol_iaddr
# Overhead Samples Code Symbol Local Weight
# ........ ............ ........................ ............
#
54.08% 1 [k] nmi_handle 192
4.51% 1 [k] finish_task_switch 16
3.66% 1 [.] malloc 13
3.10% 1 [.] __strcoll_l 11
Signed-off-by: Don Zickus <dzickus@redhat.com>
Link: http://lkml.kernel.org/n/tip-94rmgtp0lvpkc7xmb9bmrjxm@git.kernel.org
Signed-off-by: Jiri Olsa <jolsa@kernel.org>
---
tools/perf/util/hist.h | 1 +
tools/perf/util/sort.c | 37 +++++++++++++++++++++++++++++++++++++
tools/perf/util/sort.h | 1 +
3 files changed, 39 insertions(+)
diff --git a/tools/perf/util/hist.h b/tools/perf/util/hist.h
index 8c20a8f6b214..a48a2078d288 100644
--- a/tools/perf/util/hist.h
+++ b/tools/perf/util/hist.h
@@ -49,6 +49,7 @@ enum hist_column {
HISTC_MEM_LVL,
HISTC_MEM_SNOOP,
HISTC_MEM_DCACHELINE,
+ HISTC_MEM_IADDR_SYMBOL,
HISTC_TRANSACTION,
HISTC_CYCLES,
HISTC_NR_COLS, /* Last entry */
diff --git a/tools/perf/util/sort.c b/tools/perf/util/sort.c
index 6b9556d298c9..ee94b728fca4 100644
--- a/tools/perf/util/sort.c
+++ b/tools/perf/util/sort.c
@@ -655,6 +655,35 @@ static int hist_entry__daddr_snprintf(struct hist_entry *he, char *bf,
}
static int64_t
+sort__iaddr_cmp(struct hist_entry *left, struct hist_entry *right)
+{
+ uint64_t l = 0, r = 0;
+
+ if (left->mem_info)
+ l = left->mem_info->iaddr.addr;
+ if (right->mem_info)
+ r = right->mem_info->iaddr.addr;
+
+ return (int64_t)(r - l);
+}
+
+static int hist_entry__iaddr_snprintf(struct hist_entry *he, char *bf,
+ size_t size, unsigned int width)
+{
+ uint64_t addr = 0;
+ struct map *map = NULL;
+ struct symbol *sym = NULL;
+
+ if (he->mem_info) {
+ addr = he->mem_info->iaddr.addr;
+ map = he->mem_info->iaddr.map;
+ sym = he->mem_info->iaddr.sym;
+ }
+ return _hist_entry__sym_snprintf(map, sym, addr, he->level, bf, size,
+ width);
+}
+
+static int64_t
sort__dso_daddr_cmp(struct hist_entry *left, struct hist_entry *right)
{
struct map *map_l = NULL;
@@ -1077,6 +1106,13 @@ struct sort_entry sort_mem_daddr_sym = {
.se_width_idx = HISTC_MEM_DADDR_SYMBOL,
};
+struct sort_entry sort_mem_iaddr_sym = {
+ .se_header = "Code Symbol",
+ .se_cmp = sort__iaddr_cmp,
+ .se_snprintf = hist_entry__iaddr_snprintf,
+ .se_width_idx = HISTC_MEM_IADDR_SYMBOL,
+};
+
struct sort_entry sort_mem_daddr_dso = {
.se_header = "Data Object",
.se_cmp = sort__dso_daddr_cmp,
@@ -1299,6 +1335,7 @@ static struct sort_dimension bstack_sort_dimensions[] = {
static struct sort_dimension memory_sort_dimensions[] = {
DIM(SORT_MEM_DADDR_SYMBOL, "symbol_daddr", sort_mem_daddr_sym),
+ DIM(SORT_MEM_IADDR_SYMBOL, "symbol_iaddr", sort_mem_iaddr_sym),
DIM(SORT_MEM_DADDR_DSO, "dso_daddr", sort_mem_daddr_dso),
DIM(SORT_MEM_LOCKED, "locked", sort_mem_locked),
DIM(SORT_MEM_TLB, "tlb", sort_mem_tlb),
diff --git a/tools/perf/util/sort.h b/tools/perf/util/sort.h
index c06b75746613..33b3d30e18d3 100644
--- a/tools/perf/util/sort.h
+++ b/tools/perf/util/sort.h
@@ -201,6 +201,7 @@ enum sort_type {
SORT_MEM_LVL,
SORT_MEM_SNOOP,
SORT_MEM_DCACHELINE,
+ SORT_MEM_IADDR_SYMBOL,
};
/*
--
2.4.3
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread
[PATCH 0/9] perf tools: Various fixes Jiri Olsa <jolsa@kernel.org> - 2015-10-05 20:10 +0200
[PATCH 6/9] perf tests: Add parsing test for 'P' modifier Jiri Olsa <jolsa@kernel.org> - 2015-10-05 20:10 +0200
[tip:perf/core] perf tests: Add parsing test for 'P' modifier tip-bot for Jiri Olsa <tipbot@zytor.com> - 2015-10-06 09:20 +0200
[PATCH 1/9] tools lib api fs: Do not use PATH_MAX + 1 Jiri Olsa <jolsa@kernel.org> - 2015-10-05 20:10 +0200
[tip:perf/core] tools lib api fs: No need to use PATH_MAX + 1 tip-bot for Jiri Olsa <tipbot@zytor.com> - 2015-10-06 09:20 +0200
[PATCH 5/9] perf tools: Introduce 'P' modifier Jiri Olsa <jolsa@kernel.org> - 2015-10-05 20:10 +0200
Re: [PATCH 5/9] perf tools: Introduce 'P' modifier David Ahern <dsahern@gmail.com> - 2015-10-05 20:20 +0200
Re: [PATCH 5/9] perf tools: Introduce 'P' modifier Jiri Olsa <jolsa@redhat.com> - 2015-10-05 20:40 +0200
Re: [PATCH 5/9] perf tools: Introduce 'P' modifier Arnaldo Carvalho de Melo <acme@kernel.org> - 2015-10-05 21:10 +0200
Re: [PATCH 5/9] perf tools: Introduce 'P' modifier Arnaldo Carvalho de Melo <acme@kernel.org> - 2015-10-05 21:30 +0200
Re: [PATCH 5/9] perf tools: Introduce 'P' modifier Jiri Olsa <jolsa@redhat.com> - 2015-10-06 09:10 +0200
[tip:perf/core] perf tools: Introduce 'P' modifier to request max precision tip-bot for Jiri Olsa <tipbot@zytor.com> - 2015-10-06 09:20 +0200
[PATCH 7/9] perf tools: Add support for sorting on the iaddr Jiri Olsa <jolsa@kernel.org> - 2015-10-05 20:10 +0200
[tip:perf/core] perf tools: Add support for sorting on the iaddr tip-bot for Don Zickus <tipbot@zytor.com> - 2015-10-06 09:20 +0200
[PATCH 8/9] perf tools: Setup proper width for symbol_iaddr field Jiri Olsa <jolsa@kernel.org> - 2015-10-05 20:10 +0200
[tip:perf/core] perf tools: Setup proper width for symbol_iaddr field tip-bot for Jiri Olsa <tipbot@zytor.com> - 2015-10-06 09:20 +0200
[PATCH 2/9] perf evlist: Display DATA_SRC sample type bit Jiri Olsa <jolsa@kernel.org> - 2015-10-05 20:10 +0200
[tip:perf/core] perf evlist: Display DATA_SRC sample type bit tip-bot for Jiri Olsa <tipbot@zytor.com> - 2015-10-06 09:20 +0200
csiph-web