diff options
author | Arnaldo Carvalho de Melo <acme@redhat.com> | 2010-04-02 08:50:42 -0400 |
---|---|---|
committer | Arnaldo Carvalho de Melo <acme@redhat.com> | 2010-04-02 15:28:28 -0400 |
commit | b9fb93047756c5e4129dfda7591612de61b0e877 (patch) | |
tree | 76e66aed31494cdf9f287011bbe1025f21910dac /tools/perf/builtin-report.c | |
parent | 71cf8b8ff7d6a79af086be9e4c72628da9d62d58 (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.c | 4 |
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) |