aboutsummaryrefslogtreecommitdiffstats
path: root/tools
diff options
context:
space:
mode:
authorStephane Eranian <eranian@google.com>2012-05-25 17:13:44 -0400
committerArnaldo Carvalho de Melo <acme@redhat.com>2012-05-25 17:32:44 -0400
commita00dc319e98161949aa87f71a17db32e925c3257 (patch)
treee90f081b6bc4a62db6a28b5186d7cedd0d63048c /tools
parentc410431cefefd766266139ed56bca21668e4f9a7 (diff)
perf record: Fix branch_stack type in perf_record_opts
The attr.branch_sample_type field is defined as u64 by the API. As such, we need to ensure the variable holding the value of the branch stack filters is also u64 otherwise we may lose bits in the future. Note also that the bogus definition of the field in perf_record_opts caused problems on big-endian PPC systems. Thanks to Anshuman Khandual for tracking the problem on PPC. Reported-by: Anshuman Khandual <khandual@linux.vnet.ibm.com> Signed-off-by: Stephane Eranian <eranian@google.com> Cc: Anshuman Khandual <khandual@linux.vnet.ibm.com> Cc: Ingo Molnar <mingo@elte.hu> Cc: Peter Zijlstra <peterz@infradead.org> Link: http://lkml.kernel.org/r/20120525211344.GA7729@quad Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Diffstat (limited to 'tools')
-rw-r--r--tools/perf/perf.h2
1 files changed, 1 insertions, 1 deletions
diff --git a/tools/perf/perf.h b/tools/perf/perf.h
index 14f1034f14f9..f960ccb2edc6 100644
--- a/tools/perf/perf.h
+++ b/tools/perf/perf.h
@@ -227,7 +227,7 @@ struct perf_record_opts {
227 unsigned int freq; 227 unsigned int freq;
228 unsigned int mmap_pages; 228 unsigned int mmap_pages;
229 unsigned int user_freq; 229 unsigned int user_freq;
230 int branch_stack; 230 u64 branch_stack;
231 u64 default_interval; 231 u64 default_interval;
232 u64 user_interval; 232 u64 user_interval;
233}; 233};