Re: [PATCH 03/10] perf sort: Add 'addr' sort key
Hi Arnaldo, On Wed, 3 Apr 2013 14:06:10 -0300, Arnaldo Carvalho de Melo wrote: > What I expected was that the result was this: > > perf report --sort addr | grep -v ^# | sort -k2 -n | less > > And in hexadecimal, can you fix this? Oops, it was a mistake in the last minute change, sorry. :( > > Em Wed, Apr 03, 2013 at 09:26:12PM +0900, Namhyung Kim escreveu: >> 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 14c2fe20aa62..9599f805828f 100644 >> --- a/tools/perf/util/hist.h >> +++ b/tools/perf/util/hist.h >> @@ -43,12 +43,13 @@ enum hist_column { >> HISTC_COMM, >> HISTC_PARENT, >> HISTC_CPU, >> +HISTC_SRCLINE, > > Why move SRCLINE? Because it's in common dimension. I'd like to separate it to give a consistent view. Thanks, Namhyung > >> +HISTC_ADDR, >> HISTC_MISPREDICT, >> HISTC_SYMBOL_FROM, >> HISTC_SYMBOL_TO, >> HISTC_DSO_FROM, >> HISTC_DSO_TO, >> -HISTC_SRCLINE, >> HISTC_LOCAL_WEIGHT, >> HISTC_GLOBAL_WEIGHT, >> HISTC_MEM_DADDR_SYMBOL, >> diff --git a/tools/perf/util/sort.c b/tools/perf/util/sort.c >> index 1dbf16949250..5640a95b3575 100644 >> --- a/tools/perf/util/sort.c >> +++ b/tools/perf/util/sort.c >> @@ -342,6 +342,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, "%#*llu", 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 >> @@ -871,6 +899,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 0232d476da87..0815e344f38c 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 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 03/10] perf sort: Add 'addr' sort key
What I expected was that the result was this: perf report --sort addr | grep -v ^# | sort -k2 -n | less And in hexadecimal, can you fix this? - Arnaldo Em Wed, Apr 03, 2013 at 09:26:12PM +0900, Namhyung Kim escreveu: > 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 14c2fe20aa62..9599f805828f 100644 > --- a/tools/perf/util/hist.h > +++ b/tools/perf/util/hist.h > @@ -43,12 +43,13 @@ enum hist_column { > HISTC_COMM, > HISTC_PARENT, > HISTC_CPU, > + HISTC_SRCLINE, Why move SRCLINE? > + HISTC_ADDR, > HISTC_MISPREDICT, > HISTC_SYMBOL_FROM, > HISTC_SYMBOL_TO, > HISTC_DSO_FROM, > HISTC_DSO_TO, > - HISTC_SRCLINE, > HISTC_LOCAL_WEIGHT, > HISTC_GLOBAL_WEIGHT, > HISTC_MEM_DADDR_SYMBOL, > diff --git a/tools/perf/util/sort.c b/tools/perf/util/sort.c > index 1dbf16949250..5640a95b3575 100644 > --- a/tools/perf/util/sort.c > +++ b/tools/perf/util/sort.c > @@ -342,6 +342,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, "%#*llu", 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 > @@ -871,6 +899,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 0232d476da87..0815e344f38c 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 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[PATCH 03/10] perf sort: Add 'addr' sort key
From: Namhyung Kim 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 Buglink: https://bugzilla.kernel.org/show_bug.cgi?id=55561 Signed-off-by: Namhyung Kim --- tools/perf/util/hist.c | 2 ++ tools/perf/util/hist.h | 3 ++- tools/perf/util/sort.c | 29 + tools/perf/util/sort.h | 1 + 4 files changed, 34 insertions(+), 1 deletion(-) 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 14c2fe20aa62..9599f805828f 100644 --- a/tools/perf/util/hist.h +++ b/tools/perf/util/hist.h @@ -43,12 +43,13 @@ enum hist_column { HISTC_COMM, HISTC_PARENT, HISTC_CPU, + HISTC_SRCLINE, + HISTC_ADDR, HISTC_MISPREDICT, HISTC_SYMBOL_FROM, HISTC_SYMBOL_TO, HISTC_DSO_FROM, HISTC_DSO_TO, - HISTC_SRCLINE, HISTC_LOCAL_WEIGHT, HISTC_GLOBAL_WEIGHT, HISTC_MEM_DADDR_SYMBOL, diff --git a/tools/perf/util/sort.c b/tools/perf/util/sort.c index 1dbf16949250..5640a95b3575 100644 --- a/tools/perf/util/sort.c +++ b/tools/perf/util/sort.c @@ -342,6 +342,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, "%#*llu", 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 @@ -871,6 +899,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 0232d476da87..0815e344f38c 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 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/