[GIT PULL 0/8] perf/urgent fixes

[GIT PULL 0/8] perf/urgent fixes

[Arnaldo Carvalho de Melo]

From: Arnaldo Carvalho de Melo <acme@ghostprotocols.net>

Hi Ingo,

	Please consider pulling,

- Arnaldo

The following changes since commit cb16b91a449afd01b85ec4e59f30449d11c4acd7:

  s390: Fix a header dependencies related build error (2013-03-11 10:43:35 +0100)

are available in the git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/acme/linux tags/perf-urgent-for-mingo

for you to fetch changes up to d1398ccfec56e54010476efd6a316427d29045a6:

  perf tools: Fix LIBNUMA build with glibc 2.12 and older. (2013-03-14 08:06:21 -0300)

----------------------------------------------------------------
perf/urgent fixes:

. perf probe: Fix segfault due to testing the wrong pointer for NULL,
  from Ananth N Mavinakayanahalli.

. libtraceevent: Remove hard coded include to /usr/local/include in
  Makefile, which causes cross builds to include host header files,
  fix from Jack Mitchell.

. perf record: Use the right target interface for synthesizing
  threads when --cpu/-C option is used, fix from Jiri Olsa.

. Check if -DFORTIFY_SOURCE=2 is allowed, as gcc 4.7.2 defines
  it and then the build is broken when it is redefined in perf,
  fix from Marcin Slusarz.

. Fix build with NO_NEWT=1, that can happen explicitely or when
  the newt-devel package is not installed, from Michael Ellerman.

. perf/POWER7: Create a sysfs format entry for Power7 events, missing
  patch from a patchseries already merged, from Sukadev Bhattiprolu.

. Fix LIBNUMA build with glibc 2.12 and older, from Vinson Lee.

Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>

----------------------------------------------------------------
Ananth N Mavinakayanahalli (1):
      perf probe: Fix segfault

Jack Mitchell (1):
      libtraceevent: Remove hard coded include to /usr/local/include in Makefile

Jiri Olsa (1):
      perf record: Fix -C option

Marcin Slusarz (1):
      perf tools: check if -DFORTIFY_SOURCE=2 is allowed

Michael Ellerman (2):
      perf annotate: Fix build with NO_NEWT=1
      perf report: Fix build with NO_NEWT=1

Sukadev Bhattiprolu (1):
      perf/POWER7: Create a sysfs format entry for Power7 events

Vinson Lee (1):
      perf tools: Fix LIBNUMA build with glibc 2.12 and older.

 arch/powerpc/perf/power7-pmu.c | 13 +++++++++++++
 tools/lib/traceevent/Makefile  |  2 +-
 tools/perf/Makefile            |  8 +++++++-
 tools/perf/bench/bench.h       | 24 ++++++++++++++++++++++++
 tools/perf/builtin-record.c    |  6 ++++--
 tools/perf/util/hist.h         |  5 +++--
 tools/perf/util/strlist.c      |  2 +-
 7 files changed, 53 insertions(+), 7 deletions(-)

[PATCH 7/8] perf/POWER7: Create a sysfs format entry for Power7 events

[Arnaldo Carvalho de Melo]

From: Sukadev Bhattiprolu <sukadev@linux.vnet.ibm.com>

Create a sysfs entry, '/sys/bus/event_source/devices/cpu/format/event'
which describes the format of the POWER7 PMU events.

This code is based on corresponding code in x86.

Changelog[v4]:  [Michael Ellerman, Paul Mckerras] The event format is different
		for other POWER cpus. So move the code to POWER7-specific,
		power7-pmu.c Also, the POWER7 format uses bits 0-19 not 0-20.

Changelog[v2]: [Jiri Osla] Use PMU_FORMAT_ATTR rather than duplicating code.

Signed-off-by: Sukadev Bhattiprolu <sukadev@linux.vnet.ibm.com>
Acked-by: Paul Mackerras <paulus@samba.org>
Tested-by: Michael Ellerman <michael@ellerman.id.au>
Cc: Andi Kleen <ak@linux.intel.com>
Cc: Anton Blanchard <anton@au1.ibm.com>
Cc: Ingo Molnar <mingo@redhat.com>
Cc: Jiri Olsa <jolsa@redhat.com>
Cc: Michael Ellerman <michael@ellerman.id.au>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Cc: Robert Richter <robert.richter@amd.com>
Cc: Stephane Eranian <eranian@google.com>
Cc: benh@kernel.crashing.org
Cc: linuxppc-dev@ozlabs.org
Link: http://lkml.kernel.org/r/20130306054826.GA14627@us.ibm.com
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
---
 arch/powerpc/perf/power7-pmu.c |   13 +++++++++++++
 1 files changed, 13 insertions(+), 0 deletions(-)

