[kernel/f17] Add patch to fix perf build due to incorrect cherry-pick in 3.3.3

Josh Boyer jwboyer at fedoraproject.org
Tue Apr 24 13:47:32 UTC 2012


commit 37e7953c580226c3d550ffd19e1ea8abf10f82cb
Author: Josh Boyer <jwboyer at redhat.com>
Date:   Tue Apr 24 09:46:24 2012 -0400

    Add patch to fix perf build due to incorrect cherry-pick in 3.3.3

 kernel.spec                   |    5 ++++
 perf-fix-build-breakage.patch |   54 +++++++++++++++++++++++++++++++++++++++++
 2 files changed, 59 insertions(+), 0 deletions(-)
---
diff --git a/kernel.spec b/kernel.spec
index d17ac0a..980f151 100644
--- a/kernel.spec
+++ b/kernel.spec
@@ -810,6 +810,8 @@ Patch22011: input-synaptics-fix-regression-with-image-sensor-trackpads.patch
 #rhbz 802106
 Patch22012: ipw2200-Fix-race-condition-in-the-command-completion-acknowledge.patch
 
+Patch22013: perf-fix-build-breakage.patch
+
 # END OF PATCH DEFINITIONS
 
 %endif
@@ -1573,6 +1575,8 @@ ApplyPatch input-synaptics-fix-regression-with-image-sensor-trackpads.patch
 #rhbz 802106
 ApplyPatch ipw2200-Fix-race-condition-in-the-command-completion-acknowledge.patch
 
+ApplyPatch perf-fix-build-breakage.patch
+
 # END OF PATCH APPLICATIONS
 
 %endif
@@ -2428,6 +2432,7 @@ fi
 #              '-'
 %changelog
 * Tue Apr 24 2012 Josh Boyer <jwboyer at redhat.com>
+- Add patch to fix perf build due to incorrect cherry-pick in 3.3.3
 - Add patch to fix ipw2200 (rhbz 802106)
 
 * Mon Apr 23 2012 Josh Boyer <jwboyer at redhat.com>
diff --git a/perf-fix-build-breakage.patch b/perf-fix-build-breakage.patch
new file mode 100644
index 0000000..d1385dd
--- /dev/null
+++ b/perf-fix-build-breakage.patch
@@ -0,0 +1,54 @@
+From zeev.tarantov at gmail.com Sun Apr 22 23:38:36 2012
+From: Zeev Tarantov <zeev.tarantov at gmail.com>
+Date: Mon, 23 Apr 2012 09:37:04 +0300
+Subject: Perf: fix build breakage
+To: "David S. Miller" <davem at davemloft.net>, Arnaldo Carvalho de Melo <acme at redhat.com>, Greg Kroah-Hartman <gregkh at linuxfoundation.org>, Linux Kernel Mailing List <linux-kernel at vger.kernel.org>
+Message-ID: <20120423063704.GA3465 at myhost>
+Content-Disposition: inline
+
+
+From: Zeev Tarantov <zeev.tarantov at gmail.com>
+
+[Patch not needed upstream as this is a backport build bugfix - gregkh
+
+gcc correctly complains:
+
+util/hist.c: In function ‘__hists__add_entry’:
+util/hist.c:240:27: error: invalid type argument of ‘->’ (have ‘struct hist_entry’)
+util/hist.c:241:23: error: invalid type argument of ‘->’ (have ‘struct hist_entry’)
+
+for this new code:
+
++                       if (he->ms.map != entry->ms.map) {
++                               he->ms.map = entry->ms.map;
++                               if (he->ms.map)
++                                       he->ms.map->referenced = true;
++                       }
+
+because "entry" is a "struct hist_entry", not a pointer to a struct.
+
+In mainline, "entry" is a pointer to struct passed as argument to the function.
+So this is broken during backporting. But obviously not compile tested.
+
+Signed-off-by: Zeev Tarantov <zeev.tarantov at gmail.com>
+Cc: Signed-off-by: David S. Miller <davem at davemloft.net>
+Cc: Arnaldo Carvalho de Melo <acme at redhat.com>
+Signed-off-by: Greg Kroah-Hartman <gregkh at linuxfoundation.org>
+
+---
+ tools/perf/util/hist.c |    4 ++--
+ 1 file changed, 2 insertions(+), 2 deletions(-)
+
+--- a/tools/perf/util/hist.c
++++ b/tools/perf/util/hist.c
+@@ -237,8 +237,8 @@ struct hist_entry *__hists__add_entry(st
+ 			 * mis-adjust symbol addresses when computing
+ 			 * the history counter to increment.
+ 			 */
+-			if (he->ms.map != entry->ms.map) {
+-				he->ms.map = entry->ms.map;
++			if (he->ms.map != entry.ms.map) {
++				he->ms.map = entry.ms.map;
+ 				if (he->ms.map)
+ 					he->ms.map->referenced = true;
+ 			}


More information about the scm-commits mailing list