perf tests: Check proper prev_state size for sched_switch tp
authorJiri Olsa <jolsa@redhat.com>
Fri, 12 Jul 2013 14:29:06 +0000 (16:29 +0200)
committerArnaldo Carvalho de Melo <acme@redhat.com>
Wed, 17 Jul 2013 15:34:53 +0000 (12:34 -0300)
The prev_state is defined as long which is 4 bytes long on 32-bit x86.
Changing the check against sizeof(long).

Signed-off-by: Jiri Olsa <jolsa@redhat.com>
Cc: Corey Ashford <cjashfor@linux.vnet.ibm.com>
Cc: Frederic Weisbecker <fweisbec@gmail.com>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Link: http://lkml.kernel.org/r/1373639346-4547-1-git-send-email-jolsa@redhat.com
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
tools/perf/tests/evsel-tp-sched.c

index f79e7d059820d85f7f5ad38dc8c1a83d067e1dc7..9b98c1554833ede7de9bdc8d7d312352ea863486 100644 (file)
@@ -49,7 +49,7 @@ int test__perf_evsel__tp_sched_test(void)
        if (perf_evsel__test_field(evsel, "prev_prio", 4, true))
                ret = -1;
 
-       if (perf_evsel__test_field(evsel, "prev_state", 8, true))
+       if (perf_evsel__test_field(evsel, "prev_state", sizeof(long), true))
                ret = -1;
 
        if (perf_evsel__test_field(evsel, "next_comm", 16, true))