diff options
author | Arnaldo Carvalho de Melo <acme@redhat.com> | 2010-04-02 11:30:57 -0400 |
---|---|---|
committer | Arnaldo Carvalho de Melo <acme@redhat.com> | 2010-04-02 15:28:35 -0400 |
commit | 2aefa4f733f2c5ce51dd2316ffecb258463fde71 (patch) | |
tree | b1c0ef5fb2541619630c00d6596a003a11e79512 /tools/perf/builtin-report.c | |
parent | ad5b217b152d99ca3922153500c619d9758dd87a (diff) |
perf tools: sort_dimension__add shouldn't die
Propagate error instead.
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 | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/tools/perf/builtin-report.c b/tools/perf/builtin-report.c index 6767f10615ea..b13a7e2f839e 100644 --- a/tools/perf/builtin-report.c +++ b/tools/perf/builtin-report.c | |||
@@ -473,7 +473,8 @@ int cmd_report(int argc, const char **argv, const char *prefix __used) | |||
473 | setup_sorting(report_usage, options); | 473 | setup_sorting(report_usage, options); |
474 | 474 | ||
475 | if (parent_pattern != default_parent_pattern) { | 475 | if (parent_pattern != default_parent_pattern) { |
476 | sort_dimension__add("parent"); | 476 | if (sort_dimension__add("parent") < 0) |
477 | return -1; | ||
477 | sort_parent.elide = 1; | 478 | sort_parent.elide = 1; |
478 | } else | 479 | } else |
479 | symbol_conf.exclude_other = false; | 480 | symbol_conf.exclude_other = false; |