aboutsummaryrefslogtreecommitdiffstats
path: root/tools/perf/builtin-report.c
diff options
context:
space:
mode:
authorArnaldo Carvalho de Melo <acme@redhat.com>2010-04-02 08:50:42 -0400
committerArnaldo Carvalho de Melo <acme@redhat.com>2010-04-02 15:28:28 -0400
commitb9fb93047756c5e4129dfda7591612de61b0e877 (patch)
tree76e66aed31494cdf9f287011bbe1025f21910dac /tools/perf/builtin-report.c
parent71cf8b8ff7d6a79af086be9e4c72628da9d62d58 (diff)
perf hist: Only allocate callchain_node if processing callchains
The struct callchain_node size is 120 bytes, that are never used when there are no callchains or '-g none' is specified, so conditionally allocate it, reducing sizeof(struct hist_entry) from 210 bytes to only 96, greatly speeding the non-callchain processing. LKML-Reference: <new-submission> Cc: Frédéric Weisbecker <fweisbec@gmail.com> Cc: Mike Galbraith <efault@gmx.de> Cc: Peter Zijlstra <a.p.zijlstra@chello.nl> Cc: Paul Mackerras <paulus@samba.org> Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Diffstat (limited to 'tools/perf/builtin-report.c')
-rw-r--r--tools/perf/builtin-report.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/tools/perf/builtin-report.c b/tools/perf/builtin-report.c
index 381918515a5c..1fb13e5fd1f9 100644
--- a/tools/perf/builtin-report.c
+++ b/tools/perf/builtin-report.c
@@ -110,8 +110,8 @@ static int perf_session__add_hist_entry(struct perf_session *self,
110 110
111 if (symbol_conf.use_callchain) { 111 if (symbol_conf.use_callchain) {
112 if (!hit) 112 if (!hit)
113 callchain_init(&he->callchain); 113 callchain_init(he->callchain);
114 err = append_chain(&he->callchain, data->callchain, syms); 114 err = append_chain(he->callchain, data->callchain, syms);
115 free(syms); 115 free(syms);
116 116
117 if (err) 117 if (err)