<div dir="ltr">Hi,<br><br><div class="gmail_quote"><div dir="ltr">Il giorno gio 2 ago 2018 alle ore 02:06 John Stultz <<a href="mailto:john.stultz@linaro.org">john.stultz@linaro.org</a>> ha scritto:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Just as a heads up, with mesa3d master (plus my other build fixups I<br>
just sent out), I'm seeing a new build failure when building under<br>
AOSP/master:<br>
<br>
external/mesa3d/src/amd/vulkan/Android.mk: error: vulkan.radv<br>
(SHARED_LIBRARIES android-arm64) missing libLLVM (SHARED_LIBRARIES<br>
android-arm64)<br>
You can set ALLOW_MISSING_DEPENDENCIES=true in your environment if<br>
this is intentional, but that may defer real problems until later in<br>
the build.<br>
external/mesa3d/src/amd/vulkan/Android.mk: error: vulkan.radv<br>
(SHARED_LIBRARIES android-arm) missing libLLVM (SHARED_LIBRARIES<br>
android-arm)<br>
You can set ALLOW_MISSING_DEPENDENCIES=true in your environment if<br>
this is intentional, but that may defer real problems until later in<br>
the build.<br>
build/make/core/<a href="http://main.mk:842" rel="noreferrer" target="_blank">main.mk:842</a>: error: exiting from previous errors.<br>
22:56:25 ckati failed with: exit status 1<br>
<br>
This seems to be coming from the $(call mesa-build-with-llvm) line here:<br>
<a href="https://github.com/mesa3d/mesa/blob/master/src/amd/vulkan/Android.mk#L144" rel="noreferrer" target="_blank">https://github.com/mesa3d/mesa/blob/master/src/amd/vulkan/Android.mk#L144</a><br>
<br>
(as commenting that out avoids the issue)<br>
<br>
This gets pulled in as AOSP will check library dependencies for all<br>
SHARED_LIBRARIES, so that's why the vulkan.radv trips this while other<br>
"mesa-build-with-llvm" callers don't (they are built as<br>
STATIC_LIBRARIES).<br>
<br>
Last time we had this issue, it was with the libpciaccess library,<br>
which we solved it by dropping it.<br>
<br>
In this case I'm not sure what the best solution might be.<br>
<br>
* We could set it as a static library instead?<br>
<br>
* Alternatively, it seems we might be able to use the libLLVM_android<br>
library instead of libLLVM here:<br>
  <a href="https://github.com/mesa3d/mesa/blob/master/Android.mk#L107" rel="noreferrer" target="_blank">https://github.com/mesa3d/mesa/blob/master/Android.mk#L107</a><br>
<br>
Other ideas/suggestions?<br>
<br>
thanks<br>
-john<br></blockquote><div><br></div><div>A couple of ideas, but not yet tested in a build</div><div><br></div><div>option 3) - use LOCAL_MODULE_TARGET_ARCH in the local vulkan.radv Android.mk and see if that solves the problem</div><div><br></div><div>option 4) - brace the 'include $(LOCAL_PATH)/vulkan/Android.mk' in vulkan.radv src/amd/Android.mk in like this:</div><div><br></div><div><font face="monospace, monospace"> # Import variables</font></div><div><font face="monospace, monospace"> include $(LOCAL_PATH)/Makefile.sources</font></div><div><span style="font-family:monospace,monospace"> include $(LOCAL_PATH)/<a href="http://Android.addrlib.mk">Android.addrlib.mk</a></span><br></div><div><font face="monospace, monospace"> include $(LOCAL_PATH)/<a href="http://Android.common.mk">Android.common.mk</a></font></div><div><font face="monospace, monospace"><span class="gmail-pl-k" style="box-sizing:border-box;color:rgb(215,58,73);font-size:12px;white-space:pre">+ifneq</span><span style="color:rgb(36,41,46);font-size:12px;white-space:pre"> (</span><span class="gmail-pl-s" style="box-sizing:border-box;color:rgb(3,47,98);font-size:12px;white-space:pre">$(<span class="gmail-pl-c1" style="box-sizing:border-box;color:rgb(0,92,197)">filter</span> radeonsi,<span class="gmail-pl-s" style="box-sizing:border-box">$(<span class="gmail-pl-smi" style="box-sizing:border-box;color:rgb(36,41,46)">BOARD_GPU_DRIVERS</span>)</span>)</span><span style="color:rgb(36,41,46);font-size:12px;white-space:pre">,)</span><br></font></div><div><font face="monospace, monospace"><span style="color:rgb(36,41,46);font-size:12px;white-space:pre"> </span><font color="#24292e"><span style="font-size:12px;white-space:pre">include $(LOCAL_PATH)/vulkan/Android.mk</span></font></font></div><div><span style="color:rgb(36,41,46);font-size:12px;white-space:pre"><font face="monospace, monospace">+endif</font></span></div><div><span style="color:rgb(36,41,46);font-size:12px;white-space:pre"><font face="monospace, monospace"><br></font></span></div><div><span style="color:rgb(36,41,46);font-size:12px;white-space:pre"><font face="arial, helvetica, sans-serif">I think the option 4) seems simpler than the others , if the assumption that ARM does not have radeonsi stands</font></span></div><div><span style="color:rgb(36,41,46);font-size:12px;white-space:pre"><font face="arial, helvetica, sans-serif">Mauro</font></span></div></div></div>