From: Peter Zijlstra <peterz@infradead.org>
To: Will Deacon <will.deacon@arm.com>
Cc: Stephane Eranian <eranian@google.com>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"mingo@elte.hu" <mingo@elte.hu>,
"acme@redhat.com" <acme@redhat.com>,
"ming.m.lin@intel.com" <ming.m.lin@intel.com>,
"andi@firstfloor.org" <andi@firstfloor.org>,
"robert.richter@amd.com" <robert.richter@amd.com>,
"ravitillo@lbl.gov" <ravitillo@lbl.gov>,
"paulus@samba.org" <paulus@samba.org>,
"benh@kernel.crashing.org" <benh@kernel.crashing.org>,
"rth@twiddle.net" <rth@twiddle.net>,
"ralf@linux-mips.org" <ralf@linux-mips.org>,
"davem@davemloft.net" <davem@davemloft.net>,
"lethal@linux-sh.org" <lethal@linux-sh.org>
Subject: Re: [PATCH 00/12] perf_events: add support for sampling taken branches (v2)
Date: Tue, 06 Dec 2011 12:03:32 +0100 [thread overview]
Message-ID: <1323169412.32012.69.camel@twins> (raw)
In-Reply-To: <20111206094927.GA25782@mudshark.cambridge.arm.com>
On Tue, 2011-12-06 at 09:49 +0000, Will Deacon wrote:
> On Mon, Dec 05, 2011 at 10:39:26PM +0000, Peter Zijlstra wrote:
> > On Fri, 2011-10-14 at 14:37 +0200, Stephane Eranian wrote:
> > > This patchset adds an important and useful new feature to
> > > perf_events: branch stack sampling. In other words, the
> > > ability to capture taken branches into each sample.
> > >
> > Other than the few comments given it all looks good. My main worry is
> > the Intel only aspect, I'd really love for there to be another platform
> > that could implement at least part of this.
>
> I discussed this with Stephane in Prague and, although it would be lovely to
> have this on ARM, we simply don't have the hardware to do it. So the nature
> of series does seem to be x86-centric unless there's way to do a watered
> down version in software.
The only way to do this in software would be like
CONFIG_PROFILE_ALL_BRANCHES and that's horrid (and kernel only).
But there's more than Intel & ARM, but it looks like PPC doesn't have
this either and I suspect MIPS and SPARC don't either, which doesn't
leave us with much else.
So I guess we should just go ahead and merge this and hope more hardware
grows this feature in a compatible enough manner.
next prev parent reply other threads:[~2011-12-06 11:04 UTC|newest]
Thread overview: 36+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-10-14 12:37 [PATCH 00/12] perf_events: add support for sampling taken branches (v2) Stephane Eranian
2011-10-14 12:37 ` [PATCH 01/12] perf_events: add generic taken branch sampling support (v2) Stephane Eranian
2011-12-05 21:06 ` Peter Zijlstra
2011-12-06 19:42 ` Stephane Eranian
2011-12-05 22:14 ` Peter Zijlstra
2011-12-06 19:27 ` Stephane Eranian
2011-10-14 12:37 ` [PATCH 02/12] perf_events: add Intel LBR MSR definitions (v2) Stephane Eranian
2011-10-14 12:37 ` [PATCH 03/12] perf_events: add Intel X86 LBR sharing logic (v2) Stephane Eranian
2011-10-14 12:37 ` [PATCH 04/12] perf_events: sync branch stack sampling with X86 precise_sampling (v2) Stephane Eranian
2011-10-14 12:37 ` [PATCH 05/12] perf_events: add LBR mappings for PERF_SAMPLE_BRANCH filters (v2) Stephane Eranian
2011-12-05 22:35 ` Peter Zijlstra
2011-12-07 4:22 ` Stephane Eranian
2011-10-14 12:37 ` [PATCH 06/12] perf_events: implement PERF_SAMPLE_BRANCH for Intel X86 (v2) Stephane Eranian
2011-10-14 12:37 ` [PATCH 07/12] perf_events: add LBR software filter support " Stephane Eranian
2011-12-05 22:29 ` Peter Zijlstra
2011-10-14 12:37 ` [PATCH 08/12] perf_events: disable PERF_SAMPLE_BRANCH_* when not supported (v2) Stephane Eranian
2011-10-14 12:37 ` [PATCH 09/12] perf_events: add hook to flush branch_stack on context switch (v2) Stephane Eranian
2011-12-05 21:10 ` Peter Zijlstra
2011-12-05 21:37 ` Peter Zijlstra
2011-12-07 18:25 ` Stephane Eranian
2011-12-08 10:49 ` Peter Zijlstra
2011-12-08 18:04 ` Stephane Eranian
2011-12-08 18:13 ` Peter Zijlstra
2011-12-08 22:06 ` Stephane Eranian
2011-12-09 9:00 ` Peter Zijlstra
2011-10-14 12:37 ` [PATCH 10/12] perf: add code to support PERF_SAMPLE_BRANCH_STACK (v2) Stephane Eranian
2011-10-14 12:37 ` [PATCH 11/12] perf: add support for sampling taken branch to perf record (v2) Stephane Eranian
2011-10-14 12:37 ` [PATCH 12/12] perf: add support for taken branch sampling to perf report (v2) Stephane Eranian
2011-12-04 20:11 ` [PATCH 00/12] perf_events: add support for sampling taken branches (v2) Stephane Eranian
2011-12-05 15:27 ` Peter Zijlstra
2011-12-05 22:39 ` Peter Zijlstra
2011-12-06 9:49 ` Will Deacon
2011-12-06 11:03 ` Peter Zijlstra [this message]
2011-12-06 19:14 ` Stephane Eranian
2011-12-06 19:20 ` Peter Zijlstra
2011-12-06 19:22 ` Stephane Eranian
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1323169412.32012.69.camel@twins \
--to=peterz@infradead.org \
--cc=acme@redhat.com \
--cc=andi@firstfloor.org \
--cc=benh@kernel.crashing.org \
--cc=davem@davemloft.net \
--cc=eranian@google.com \
--cc=lethal@linux-sh.org \
--cc=linux-kernel@vger.kernel.org \
--cc=ming.m.lin@intel.com \
--cc=mingo@elte.hu \
--cc=paulus@samba.org \
--cc=ralf@linux-mips.org \
--cc=ravitillo@lbl.gov \
--cc=robert.richter@amd.com \
--cc=rth@twiddle.net \
--cc=will.deacon@arm.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.