[Buildroot] [PATCH/RFC 1/1] package/mesa3d: disable libunwind for Gallium drivers on arm
Romain Naour
romain.naour at smile.fr
Mon Jul 3 12:56:00 UTC 2017
Hi Bernd,
Le 17/06/2017 à 18:54, Bernd Kuhls a écrit :
> Building a Gallium driver on ARM is broken if optional libunwind
> support is enabled. The build fails here:
> https://cgit.freedesktop.org/mesa/mesa/tree/src/gallium/auxiliary/util/u_debug_stack.c?h=17.1#n114
>
> util/u_debug_stack.c:114:4: error: 'unw_base' undeclared (first use in this function)
> unw_getcontext(&context);
>
> unw_base is declared in the libunwind package in
> include/libunwind-arm.h:
> https://github.com/libunwind/libunwind/blob/7c079200d024d5868073246c4ec8f80446b0a4c0/include/libunwind-arm.h#L262
>
> which contains a comment:
>
> /* There is no getcontext() on ARM. Use a stub version which only
> saves GP registers. FIXME: Not ideal, may not be sufficient for
> all libunwind use cases. Stores pc+8, which is only approximately
> correct, really. */
>
> To fix the build error we just disable libunwind support on ARM when a
> Gallium driver is enabled until upstream comes up with a working
> solution. Tested using this defconfig with test_pkg
>
> BR2_PACKAGE_MESA3D=y
> BR2_PACKAGE_MESA3D_GALLIUM_DRIVER_SWRAST=y
> BR2_PACKAGE_MESA3D_OPENGL_EGL=y
> BR2_PACKAGE_MESA3D_OPENGL_ES=y
> BR2_PACKAGE_LIBUNWIND=y
>
> without this patch:
>
> armv5-ctng-linux-gnueabi [ 1/49]: FAILED
> armv7-ctng-linux-gnueabihf [ 2/49]: FAILED
> br-arm-cortex-a9-glibc [ 6/49]: FAILED
>
> and with this patch:
>
> armv5-ctng-linux-gnueabi [ 1/49]: OK
> armv7-ctng-linux-gnueabihf [ 2/49]: OK
> br-arm-cortex-a9-glibc [ 6/49]: OK
>
> Fixes
> http://autobuild.buildroot.net/results/3ef/3efe156b6494e4392b6c31de447ee2c72acc1a53/
Can you test with http://patchwork.ozlabs.org/patch/783462/
I believe it's a problem with asm().
Thanks for the initial investigation.
Best regards,
Romain
>
> Signed-off-by: Bernd Kuhls <bernd.kuhls at t-online.de>
> ---
> I have no idea whether the problem can be fixed by patching the code,
> disabling libunwind in the gallium at arm case at least fixes the build
> error.
>
> package/mesa3d/mesa3d.mk | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/package/mesa3d/mesa3d.mk b/package/mesa3d/mesa3d.mk
> index 04a7d13b9..bc01a37d6 100644
> --- a/package/mesa3d/mesa3d.mk
> +++ b/package/mesa3d/mesa3d.mk
> @@ -193,11 +193,15 @@ MESA3D_CONF_OPTS += --disable-xvmc
> endif
>
> ifeq ($(BR2_PACKAGE_LIBUNWIND),y)
> +ifneq ($(BR2_ARM_CPU_HAS_ARM)$(BR2_PACKAGE_MESA3D_GALLIUM_DRIVER),yy)
> MESA3D_CONF_OPTS += --enable-libunwind
> MESA3D_DEPENDENCIES += libunwind
> else
> MESA3D_CONF_OPTS += --disable-libunwind
> endif
> +else
> +MESA3D_CONF_OPTS += --disable-libunwind
> +endif
>
> ifeq ($(BR2_PACKAGE_LIBVDPAU),y)
> MESA3D_DEPENDENCIES += libvdpau
>
More information about the buildroot
mailing list