perf script: Fix unknown feature comment
authorDavid Ahern <dsahern@gmail.com>
Wed, 19 Oct 2011 00:34:24 +0000 (18:34 -0600)
committerArnaldo Carvalho de Melo <acme@redhat.com>
Wed, 19 Oct 2011 15:12:56 +0000 (13:12 -0200)
"perf script -v" emits:

unknown feature 3, continuing...
unknown feature 4, continuing...
unknown feature 5, continuing...
unknown feature 6, continuing...
unknown feature 7, continuing...
unknown feature 8, continuing...
unknown feature 9, continuing...
unknown feature 10, continuing...
unknown feature 11, continuing...
unknown feature 12, continuing...
unknown feature 13, continuing...
unknown feature 14, continuing...

These are all new features added by fbe96f2. Update
perf_file_section__process to know they are valid feature ids.

Cc: Frederic Weisbecker <fweisbec@gmail.com>
Cc: Ingo Molnar <mingo@elte.hu>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Stephane Eranian <eranian@google.com>
Cc: Thomas Gleixner <tglx@linutronix.de>
Link: http://lkml.kernel.org/r/1318984464-20650-1-git-send-email-dsahern@gmail.com
Signed-off-by: David Ahern <dsahern@gmail.com>
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
tools/perf/util/header.c

index 2143a32638c26c8f469e73f5343fafe7bfd0656c..3724707425ea2f8f6885610790316116653d0889 100644 (file)
@@ -1895,6 +1895,21 @@ static int perf_file_section__process(struct perf_file_section *section,
                if (perf_header__read_build_ids(ph, fd, section->offset, section->size))
                        pr_debug("Failed to read buildids, continuing...\n");
                break;
+
+       case HEADER_HOSTNAME:
+       case HEADER_OSRELEASE:
+       case HEADER_VERSION:
+       case HEADER_ARCH:
+       case HEADER_NRCPUS:
+       case HEADER_CPUDESC:
+       case HEADER_CPUID:
+       case HEADER_TOTAL_MEM:
+       case HEADER_CMDLINE:
+       case HEADER_EVENT_DESC:
+       case HEADER_CPU_TOPOLOGY:
+       case HEADER_NUMA_TOPOLOGY:
+               break;
+
        default:
                pr_debug("unknown feature %d, continuing...\n", feat);
        }