Hi, This is a couple of bugfix, cleanup and enhancement for sort keys.
When I rebased previous --sort addr* patches, I found that separating sort mode might be helpful to users so I did the cleanup. But during the test I, in turn, found a bug in freeing the branch info of a hist entry. And the mem info patch is only compile-tested - it'd be great if Stephane could take a look and help to test. You can get it from 'perf/sort-v3' branch on my tree at: git://git.kernel.org/pub/scm/linux/kernel/git/namhyung/linux-perf.git Any comments are welcome, thanks. Namhyung Namhyung Kim (9): perf hists: Fix an invalid memory free on he->branch_info perf hists: Free unused mem info of a matched hist entry perf report: Fix alignment of symbol column when -v is given perf sort: Introduce sort__mode variable perf sort: Separate out memory-specific sort keys perf sort: Add 'addr' sort key perf sort: Add 'addr_to/from' sort key perf sort: Update documentation for sort keys perf hists: Move column length setting code tools/perf/Documentation/perf-diff.txt | 2 +- tools/perf/Documentation/perf-report.txt | 5 +- tools/perf/Documentation/perf-top.txt | 2 +- tools/perf/builtin-diff.c | 2 +- tools/perf/builtin-report.c | 40 ++++++---- tools/perf/builtin-top.c | 3 +- tools/perf/ui/browsers/hists.c | 4 +- tools/perf/util/hist.c | 63 ++++++++++----- tools/perf/util/hist.h | 5 +- tools/perf/util/sort.c | 129 ++++++++++++++++++++++++++++--- tools/perf/util/sort.h | 30 ++++--- 11 files changed, 223 insertions(+), 62 deletions(-) -- 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/