[Buildroot] [PATCH 1/1] package/mesa3d: add valgrind support
romain.naour at gmail.com
Wed Jun 10 20:03:35 UTC 2020
Le 10/06/2020 à 17:45, James Hilliard a écrit :
> On Wed, Jun 10, 2020 at 7:41 AM Romain Naour <romain.naour at gmail.com> wrote:
>> Hi James, Thomas,
>> Le 28/04/2020 à 07:47, James Hilliard a écrit :
>>> On Mon, Apr 27, 2020 at 11:45 PM Thomas Petazzoni
>>> <thomas.petazzoni at bootlin.com> wrote:
>>>> On Mon, 27 Apr 2020 23:22:25 -0600
>>>> James Hilliard <james.hilliard1 at gmail.com> wrote:
>>>>> Signed-off-by: James Hilliard <james.hilliard1 at gmail.com>
>>>>> package/mesa3d/mesa3d.mk | 10 ++++++++--
>>>>> 1 file changed, 8 insertions(+), 2 deletions(-)
>>>> Speaking of mesa3d/valgrind, we have this:
>>>> Seems like mesa3d is trying to link with some static libraries from
>>>> valgrind, which are not built with -fPIC. Have you looked into this?
>>> Yeah, I had added this trying to fix that but it didn't seem to help.
>> I spent some time investigating this issue but the problem is that mesa try to
>> link with libcoregrind-<arch>-linux.a to build libgbm.so. But even by hacking
>> the valgrind build system, the build is still failing.
>> I tried with several toolchain (glibc and uclibc).
>> Out of curiosity, what's your use case that require valgrind in mesa3d ?
> Well I sometimes use valgrind for tracing certain bugs, figured since
> mesa3d has a valgrind option we should use it when building valgrind.
>> Note: The valgrind support is enabled in archlinux and fedora
>> But no problem on their side ?
>> Any idea ? otherwise we have to revert this patch because the build is still
>> failing in the autobuilder.
> The builds were failing before this patch so I don't see how reverting
> would help.
Indeed it's not directly this patch, I thought that because it's a new issue
related to valgrind... but it's more complicated than that. Actually It's a
mesa/valgrind bug. mesa try to build the shared library libgbm.so using the
valgrind static libraries. This should only be possible if valgrind static
libraries are built with -fPIC.
As soon we have BR2_PACKAGE_VALGRIND=y and BR2_PACKAGE_MESA3D_GBM=y, the build
issue should be reproducible.
I can reproduce it with the following defconfig:
Even before your patch, the problem can be reproduced due to libdrm built with
valgrind support. Added long time ago .
Even if -Dvalgrind=false on meson command line to build mesa, the valgrind
libraries come from pkg-config libdrm...
output/host/bin/pkg-config libdrm --libs --static
-L[...]/sysroot/usr/lib -ldrm -lm -L[...]/sysroot/usr/lib/valgrind
-lcoregrind-arm64-linux -lvex-arm64-linux -lgcc
... and break the build.
I tried to modify the valgrind build system to add -fPIC but the problem is
I don't know how yet how to fix the problem.
>> Best regards,
>>>> Thomas Petazzoni, CTO, Bootlin
>>>> Embedded Linux and Kernel engineering
More information about the buildroot