* [PATCH] perf: powerpc: Fix compile errors in bp_signal files
@ 2013-04-26 17:33 Sukadev Bhattiprolu
2013-04-29 9:59 ` Jiri Olsa
2013-05-31 11:30 ` [tip:perf/core] perf tests: " tip-bot for Sukadev Bhattiprolu
0 siblings, 2 replies; 3+ messages in thread
From: Sukadev Bhattiprolu @ 2013-04-26 17:33 UTC (permalink / raw)
To: Arnaldo Carvalho de Melo; +Cc: Jiri Olsa, linux-kernel
>From 0e27e218def9ae630cc07efbd21664c150a3727a Mon Sep 17 00:00:00 2001
From: Sukadev Bhattiprolu <sukadev@linux.vnet.ibm.com>
Date: Fri, 26 Apr 2013 10:17:56 -0700
Subject: [PATCH] perf: powerpc: Fix compile errors in bp_signal files
When building on powerpc, we get compile errors in bp_signal.c and
bp_signal_overflow.c due to __u64 and '%llx'.
Powerpc, needs __SANE_USERSPACE_TYPES__ to be defined so we pick up
<asm-generic/int-ll64.h> and define __u64 as unsigned long long.
Signed-off-by: Sukadev Bhattiprolu <sukadev@linux.vnet.ibm.com>
---
tools/perf/tests/bp_signal.c | 6 ++++++
tools/perf/tests/bp_signal_overflow.c | 6 ++++++
2 files changed, 12 insertions(+), 0 deletions(-)
diff --git a/tools/perf/tests/bp_signal.c b/tools/perf/tests/bp_signal.c
index 68daa28..aba0954 100644
--- a/tools/perf/tests/bp_signal.c
+++ b/tools/perf/tests/bp_signal.c
@@ -4,6 +4,12 @@
* (git://github.com/deater/perf_event_tests)
*/
+/*
+ * Powerpc needs __SANE_USERSPACE_TYPES__ before <linux/types.h> to select
+ * 'int-ll64.h' and avoid compile warnings when printing __u64 with %llu.
+ */
+#define __SANE_USERSPACE_TYPES__
+
#include <stdlib.h>
#include <stdio.h>
#include <unistd.h>
diff --git a/tools/perf/tests/bp_signal_overflow.c b/tools/perf/tests/bp_signal_overflow.c
index fe7ed28..44ac821 100644
--- a/tools/perf/tests/bp_signal_overflow.c
+++ b/tools/perf/tests/bp_signal_overflow.c
@@ -3,6 +3,12 @@
* perf_event_tests (git://github.com/deater/perf_event_tests)
*/
+/*
+ * Powerpc needs __SANE_USERSPACE_TYPES__ before <linux/types.h> to select
+ * 'int-ll64.h' and avoid compile warnings when printing __u64 with %llu.
+ */
+#define __SANE_USERSPACE_TYPES__
+
#include <stdlib.h>
#include <stdio.h>
#include <unistd.h>
--
1.7.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] perf: powerpc: Fix compile errors in bp_signal files
2013-04-26 17:33 [PATCH] perf: powerpc: Fix compile errors in bp_signal files Sukadev Bhattiprolu
@ 2013-04-29 9:59 ` Jiri Olsa
2013-05-31 11:30 ` [tip:perf/core] perf tests: " tip-bot for Sukadev Bhattiprolu
1 sibling, 0 replies; 3+ messages in thread
From: Jiri Olsa @ 2013-04-29 9:59 UTC (permalink / raw)
To: Sukadev Bhattiprolu; +Cc: Arnaldo Carvalho de Melo, linux-kernel
On Fri, Apr 26, 2013 at 10:33:21AM -0700, Sukadev Bhattiprolu wrote:
> From 0e27e218def9ae630cc07efbd21664c150a3727a Mon Sep 17 00:00:00 2001
> From: Sukadev Bhattiprolu <sukadev@linux.vnet.ibm.com>
> Date: Fri, 26 Apr 2013 10:17:56 -0700
> Subject: [PATCH] perf: powerpc: Fix compile errors in bp_signal files
>
> When building on powerpc, we get compile errors in bp_signal.c and
> bp_signal_overflow.c due to __u64 and '%llx'.
>
> Powerpc, needs __SANE_USERSPACE_TYPES__ to be defined so we pick up
> <asm-generic/int-ll64.h> and define __u64 as unsigned long long.
matches the use in tests/attr.c
Acked-by: Jiri Olsa <jolsa@redhat.com>
thanks,
jirka
>
> Signed-off-by: Sukadev Bhattiprolu <sukadev@linux.vnet.ibm.com>
> ---
> tools/perf/tests/bp_signal.c | 6 ++++++
> tools/perf/tests/bp_signal_overflow.c | 6 ++++++
> 2 files changed, 12 insertions(+), 0 deletions(-)
>
> diff --git a/tools/perf/tests/bp_signal.c b/tools/perf/tests/bp_signal.c
> index 68daa28..aba0954 100644
> --- a/tools/perf/tests/bp_signal.c
> +++ b/tools/perf/tests/bp_signal.c
> @@ -4,6 +4,12 @@
> * (git://github.com/deater/perf_event_tests)
> */
>
> +/*
> + * Powerpc needs __SANE_USERSPACE_TYPES__ before <linux/types.h> to select
> + * 'int-ll64.h' and avoid compile warnings when printing __u64 with %llu.
> + */
> +#define __SANE_USERSPACE_TYPES__
> +
> #include <stdlib.h>
> #include <stdio.h>
> #include <unistd.h>
> diff --git a/tools/perf/tests/bp_signal_overflow.c b/tools/perf/tests/bp_signal_overflow.c
> index fe7ed28..44ac821 100644
> --- a/tools/perf/tests/bp_signal_overflow.c
> +++ b/tools/perf/tests/bp_signal_overflow.c
> @@ -3,6 +3,12 @@
> * perf_event_tests (git://github.com/deater/perf_event_tests)
> */
>
> +/*
> + * Powerpc needs __SANE_USERSPACE_TYPES__ before <linux/types.h> to select
> + * 'int-ll64.h' and avoid compile warnings when printing __u64 with %llu.
> + */
> +#define __SANE_USERSPACE_TYPES__
> +
> #include <stdlib.h>
> #include <stdio.h>
> #include <unistd.h>
> --
> 1.7.1
>
^ permalink raw reply [flat|nested] 3+ messages in thread
* [tip:perf/core] perf tests: Fix compile errors in bp_signal files
2013-04-26 17:33 [PATCH] perf: powerpc: Fix compile errors in bp_signal files Sukadev Bhattiprolu
2013-04-29 9:59 ` Jiri Olsa
@ 2013-05-31 11:30 ` tip-bot for Sukadev Bhattiprolu
1 sibling, 0 replies; 3+ messages in thread
From: tip-bot for Sukadev Bhattiprolu @ 2013-05-31 11:30 UTC (permalink / raw)
To: linux-tip-commits; +Cc: acme, linux-kernel, hpa, mingo, sukadev, tglx, jolsa
Commit-ID: b3539d214f6000217aae97b5ae32df5b03faf850
Gitweb: http://git.kernel.org/tip/b3539d214f6000217aae97b5ae32df5b03faf850
Author: Sukadev Bhattiprolu <sukadev@linux.vnet.ibm.com>
AuthorDate: Fri, 26 Apr 2013 10:17:56 -0700
Committer: Arnaldo Carvalho de Melo <acme@redhat.com>
CommitDate: Tue, 28 May 2013 16:23:57 +0300
perf tests: Fix compile errors in bp_signal files
When building on powerpc, we get compile errors in bp_signal.c and
bp_signal_overflow.c due to __u64 and '%llx'.
Powerpc, needs __SANE_USERSPACE_TYPES__ to be defined so we pick up
<asm-generic/int-ll64.h> and define __u64 as unsigned long long.
Signed-off-by: Sukadev Bhattiprolu <sukadev@linux.vnet.ibm.com>
Cc: Jiri Olsa <jolsa@redhat.com>
Link: http://lkml.kernel.org/r/20130426173320.GA7029@us.ibm.com
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
---
tools/perf/tests/bp_signal.c | 6 ++++++
tools/perf/tests/bp_signal_overflow.c | 6 ++++++
2 files changed, 12 insertions(+)
diff --git a/tools/perf/tests/bp_signal.c b/tools/perf/tests/bp_signal.c
index 68daa28..aba0954 100644
--- a/tools/perf/tests/bp_signal.c
+++ b/tools/perf/tests/bp_signal.c
@@ -4,6 +4,12 @@
* (git://github.com/deater/perf_event_tests)
*/
+/*
+ * Powerpc needs __SANE_USERSPACE_TYPES__ before <linux/types.h> to select
+ * 'int-ll64.h' and avoid compile warnings when printing __u64 with %llu.
+ */
+#define __SANE_USERSPACE_TYPES__
+
#include <stdlib.h>
#include <stdio.h>
#include <unistd.h>
diff --git a/tools/perf/tests/bp_signal_overflow.c b/tools/perf/tests/bp_signal_overflow.c
index fe7ed28..44ac821 100644
--- a/tools/perf/tests/bp_signal_overflow.c
+++ b/tools/perf/tests/bp_signal_overflow.c
@@ -3,6 +3,12 @@
* perf_event_tests (git://github.com/deater/perf_event_tests)
*/
+/*
+ * Powerpc needs __SANE_USERSPACE_TYPES__ before <linux/types.h> to select
+ * 'int-ll64.h' and avoid compile warnings when printing __u64 with %llu.
+ */
+#define __SANE_USERSPACE_TYPES__
+
#include <stdlib.h>
#include <stdio.h>
#include <unistd.h>
^ permalink raw reply related [flat|nested] 3+ messages in thread
end of thread, other threads:[~2013-05-31 11:30 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-04-26 17:33 [PATCH] perf: powerpc: Fix compile errors in bp_signal files Sukadev Bhattiprolu
2013-04-29 9:59 ` Jiri Olsa
2013-05-31 11:30 ` [tip:perf/core] perf tests: " tip-bot for Sukadev Bhattiprolu
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox