diff options
author | Namhyung Kim <namhyung.kim@lge.com> | 2013-10-31 02:56:03 -0400 |
---|---|---|
committer | Arnaldo Carvalho de Melo <acme@redhat.com> | 2013-11-04 18:59:09 -0500 |
commit | 41a4e6e2a0237e8ac895f43158ef7c91ab7af157 (patch) | |
tree | 3ab45245e866404b7bc05b494e6770fc580d408a /tools/perf/builtin-diff.c | |
parent | 87968f94fbea47df334502a0db645833ce8a848b (diff) |
perf hists: Consolidate __hists__add_*entry()
The __hists__add_{branch,mem}_entry() does almost the same thing that
__hists__add_entry() does. Consolidate them into one.
Signed-off-by: Namhyung Kim <namhyung@kernel.org>
Acked-by: Jiri Olsa <jolsa@redhat.com>
Cc: Frederic Weisbecker <fweisbec@gmail.com>
Cc: Ingo Molnar <mingo@kernel.org>
Cc: Jiri Olsa <jolsa@redhat.com>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Cc: Rodrigo Campos <rodrigo@sdfg.com.ar>
Cc: Stephane Eranian <eranian@google.com>
Link: http://lkml.kernel.org/r/1383202576-28141-2-git-send-email-namhyung@kernel.org
[ Fixup clash with new COMM infrastructure ]
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Diffstat (limited to 'tools/perf/builtin-diff.c')
-rw-r--r-- | tools/perf/builtin-diff.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/tools/perf/builtin-diff.c b/tools/perf/builtin-diff.c index b605009e803f..3b67ea2444bd 100644 --- a/tools/perf/builtin-diff.c +++ b/tools/perf/builtin-diff.c | |||
@@ -307,7 +307,8 @@ static int hists__add_entry(struct hists *hists, | |||
307 | struct addr_location *al, u64 period, | 307 | struct addr_location *al, u64 period, |
308 | u64 weight, u64 transaction) | 308 | u64 weight, u64 transaction) |
309 | { | 309 | { |
310 | if (__hists__add_entry(hists, al, NULL, period, weight, transaction) != NULL) | 310 | if (__hists__add_entry(hists, al, NULL, NULL, NULL, period, weight, |
311 | transaction) != NULL) | ||
311 | return 0; | 312 | return 0; |
312 | return -ENOMEM; | 313 | return -ENOMEM; |
313 | } | 314 | } |