Messages in this thread Patch in this message | | | From | Namhyung Kim <> | Subject | [PATCH 2/8] perf sort: Add 'addr' sort key | Date | Fri, 5 Apr 2013 10:26:32 +0900 |
| |
From: Namhyung Kim <namhyung.kim@lge.com>
New addr sort key provides a way to sort the entries by the symbol addresses. It can be helpful to figure out symbol resolution problem when a dso cannot do it properly as well as finding hotpath in a dso and/or a function.
Suggested-by: Arnaldo Carvalho de Melo <acme@ghostprotocols.net> Buglink: https://bugzilla.kernel.org/show_bug.cgi?id=55561 Signed-off-by: Namhyung Kim <namhyung@kernel.org> --- tools/perf/util/hist.c | 2 ++ tools/perf/util/hist.h | 1 + tools/perf/util/sort.c | 29 +++++++++++++++++++++++++++++ tools/perf/util/sort.h | 1 + 4 files changed, 33 insertions(+)
diff --git a/tools/perf/util/hist.c b/tools/perf/util/hist.c index 72b4eec820c3..c098d6ebab1f 100644 --- a/tools/perf/util/hist.c +++ b/tools/perf/util/hist.c @@ -52,6 +52,8 @@ void hists__reset_col_len(struct hists *hists) for (col = 0; col < HISTC_NR_COLS; ++col) hists__set_col_len(hists, col, 0); + + hists__set_col_len(hists, HISTC_ADDR, BITS_PER_LONG / 4 + 2); } static void hists__set_unres_dso_col_len(struct hists *hists, int dso) diff --git a/tools/perf/util/hist.h b/tools/perf/util/hist.h index 6be88dc12b9a..9599f805828f 100644 --- a/tools/perf/util/hist.h +++ b/tools/perf/util/hist.h @@ -44,6 +44,7 @@ enum hist_column { HISTC_PARENT, HISTC_CPU, HISTC_SRCLINE, + HISTC_ADDR, HISTC_MISPREDICT, HISTC_SYMBOL_FROM, HISTC_SYMBOL_TO, diff --git a/tools/perf/util/sort.c b/tools/perf/util/sort.c index 701ab1d84894..1543ab124052 100644 --- a/tools/perf/util/sort.c +++ b/tools/perf/util/sort.c @@ -343,6 +343,34 @@ struct sort_entry sort_cpu = { .se_width_idx = HISTC_CPU, }; +/* --sort addr */ + +static int64_t +sort__addr_cmp(struct hist_entry *left, struct hist_entry *right) +{ + return right->ip - left->ip; +} + +static int hist_entry__addr_snprintf(struct hist_entry *self, char *bf, + size_t size, unsigned int width) +{ + struct map *map = self->ms.map; + u64 addr = self->ip; + + if (map) + addr = map->unmap_ip(map, self->ip); + + return repsep_snprintf(bf, size, "%#*llx", width, addr); +} + +struct sort_entry sort_addr = { + .se_header = "Address", + .se_cmp = sort__addr_cmp, + .se_snprintf = hist_entry__addr_snprintf, + .se_width_idx = HISTC_ADDR, +}; + + /* sort keys for branch stacks */ static int64_t @@ -872,6 +900,7 @@ static struct sort_dimension common_sort_dimensions[] = { DIM(SORT_PARENT, "parent", sort_parent), DIM(SORT_CPU, "cpu", sort_cpu), DIM(SORT_SRCLINE, "srcline", sort_srcline), + DIM(SORT_ADDR, "addr", sort_addr), }; #undef DIM diff --git a/tools/perf/util/sort.h b/tools/perf/util/sort.h index 51f1b5a854e7..57a6fc1a1f7f 100644 --- a/tools/perf/util/sort.h +++ b/tools/perf/util/sort.h @@ -138,6 +138,7 @@ enum sort_type { SORT_PARENT, SORT_CPU, SORT_SRCLINE, + SORT_ADDR, /* branch stack specific sort keys */ __SORT_BRANCH_STACK, -- 1.7.11.7
| |