Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1631946 > unrolled thread
| Started by | Adrian Hunter <adrian.hunter@intel.com> |
|---|---|
| First post | 2017-04-27 10:20 +0200 |
| Last post | 2017-04-27 10:20 +0200 |
| Articles | 1 — 1 participant |
Back to article view | Back to linux.kernel
This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by
below is the oldest one visible, not the original post.
Re: [PATCH] perf/x86/intel/pt: Allow disabling branch tracing Adrian Hunter <adrian.hunter@intel.com> - 2017-04-27 10:20 +0200
| From | Adrian Hunter <adrian.hunter@intel.com> |
|---|---|
| Date | 2017-04-27 10:20 +0200 |
| Subject | Re: [PATCH] perf/x86/intel/pt: Allow disabling branch tracing |
| Message-ID | <tAMOC-6i9-19@gated-at.bofh.it> |
On 06/02/17 19:19, Andi Kleen wrote: > On Mon, Feb 06, 2017 at 06:05:29PM +0200, Alexander Shishkin wrote: >> Andi Kleen <andi@firstfloor.org> writes: >> >>> Alexander Shishkin <alexander.shishkin@linux.intel.com> writes: >>> >>>> Now that Intel PT supports more types of trace content than just branch >>>> tracing, it may be useful to allow the user to disable branch tracing >>>> when it is not needed. >>>> >>>> The special case is BDW, where not setting BranchEn is not supported. >>>> >>>> This is slightly trickier than necessary, because up to this moment >>>> the driver has been setting BranchEn automatically and the userspace >>>> assumes as much. Instead of reversing the semantics of BranchEn, we >>>> introduce a 'passthrough' bit, which will forego the default and allow >>>> the user to set BranchEn to their heart's content. >>> >>> cpu/passthrough=1,branchen=1/ seems far uglier/more complicanted to me >>> than the original cpu/nobranch=1/ >> >> It's /passthrough=1,branch=0/ or simply /passthrough=1/. > > Ok, but still you have to list exactly to which flags passthrough > applies to, and it will only ever be branchen. It doesn't have to be that bad. For Intel PT, perf tools already provides default config which the user must override. So, when 'passthrough' is supported, the default config would include 'passthrough=1,branch=1'. So then the user would only have to put 'branch=0' when they want no branches.
Back to top | Article view | linux.kernel
csiph-web