Re: [OE-core] [PATCH 1/2] mesa: add more details to elf-tls patch

2020-12-11 Thread Alistair Francis
On Thu, Dec 10, 2020 at 11:37 PM Anuj Mittal  wrote:
>
> Include link to upstream report and the last autobuilder failure [1]
> so it's easy to identify why this is needed in future.
>
> [1] 
> https://autobuilder.yoctoproject.org/typhoon/#/builders/64/builds/1160/steps/8/logs/step1c
>
> Signed-off-by: Anuj Mittal 

Reviewed-by: Alistair Francis 

Alistair

> ---
>  .../0002-meson.build-make-TLS-ELF-optional.patch  | 15 +++
>  1 file changed, 15 insertions(+)
>
> diff --git 
> a/meta/recipes-graphics/mesa/files/0002-meson.build-make-TLS-ELF-optional.patch
>  
> b/meta/recipes-graphics/mesa/files/0002-meson.build-make-TLS-ELF-optional.patch
> index 74f7fe5c2d..89c0d150fa 100644
> --- 
> a/meta/recipes-graphics/mesa/files/0002-meson.build-make-TLS-ELF-optional.patch
> +++ 
> b/meta/recipes-graphics/mesa/files/0002-meson.build-make-TLS-ELF-optional.patch
> @@ -6,6 +6,21 @@ Subject: [PATCH] meson.build: make TLS ELF optional
>  USE_ELF_TLS has replaced GLX_USE_TLS so this patch is the original "make
>  TLS GLX optional again" patch updated to the latest mesa.
>
> +For details, see:
> +https://gitlab.freedesktop.org/mesa/mesa/-/issues/966
> +
> +This prevents runtime segfault on musl:
> +
> +Traceback (most recent call last):
> +  File 
> "/home/pokybuild/yocto-worker/musl-qemux86/build/meta/lib/oeqa/core/decorator/__init__.py",
>  line 36, in wrapped_f
> +return func(*args, **kwargs)
> +  File 
> "/home/pokybuild/yocto-worker/musl-qemux86/build/meta/lib/oeqa/runtime/cases/parselogs.py",
>  line 378, in test_parselogs
> +self.assertEqual(errcount, 0, msg=self.msg)
> +AssertionError: 1 != 0 : Log: 
> /home/pokybuild/yocto-worker/musl-qemux86/build/build/tmp/work/qemux86-poky-linux-musl/core-image-sato-sdk/1.0-r0/target_logs/Xorg.0.log
> +---
> +Central error: [10.477] (EE) Failed to load 
> /usr/lib/xorg/modules/extensions/libglx.so: Error relocating 
> /usr/lib/libGL.so.1: alphasort: initial-exec TLS resolves to dynamic 
> definition in /usr/lib/libGL.so.1
> +***
> +
>  Upstream-Status: Inappropriate [configuration]
>  Signed-off-by: Alistair Francis 
>
> --
> 2.29.2
>
>
> 
>

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#145473): 
https://lists.openembedded.org/g/openembedded-core/message/145473
Mute This Topic: https://lists.openembedded.org/mt/78874718/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[OE-core] [PATCH 1/2] mesa: add more details to elf-tls patch

2020-12-10 Thread Anuj Mittal
Include link to upstream report and the last autobuilder failure [1]
so it's easy to identify why this is needed in future.

[1] 
https://autobuilder.yoctoproject.org/typhoon/#/builders/64/builds/1160/steps/8/logs/step1c

Signed-off-by: Anuj Mittal 
---
 .../0002-meson.build-make-TLS-ELF-optional.patch  | 15 +++
 1 file changed, 15 insertions(+)

diff --git 
a/meta/recipes-graphics/mesa/files/0002-meson.build-make-TLS-ELF-optional.patch 
b/meta/recipes-graphics/mesa/files/0002-meson.build-make-TLS-ELF-optional.patch
index 74f7fe5c2d..89c0d150fa 100644
--- 
a/meta/recipes-graphics/mesa/files/0002-meson.build-make-TLS-ELF-optional.patch
+++ 
b/meta/recipes-graphics/mesa/files/0002-meson.build-make-TLS-ELF-optional.patch
@@ -6,6 +6,21 @@ Subject: [PATCH] meson.build: make TLS ELF optional
 USE_ELF_TLS has replaced GLX_USE_TLS so this patch is the original "make
 TLS GLX optional again" patch updated to the latest mesa.
 
+For details, see:
+https://gitlab.freedesktop.org/mesa/mesa/-/issues/966
+
+This prevents runtime segfault on musl:
+
+Traceback (most recent call last):
+  File 
"/home/pokybuild/yocto-worker/musl-qemux86/build/meta/lib/oeqa/core/decorator/__init__.py",
 line 36, in wrapped_f
+return func(*args, **kwargs)
+  File 
"/home/pokybuild/yocto-worker/musl-qemux86/build/meta/lib/oeqa/runtime/cases/parselogs.py",
 line 378, in test_parselogs
+self.assertEqual(errcount, 0, msg=self.msg)
+AssertionError: 1 != 0 : Log: 
/home/pokybuild/yocto-worker/musl-qemux86/build/build/tmp/work/qemux86-poky-linux-musl/core-image-sato-sdk/1.0-r0/target_logs/Xorg.0.log
+---
+Central error: [10.477] (EE) Failed to load 
/usr/lib/xorg/modules/extensions/libglx.so: Error relocating 
/usr/lib/libGL.so.1: alphasort: initial-exec TLS resolves to dynamic definition 
in /usr/lib/libGL.so.1
+***
+
 Upstream-Status: Inappropriate [configuration]
 Signed-off-by: Alistair Francis 
 
-- 
2.29.2


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#145466): 
https://lists.openembedded.org/g/openembedded-core/message/145466
Mute This Topic: https://lists.openembedded.org/mt/78874718/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-