diff options
-rw-r--r-- | tools/perf/builtin-record.c | 15 |
1 files changed, 9 insertions, 6 deletions
diff --git a/tools/perf/builtin-record.c b/tools/perf/builtin-record.c index e382d93d369c..ac989e9ba8fe 100644 --- a/tools/perf/builtin-record.c +++ b/tools/perf/builtin-record.c | |||
@@ -666,12 +666,15 @@ static int __cmd_record(int argc, const char **argv) | |||
666 | nr_counters, | 666 | nr_counters, |
667 | process_synthesized_event, | 667 | process_synthesized_event, |
668 | session); | 668 | session); |
669 | if (err <= 0) { | 669 | /* |
670 | pr_err("Couldn't record tracing data.\n"); | 670 | * FIXME err <= 0 here actually means that there were no tracepoints |
671 | return err; | 671 | * so its not really an error, just that we don't need to synthesize |
672 | } | 672 | * anything. |
673 | 673 | * We really have to return this more properly and also propagate | |
674 | advance_output(err); | 674 | * errors that now are calling die() |
675 | */ | ||
676 | if (err > 0) | ||
677 | advance_output(err); | ||
675 | } | 678 | } |
676 | 679 | ||
677 | machine = perf_session__find_host_machine(session); | 680 | machine = perf_session__find_host_machine(session); |