diff --git a/arch/powerpc/perf/power7-pmu.c b/arch/powerpc/perf/power7-pmu.c
index b554879..3c475d6 100644
--- a/arch/powerpc/perf/power7-pmu.c
+++ b/arch/powerpc/perf/power7-pmu.c
@@ -420,7 +420,20 @@ static struct attribute_group power7_pmu_events_group = {
 	.attrs = power7_events_attr,
 };
 
+PMU_FORMAT_ATTR(event, "config:0-19");
+
+static struct attribute *power7_pmu_format_attr[] = {
+	&format_attr_event.attr,
+	NULL,
+};
+
+struct attribute_group power7_pmu_format_group = {
+	.name = "format",
+	.attrs = power7_pmu_format_attr,
+};
+
 static const struct attribute_group *power7_pmu_attr_groups[] = {
+	&power7_pmu_format_group,
 	&power7_pmu_events_group,
 	NULL,
 };
-- 
1.7.1

Re: [GIT PULL 0/8] perf/urgent fixes

[Ingo Molnar]

* Arnaldo Carvalho de Melo <acme@infradead.org> wrote:

> From: Arnaldo Carvalho de Melo <acme@ghostprotocols.net>
> 
> Hi Ingo,
> 
> 	Please consider pulling,
> 
> - Arnaldo
> 
> The following changes since commit cb16b91a449afd01b85ec4e59f30449d11c4acd7:
> 
>   s390: Fix a header dependencies related build error (2013-03-11 10:43:35 +0100)
> 
> are available in the git repository at:
> 
>   git://git.kernel.org/pub/scm/linux/kernel/git/acme/linux tags/perf-urgent-for-mingo
> 
> for you to fetch changes up to d1398ccfec56e54010476efd6a316427d29045a6:
> 
>   perf tools: Fix LIBNUMA build with glibc 2.12 and older. (2013-03-14 08:06:21 -0300)
> 
> ----------------------------------------------------------------
> perf/urgent fixes:
> 
> . perf probe: Fix segfault due to testing the wrong pointer for NULL,
>   from Ananth N Mavinakayanahalli.
> 
> . libtraceevent: Remove hard coded include to /usr/local/include in
>   Makefile, which causes cross builds to include host header files,
>   fix from Jack Mitchell.
> 
> . perf record: Use the right target interface for synthesizing
>   threads when --cpu/-C option is used, fix from Jiri Olsa.
> 
> . Check if -DFORTIFY_SOURCE=2 is allowed, as gcc 4.7.2 defines
>   it and then the build is broken when it is redefined in perf,
>   fix from Marcin Slusarz.
> 
> . Fix build with NO_NEWT=1, that can happen explicitely or when
>   the newt-devel package is not installed, from Michael Ellerman.
> 
> . perf/POWER7: Create a sysfs format entry for Power7 events, missing
>   patch from a patchseries already merged, from Sukadev Bhattiprolu.
> 
> . Fix LIBNUMA build with glibc 2.12 and older, from Vinson Lee.
> 
> Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
> 
> ----------------------------------------------------------------
> Ananth N Mavinakayanahalli (1):
>       perf probe: Fix segfault
> 
> Jack Mitchell (1):
>       libtraceevent: Remove hard coded include to /usr/local/include in Makefile
> 
> Jiri Olsa (1):
>       perf record: Fix -C option
> 
> Marcin Slusarz (1):
>       perf tools: check if -DFORTIFY_SOURCE=2 is allowed
> 
> Michael Ellerman (2):
>       perf annotate: Fix build with NO_NEWT=1
>       perf report: Fix build with NO_NEWT=1
> 
> Sukadev Bhattiprolu (1):
>       perf/POWER7: Create a sysfs format entry for Power7 events
> 
> Vinson Lee (1):
>       perf tools: Fix LIBNUMA build with glibc 2.12 and older.
> 
>  arch/powerpc/perf/power7-pmu.c | 13 +++++++++++++
>  tools/lib/traceevent/Makefile  |  2 +-
>  tools/perf/Makefile            |  8 +++++++-
>  tools/perf/bench/bench.h       | 24 ++++++++++++++++++++++++
>  tools/perf/builtin-record.c    |  6 ++++--
>  tools/perf/util/hist.h         |  5 +++--
>  tools/perf/util/strlist.c      |  2 +-
>  7 files changed, 53 insertions(+), 7 deletions(-)

Pulled, thanks Arnaldo!

	Ingo