Commit a00dc319e98161949aa87f71a17db32e925c3257

Authored by Stephane Eranian
Committed by Arnaldo Carvalho de Melo
1 parent c410431cef

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>

Showing 1 changed file with 1 additions and 1 deletions Side-by-side Diff

... ... @@ -227,7 +227,7 @@
227 227 unsigned int freq;
228 228 unsigned int mmap_pages;
229 229 unsigned int user_freq;
230   - int branch_stack;
  230 + u64 branch_stack;
231 231 u64 default_interval;
232 232 u64 user_interval;
233 233 };