All of lore.kernel.org
 help / color / mirror / Atom feed
* [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 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.