perf tools: Continue processing header on unknown features
authorRobert Richter <robert.richter@amd.com>
Wed, 7 Dec 2011 09:02:51 +0000 (10:02 +0100)
committerArnaldo Carvalho de Melo <acme@redhat.com>
Fri, 23 Dec 2011 18:57:09 +0000 (16:57 -0200)
A feature may be unknown if perf.data is created and parsed on different
perf tool versions. This should not stop the header to be processed,
instead continue processing it.

Cc: Frederic Weisbecker <fweisbec@gmail.com>
Cc: Ingo Molnar <mingo@elte.hu>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Stephane Eranian <eranian@google.com>
Link: http://lkml.kernel.org/r/1323248577-11268-2-git-send-email-robert.richter@amd.com
Signed-off-by: Robert Richter <robert.richter@amd.com>
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
tools/perf/util/header.c

index 4c48be80dcdf8b654a3f8fd71410e8cb3b940c45..428a4a2ce35028f016d7de71ecbe8f382b575ca0 100644 (file)
@@ -1109,7 +1109,7 @@ static int perf_file_section__fprintf_info(struct perf_file_section *section,
        }
        if (feat < HEADER_TRACE_INFO || feat >= HEADER_LAST_FEATURE) {
                pr_warning("unknown feature %d\n", feat);
-               return -1;
+               return 0;
        }
        if (!feat_ops[feat].print)
                return 0;