Re: [PATCH 1/3] perf sort: Add 'addr' sort key

2013-03-27 Thread Arnaldo Carvalho de Melo
Em Wed, Mar 27, 2013 at 03:12:34PM +0900, Namhyung Kim escreveu: > On Tue, 26 Mar 2013 15:40:23 +0100, Ingo Molnar wrote: > > * Namhyung Kim wrote: > >> New addr sort key provides a way to sort the entries by the symbol > >> addresses. > > No objections from me - just wondering about the motivati

Re: [PATCH 1/3] perf sort: Add 'addr' sort key

2013-03-26 Thread Namhyung Kim
Hi Ingo, On Tue, 26 Mar 2013 15:40:23 +0100, Ingo Molnar wrote: > * Namhyung Kim wrote: > >> From: Namhyung Kim >> >> New addr sort key provides a way to sort the entries by the symbol >> addresses. > > No objections from me - just wondering about the motivation: why would we > want to sort by

Re: [PATCH 1/3] perf sort: Add 'addr' sort key

2013-03-26 Thread Ingo Molnar
* Namhyung Kim wrote: > From: Namhyung Kim > > New addr sort key provides a way to sort the entries by the symbol > addresses. No objections from me - just wondering about the motivation: why would we want to sort by symbol address? Perhaps to see the overhead layout/distribution within a D

[PATCH 1/3] perf sort: Add 'addr' sort key

2013-03-26 Thread Namhyung Kim
From: Namhyung Kim New addr sort key provides a way to sort the entries by the symbol addresses. Suggested-by: Arnaldo Carvalho de Melo Signed-off-by: Namhyung Kim --- tools/perf/util/hist.c | 2 ++ tools/perf/util/hist.h | 1 + tools/perf/util/sort.c | 23 +++ tools/per