Re: [PATCH 11/34] perf, tools: Add abort,notx,intx branch filter options to perf report -j

2012-10-23 Thread Peter Zijlstra
On Thu, 2012-10-18 at 16:19 -0700, Andi Kleen wrote: > + BRANCH_OPT("abort", PERF_SAMPLE_BRANCH_ABORT), > + BRANCH_OPT("intx", PERF_SAMPLE_BRANCH_INTX), > + BRANCH_OPT("notx", PERF_SAMPLE_BRANCH_NOTX), I think we want tx in the abort name, its very much a transaction abort, not

Re: [PATCH 11/34] perf, tools: Add abort,notx,intx branch filter options to perf report -j

2012-10-23 Thread Peter Zijlstra
On Thu, 2012-10-18 at 16:19 -0700, Andi Kleen wrote: + BRANCH_OPT(abort, PERF_SAMPLE_BRANCH_ABORT), + BRANCH_OPT(intx, PERF_SAMPLE_BRANCH_INTX), + BRANCH_OPT(notx, PERF_SAMPLE_BRANCH_NOTX), I think we want tx in the abort name, its very much a transaction abort, not any

[PATCH 11/34] perf, tools: Add abort,notx,intx branch filter options to perf report -j

2012-10-18 Thread Andi Kleen
From: Andi Kleen Make perf report -j aware of the new intx,notx,abort branch qualifiers. Signed-off-by: Andi Kleen --- tools/perf/builtin-record.c |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/tools/perf/builtin-record.c b/tools/perf/builtin-record.c index

[PATCH 11/34] perf, tools: Add abort,notx,intx branch filter options to perf report -j

2012-10-18 Thread Andi Kleen
From: Andi Kleen a...@linux.intel.com Make perf report -j aware of the new intx,notx,abort branch qualifiers. Signed-off-by: Andi Kleen a...@linux.intel.com --- tools/perf/builtin-record.c |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/tools/perf/builtin-record.c