perf header: Swap pmu mapping numbers if needed
authorNamhyung Kim <namhyung.kim@lge.com>
Wed, 5 Sep 2012 05:02:49 +0000 (14:02 +0900)
committerArnaldo Carvalho de Melo <acme@redhat.com>
Wed, 5 Sep 2012 20:47:11 +0000 (17:47 -0300)
Like others, the numbers can be saved in a different endian format than
a host machine.  Swap them if needed.

Signed-off-by: Namhyung Kim <namhyung@kernel.org>
Cc: Ingo Molnar <mingo@kernel.org>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Cc: Robert Richter <robert.richter@amd.com>
Link: http://lkml.kernel.org/r/1346821373-31621-4-git-send-email-namhyung@kernel.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
tools/perf/util/header.c

index 05c9310..43425b7 100644 (file)
@@ -1440,6 +1440,9 @@ static void print_pmu_mappings(struct perf_header *ph, int fd, FILE *fp)
        if (ret != sizeof(pmu_num))
                goto error;
 
+       if (ph->needs_swap)
+               pmu_num = bswap_32(pmu_num);
+
        if (!pmu_num) {
                fprintf(fp, "# pmu mappings: not available\n");
                return;
@@ -1448,6 +1451,9 @@ static void print_pmu_mappings(struct perf_header *ph, int fd, FILE *fp)
        while (pmu_num) {
                if (read(fd, &type, sizeof(type)) != sizeof(type))
                        break;
+               if (ph->needs_swap)
+                       type = bswap_32(type);
+
                name = do_read_string(fd, ph);
                if (!name)
                        break;