Re: [PATCH V4 18/24] perf tools: Add Intel BTS support

[Arnaldo Carvalho de Melo <acme@kernel.org>]

Em Tue, May 05, 2015 at 04:51:41PM -0300, Arnaldo Carvalho de Melo escreveu:
> Em Thu, Apr 30, 2015 at 05:37:41PM +0300, Adrian Hunter escreveu:
> > Signed-off-by: Adrian Hunter <adrian.hunter@intel.com>
> > ---
> >  tools/perf/Documentation/intel-bts.txt |  69 +++
> > 
> > diff --git a/tools/perf/Documentation/intel-bts.txt b/tools/perf/Documentation/intel-bts.txt
> > new file mode 100644
> > index 0000000..7a79d36
> > --- /dev/null
> > +++ b/tools/perf/Documentation/intel-bts.txt
> > @@ -0,0 +1,69 @@
> > +Intel Branch Trace Store
> > +========================
> > +
> > +perf record
> > +===========
> > +
> > +new event
> > +---------
> > +
> > +The Intel BTS kernel driver creates a new PMU for Intel BTS.  The perf record
> > +option is:
> > +
> > +	-e intel_bts//
> 
> So, I applied up where Jiri acked, and will process more, but without
> digging deep, lemme ask you you could please reorder the patches so that
> what is available in more machines can be tested first, i.e. I think
> that on this machine:
> 
> [acme@ssdandy ~]$ perf report --header --stdio
> Error:
> The perf.data file has no samples!
> # ========
> # captured on: Tue May  5 16:35:13 2015
> # hostname : ssdandy.ghostprotocols.net
> # os release : 4.0.0-rc6+
> # perf version : 4.0.rc6.gef42845
> # arch : x86_64
> # nrcpus online : 8
> # nrcpus avail : 8
> # cpudesc : Intel(R) Core(TM) i7-2920XM CPU @ 2.50GHz
> # cpuid : GenuineIntel,6,42,7
> # total memory : 16318708 kB
> # cmdline : /home/acme/bin/perf record --per-thread -e intel_bts//u ls 
> # event : name = intel_bts//u, , type = 7, size = 112, { sample_period,
> # sample_freq } = 4000, sample_type = IP|TID|PERIOD, disabled = 1,
> # exclude_kernel = 1, exclude_hv = 1, mmap = 1, comm = 
> # HEADER_CPU_TOPOLOGY info available, use -I to display
> # HEADER_NUMA_TOPOLOGY info available, use -I to display
> # pmu mappings: cpu = 4, intel_bts = 7, software = 1, power = 6,
> # tracepoint = 2, breakpoint = 5
> # ========
> #
> (END)
> 
> It has BTS on the cpuinfo and yeah, intel_bts is there on the pmu
> mappings.
> 
> So, this is what I have from you already merged:
> 
> [acme@ssdandy linux]$ git log --oneline --author hunter | head -20
> 030026db927b perf record: Add AUX area tracing Snapshot Mode support
> 1715de1a37b5 perf tools: Add AUX area tracing Snapshot Mode
> 41a38c374351 perf tools: Add support for PERF_RECORD_ITRACE_START
> c11e07a556d0 perf tools: Add support for PERF_RECORD_AUX
> 61c90246aeeb perf auxtrace: Add option to synthesize events for transactions
> 8214d444516a perf tools: Add build option NO_AUXTRACE to exclude AUX area tracing
> cd10b2895205 perf tools: Hit all build ids when AUX area tracing
> 99fa29845349 perf tools: Add AUX area tracing index
> 64a7e61f11a2 perf report: Fix placement of itrace option in documentation
> 520a2ebc2f0b perf report: Add Instruction Tracing support
> 6d5cdd64f314 perf script: Always allow fields 'addr' and 'cpu' for auxtrace
> b72e74d1f140 perf evlist: Amend mmap ref counting for the AUX area mmap
> 70d73de4cd57 perf tools: Add aux_watermark member of struct perf_event_attr
> 400ea6d327ff perf script: Add field option 'flags' to print sample flags
> 0f0aa5e0693c perf inject: Add Instruction Tracing support
> cd17a9b54463 perf inject: Re-pipe AUX area tracing events
> 7a680eb990b6 perf script: Add Instruction Tracing support
> cfe9174fcfe2 perf tools: Add member to struct dso for an instruction cache
> c3278f0231e8 perf auxtrace: Add a hashtable for caching
> 73f75fb1bdc5 perf auxtrace: Add processing for AUX area tracing events
> [acme@ssdandy linux]$
> 
> Will try processing these BTS first, i.e. do the reorder here,

Not that easy...

- Arnaldo