<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Apr 19, 2016 at 8:57 AM, Ross Burton <span dir="ltr">&lt;<a href="mailto:ross.burton@intel.com" target="_blank">ross.burton@intel.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Instead of passing DESTDIR just in the make install invocation, pass it in<br>
EXTRA_OEMAKE.  This appears to stop perf from rebuilding at instal time for me,<br>
which appears to be the trigger for the random build failure.<br></blockquote><div><br></div><div>This also passed my local build tests, but then again, in two weeks of trying (off and</div><div>on), I&#39;ve never reproduced the race condition .. so I can&#39;t comment if it actually fixes</div><div>it or not.</div><div><br></div><div>Having a the destdir be consistent for all phases makes sense though, so I see no</div><div>problem with the change as-is.</div><div><br></div><div>I&#39;ll suspend my nasty perf Makefile hacks to serialize parts of the build to see if this</div><div>holds as a permanent fix for the problem.</div><div><br></div><div>Tested-by: Bruce Ashfield &lt;<a href="mailto:bruce.ashfield@windriver.com">bruce.ashfield@windriver.com</a>&gt;</div><div><br></div><div>Bruce</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
[ YOCTO #9182 ]<br>
<br>
Signed-off-by: Ross Burton &lt;<a href="mailto:ross.burton@intel.com">ross.burton@intel.com</a>&gt;<br>
---<br>
 meta/recipes-kernel/perf/<a href="http://perf.bb" rel="noreferrer" target="_blank">perf.bb</a> | 3 ++-<br>
 1 file changed, 2 insertions(+), 1 deletion(-)<br>
<br>
diff --git a/meta/recipes-kernel/perf/<a href="http://perf.bb" rel="noreferrer" target="_blank">perf.bb</a> b/meta/recipes-kernel/perf/<a href="http://perf.bb" rel="noreferrer" target="_blank">perf.bb</a><br>
index c43f39c..3d6ffb3 100644<br>
--- a/meta/recipes-kernel/perf/<a href="http://perf.bb" rel="noreferrer" target="_blank">perf.bb</a><br>
+++ b/meta/recipes-kernel/perf/<a href="http://perf.bb" rel="noreferrer" target="_blank">perf.bb</a><br>
@@ -88,6 +88,7 @@ EXTRA_OEMAKE = &#39;\<br>
 &#39;<br>
<br>
 EXTRA_OEMAKE += &quot;\<br>
+    &#39;DESTDIR=${D}&#39; \<br>
     &#39;prefix=${prefix}&#39; \<br>
     &#39;bindir=${bindir}&#39; \<br>
     &#39;sharedir=${datadir}&#39; \<br>
@@ -110,7 +111,7 @@ do_compile() {<br>
 do_install() {<br>
        # Linux kernel build system is expected to do the right thing<br>
        unset CFLAGS<br>
-       oe_runmake DESTDIR=${D} install<br>
+       oe_runmake install<br>
        # we are checking for this make target to be compatible with older perf versions<br>
        if [ &quot;${@perf_feature_enabled(&#39;perf-scripting&#39;, 1, 0, d)}&quot; = &quot;1&quot; ] &amp;&amp; grep -q install-python_ext ${S}/tools/perf/Makefile*; then<br>
                oe_runmake DESTDIR=${D} install-python_ext<br>
<span class="HOEnZb"><font color="#888888">--<br>
2.8.0.rc3<br>
<br>
--<br>
_______________________________________________<br>
Openembedded-core mailing list<br>
<a href="mailto:Openembedded-core@lists.openembedded.org">Openembedded-core@lists.openembedded.org</a><br>
<a href="http://lists.openembedded.org/mailman/listinfo/openembedded-core" rel="noreferrer" target="_blank">http://lists.openembedded.org/mailman/listinfo/openembedded-core</a><br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature">&quot;Thou shalt not follow the NULL pointer, for chaos and madness await thee at its end&quot;</div>
</div></div>