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

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

[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 <anuj.mittal@intel.com>
---
 .../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 <alistair@alistair23.me>
 
-- 
2.29.2

[PATCH 2/2] mesa: remove patch disabling asm

[Anuj Mittal]

This was originally added for musl only [1]. Upstream has removed this
option but we reverted that change during an upgrade to fix certain
musl specific compile time issues [2].

I compile tested without this patch on musl-x86 and musl-x86-64 and
don't see the warnings anymore. Similar textrel issues were fixed
upstream so perhaps those helped [3].

[1] https://git.yoctoproject.org/cgit/cgit.cgi/poky/commit?id=68bf128e901e89d5b49a28b15d5083b2bf6b536f
[2] https://www.openembedded.org/pipermail/openembedded-core/2019-November/289222.html
[3] https://gitlab.freedesktop.org/mesa/mesa/-/commit/45206d7673adb1484cbdb3eadaf82e0849c9cdcf?merge_request_iid=1974

Signed-off-by: Anuj Mittal <anuj.mittal@intel.com>
---
 ...le-asm-unconditionally-now-that-gen_.patch | 147 ------------------
 meta/recipes-graphics/mesa/mesa.inc           |   4 -
 2 files changed, 151 deletions(-)
 delete mode 100644 meta/recipes-graphics/mesa/files/0004-Revert-mesa-Enable-asm-unconditionally-now-that-gen_.patch

diff --git a/meta/recipes-graphics/mesa/files/0004-Revert-mesa-Enable-asm-unconditionally-now-that-gen_.patch b/meta/recipes-graphics/mesa/files/0004-Revert-mesa-Enable-asm-unconditionally-now-that-gen_.patch
deleted file mode 100644
index 833742359f..0000000000
--- a/meta/recipes-graphics/mesa/files/0004-Revert-mesa-Enable-asm-unconditionally-now-that-gen_.patch
+++ /dev/null
@@ -1,147 +0,0 @@
-From 43d9e40db7357f27e91002b2bb7688b6775ebb43 Mon Sep 17 00:00:00 2001
-From: Alistair Francis <alistair@alistair23.me>
-Date: Thu, 14 Nov 2019 09:06:02 -0800
-Subject: [PATCH] Revert "mesa: Enable asm unconditionally, now that
- gen_matypes is gone."
-
-This reverts commit 20294dceebc23236e33b22578245f7e6f41b6997.
-
-Upstream-Status: Inappropriate [configuration]
-Signed-off-by: Alistair Francis <alistair@alistair23.me>
-
----
- meson.build       | 94 ++++++++++++++++++++++++++++++-----------------
- meson_options.txt |  6 +++
- 2 files changed, 67 insertions(+), 33 deletions(-)
-
-diff --git a/meson.build b/meson.build
-index e7dc599..e2fc934 100644
---- a/meson.build
-+++ b/meson.build
-@@ -52,6 +52,7 @@ pre_args = [
- with_vulkan_icd_dir = get_option('vulkan-icd-dir')
- with_tests = get_option('build-tests')
- with_aco_tests = get_option('build-aco-tests')
-+with_asm = get_option('asm')
- with_glx_read_only_text = get_option('glx-read-only-text')
- with_glx_direct = get_option('glx-direct')
- with_osmesa = get_option('osmesa')
-@@ -1154,41 +1155,68 @@ dep_ws2_32 = cc.find_library('ws2_32', required : with_platform_windows)
- 
- # TODO: shared/static? Is this even worth doing?
- 
-+# When cross compiling we generally need to turn off the use of assembly,
-+# because mesa's assembly relies on building an executable for the host system,
-+# and running it to get information about struct sizes. There is at least one
-+# case of cross compiling where we can use asm, and that's x86_64 -> x86 when
-+# host OS == build OS, since in that case the build machine can run the host's
-+# binaries.
-+if with_asm and meson.is_cross_build()
-+  if build_machine.system() != host_machine.system()
-+    # TODO: It may be possible to do this with an exe_wrapper (like wine).
-+    message('Cross compiling from one OS to another, disabling assembly.')
-+    with_asm = false
-+  elif not (build_machine.cpu_family().startswith('x86') and host_machine.cpu_family() == 'x86')
-+    # FIXME: Gentoo always sets -m32 for x86_64 -> x86 builds, resulting in an
-+    # x86 -> x86 cross compile. We use startswith rather than == to handle this
-+    # case.
-+    # TODO: There may be other cases where the 64 bit version of the
-+    # architecture can run 32 bit binaries (aarch64 and armv7 for example)
-+    message('''
-+      Cross compiling to different architectures, and the host cannot run
-+      the build machine's binaries. Disabling assembly.
-+    ''')
-+    with_asm = false
-+  endif
-+endif
-+
- with_asm_arch = ''
--if host_machine.cpu_family() == 'x86'
--  if system_has_kms_drm or host_machine.system() == 'gnu'
--    with_asm_arch = 'x86'
--    pre_args += ['-DUSE_X86_ASM', '-DUSE_MMX_ASM', '-DUSE_3DNOW_ASM',
--                 '-DUSE_SSE_ASM']
--
--    if with_glx_read_only_text
--      pre_args += ['-DGLX_X86_READONLY_TEXT']
-+if with_asm
-+  if host_machine.cpu_family() == 'x86'
-+    if system_has_kms_drm or host_machine.system() == 'gnu'
-+      with_asm_arch = 'x86'
-+      pre_args += ['-DUSE_X86_ASM', '-DUSE_MMX_ASM', '-DUSE_3DNOW_ASM',
-+                   '-DUSE_SSE_ASM']
-+
-+      if with_glx_read_only_text
-+         pre_args += ['-DGLX_X86_READONLY_TEXT']
-+      endif
-+    endif
-+  elif host_machine.cpu_family() == 'x86_64'
-+    if system_has_kms_drm
-+      with_asm_arch = 'x86_64'
-+      pre_args += ['-DUSE_X86_64_ASM']
-+    endif
-+  elif host_machine.cpu_family() == 'arm'
-+    if system_has_kms_drm
-+      with_asm_arch = 'arm'
-+      pre_args += ['-DUSE_ARM_ASM']
-+    endif
-+  elif host_machine.cpu_family() == 'aarch64'
-+    if system_has_kms_drm
-+      with_asm_arch = 'aarch64'
-+      pre_args += ['-DUSE_AARCH64_ASM']
-+    endif
-+  elif host_machine.cpu_family() == 'sparc64'
-+    if system_has_kms_drm
-+      with_asm_arch = 'sparc'
-+      pre_args += ['-DUSE_SPARC_ASM']
-+    endif
-+  elif host_machine.cpu_family().startswith('ppc64') and host_machine.endian() == 'little'
-+    if system_has_kms_drm
-+      with_asm_arch = 'ppc64le'
-+      pre_args += ['-DUSE_PPC64LE_ASM']
-     endif
--  endif
--elif host_machine.cpu_family() == 'x86_64'
--  if system_has_kms_drm
--    with_asm_arch = 'x86_64'
--    pre_args += ['-DUSE_X86_64_ASM']
--  endif
--elif host_machine.cpu_family() == 'arm'
--  if system_has_kms_drm
--    with_asm_arch = 'arm'
--    pre_args += ['-DUSE_ARM_ASM']
--  endif
--elif host_machine.cpu_family() == 'aarch64'
--  if system_has_kms_drm
--    with_asm_arch = 'aarch64'
--    pre_args += ['-DUSE_AARCH64_ASM']
--  endif
--elif host_machine.cpu_family() == 'sparc64'
--  if system_has_kms_drm
--    with_asm_arch = 'sparc'
--    pre_args += ['-DUSE_SPARC_ASM']
--  endif
--elif host_machine.cpu_family().startswith('ppc64') and host_machine.endian() == 'little'
--  if system_has_kms_drm
--    with_asm_arch = 'ppc64le'
--    pre_args += ['-DUSE_PPC64LE_ASM']
-   endif
- endif
- 
-diff --git a/meson_options.txt b/meson_options.txt
-index 147cccb..562b059 100644
---- a/meson_options.txt
-+++ b/meson_options.txt
-@@ -254,6 +254,12 @@ option(
-   value : false,
-   description : 'Enable GLVND support.'
- )
-+option(
-+  'asm',
-+  type : 'boolean',
-+  value : true,
-+  description : 'Build assembly code if possible'
-+)
- option(
-    'glx-read-only-text',
-    type : 'boolean',
diff --git a/meta/recipes-graphics/mesa/mesa.inc b/meta/recipes-graphics/mesa/mesa.inc
index 7956d95fc1..dba23f586f 100644
--- a/meta/recipes-graphics/mesa/mesa.inc
+++ b/meta/recipes-graphics/mesa/mesa.inc
@@ -17,7 +17,6 @@ PE = "2"
 SRC_URI = "https://mesa.freedesktop.org/archive/mesa-${PV}.tar.xz \
            file://0001-meson.build-check-for-all-linux-host_os-combinations.patch \
            file://0002-meson.build-make-TLS-ELF-optional.patch \
-           file://0004-Revert-mesa-Enable-asm-unconditionally-now-that-gen_.patch \
            file://0005-vc4-use-intmax_t-for-formatted-output-of-timespec-me.patch \
            file://0001-meson-misdetects-64bit-atomics-on-mips-clang.patch \
            file://0001-futex.h-Define-__NR_futex-if-it-does-not-exist.patch \
@@ -178,9 +177,6 @@ PACKAGECONFIG[osmesa] = "-Dosmesa=${OSMESA},-Dosmesa=none"
 
 PACKAGECONFIG[unwind] = "-Dlibunwind=true,-Dlibunwind=false,libunwind"
 
-# mesa tries to run cross-built gen_matypes on build machine to get struct size information
-EXTRA_OEMESON_append = " -Dasm=false"
-
 # llvmpipe is slow if compiled with -fomit-frame-pointer (e.g. -O2)
 FULL_OPTIMIZATION_append = " -fno-omit-frame-pointer"
 
-- 
2.29.2

Re: [OE-core] [PATCH 2/2] mesa: remove patch disabling asm

[Alistair Francis]

On Thu, Dec 10, 2020 at 11:37 PM Anuj Mittal <anuj.mittal@intel.com> wrote:
>
> This was originally added for musl only [1]. Upstream has removed this
> option but we reverted that change during an upgrade to fix certain
> musl specific compile time issues [2].
>
> I compile tested without this patch on musl-x86 and musl-x86-64 and
> don't see the warnings anymore. Similar textrel issues were fixed
> upstream so perhaps those helped [3].
>
> [1] https://git.yoctoproject.org/cgit/cgit.cgi/poky/commit?id=68bf128e901e89d5b49a28b15d5083b2bf6b536f
> [2] https://www.openembedded.org/pipermail/openembedded-core/2019-November/289222.html
> [3] https://gitlab.freedesktop.org/mesa/mesa/-/commit/45206d7673adb1484cbdb3eadaf82e0849c9cdcf?merge_request_iid=1974
>
> Signed-off-by: Anuj Mittal <anuj.mittal@intel.com>

Reviewed-by: Alistair Francis <alistair.francis@wdc.com>

Alistair

> ---
>  ...le-asm-unconditionally-now-that-gen_.patch | 147 ------------------
>  meta/recipes-graphics/mesa/mesa.inc           |   4 -
>  2 files changed, 151 deletions(-)
>  delete mode 100644 meta/recipes-graphics/mesa/files/0004-Revert-mesa-Enable-asm-unconditionally-now-that-gen_.patch
>
> diff --git a/meta/recipes-graphics/mesa/files/0004-Revert-mesa-Enable-asm-unconditionally-now-that-gen_.patch b/meta/recipes-graphics/mesa/files/0004-Revert-mesa-Enable-asm-unconditionally-now-that-gen_.patch
> deleted file mode 100644
> index 833742359f..0000000000
> --- a/meta/recipes-graphics/mesa/files/0004-Revert-mesa-Enable-asm-unconditionally-now-that-gen_.patch
> +++ /dev/null
> @@ -1,147 +0,0 @@
> -From 43d9e40db7357f27e91002b2bb7688b6775ebb43 Mon Sep 17 00:00:00 2001
> -From: Alistair Francis <alistair@alistair23.me>
> -Date: Thu, 14 Nov 2019 09:06:02 -0800
> -Subject: [PATCH] Revert "mesa: Enable asm unconditionally, now that
> - gen_matypes is gone."
> -
> -This reverts commit 20294dceebc23236e33b22578245f7e6f41b6997.
> -
> -Upstream-Status: Inappropriate [configuration]
> -Signed-off-by: Alistair Francis <alistair@alistair23.me>
> -
> ----
> - meson.build       | 94 ++++++++++++++++++++++++++++++-----------------
> - meson_options.txt |  6 +++
> - 2 files changed, 67 insertions(+), 33 deletions(-)
> -
> -diff --git a/meson.build b/meson.build
> -index e7dc599..e2fc934 100644
> ---- a/meson.build
> -+++ b/meson.build
> -@@ -52,6 +52,7 @@ pre_args = [
> - with_vulkan_icd_dir = get_option('vulkan-icd-dir')
> - with_tests = get_option('build-tests')
> - with_aco_tests = get_option('build-aco-tests')
> -+with_asm = get_option('asm')
> - with_glx_read_only_text = get_option('glx-read-only-text')
> - with_glx_direct = get_option('glx-direct')
> - with_osmesa = get_option('osmesa')
> -@@ -1154,41 +1155,68 @@ dep_ws2_32 = cc.find_library('ws2_32', required : with_platform_windows)
> -
> - # TODO: shared/static? Is this even worth doing?
> -
> -+# When cross compiling we generally need to turn off the use of assembly,
> -+# because mesa's assembly relies on building an executable for the host system,
> -+# and running it to get information about struct sizes. There is at least one
> -+# case of cross compiling where we can use asm, and that's x86_64 -> x86 when
> -+# host OS == build OS, since in that case the build machine can run the host's
> -+# binaries.
> -+if with_asm and meson.is_cross_build()
> -+  if build_machine.system() != host_machine.system()
> -+    # TODO: It may be possible to do this with an exe_wrapper (like wine).
> -+    message('Cross compiling from one OS to another, disabling assembly.')
> -+    with_asm = false
> -+  elif not (build_machine.cpu_family().startswith('x86') and host_machine.cpu_family() == 'x86')
> -+    # FIXME: Gentoo always sets -m32 for x86_64 -> x86 builds, resulting in an
> -+    # x86 -> x86 cross compile. We use startswith rather than == to handle this
> -+    # case.
> -+    # TODO: There may be other cases where the 64 bit version of the
> -+    # architecture can run 32 bit binaries (aarch64 and armv7 for example)
> -+    message('''
> -+      Cross compiling to different architectures, and the host cannot run
> -+      the build machine's binaries. Disabling assembly.
> -+    ''')
> -+    with_asm = false
> -+  endif
> -+endif
> -+
> - with_asm_arch = ''
> --if host_machine.cpu_family() == 'x86'
> --  if system_has_kms_drm or host_machine.system() == 'gnu'
> --    with_asm_arch = 'x86'
> --    pre_args += ['-DUSE_X86_ASM', '-DUSE_MMX_ASM', '-DUSE_3DNOW_ASM',
> --                 '-DUSE_SSE_ASM']
> --
> --    if with_glx_read_only_text
> --      pre_args += ['-DGLX_X86_READONLY_TEXT']
> -+if with_asm
> -+  if host_machine.cpu_family() == 'x86'
> -+    if system_has_kms_drm or host_machine.system() == 'gnu'
> -+      with_asm_arch = 'x86'
> -+      pre_args += ['-DUSE_X86_ASM', '-DUSE_MMX_ASM', '-DUSE_3DNOW_ASM',
> -+                   '-DUSE_SSE_ASM']
> -+
> -+      if with_glx_read_only_text
> -+         pre_args += ['-DGLX_X86_READONLY_TEXT']
> -+      endif
> -+    endif
> -+  elif host_machine.cpu_family() == 'x86_64'
> -+    if system_has_kms_drm
> -+      with_asm_arch = 'x86_64'
> -+      pre_args += ['-DUSE_X86_64_ASM']
> -+    endif
> -+  elif host_machine.cpu_family() == 'arm'
> -+    if system_has_kms_drm
> -+      with_asm_arch = 'arm'
> -+      pre_args += ['-DUSE_ARM_ASM']
> -+    endif
> -+  elif host_machine.cpu_family() == 'aarch64'
> -+    if system_has_kms_drm
> -+      with_asm_arch = 'aarch64'
> -+      pre_args += ['-DUSE_AARCH64_ASM']
> -+    endif
> -+  elif host_machine.cpu_family() == 'sparc64'
> -+    if system_has_kms_drm
> -+      with_asm_arch = 'sparc'
> -+      pre_args += ['-DUSE_SPARC_ASM']
> -+    endif
> -+  elif host_machine.cpu_family().startswith('ppc64') and host_machine.endian() == 'little'
> -+    if system_has_kms_drm
> -+      with_asm_arch = 'ppc64le'
> -+      pre_args += ['-DUSE_PPC64LE_ASM']
> -     endif
> --  endif
> --elif host_machine.cpu_family() == 'x86_64'
> --  if system_has_kms_drm
> --    with_asm_arch = 'x86_64'
> --    pre_args += ['-DUSE_X86_64_ASM']
> --  endif
> --elif host_machine.cpu_family() == 'arm'
> --  if system_has_kms_drm
> --    with_asm_arch = 'arm'
> --    pre_args += ['-DUSE_ARM_ASM']
> --  endif
> --elif host_machine.cpu_family() == 'aarch64'
> --  if system_has_kms_drm
> --    with_asm_arch = 'aarch64'
> --    pre_args += ['-DUSE_AARCH64_ASM']
> --  endif
> --elif host_machine.cpu_family() == 'sparc64'
> --  if system_has_kms_drm
> --    with_asm_arch = 'sparc'
> --    pre_args += ['-DUSE_SPARC_ASM']
> --  endif
> --elif host_machine.cpu_family().startswith('ppc64') and host_machine.endian() == 'little'
> --  if system_has_kms_drm
> --    with_asm_arch = 'ppc64le'
> --    pre_args += ['-DUSE_PPC64LE_ASM']
> -   endif
> - endif
> -
> -diff --git a/meson_options.txt b/meson_options.txt
> -index 147cccb..562b059 100644
> ---- a/meson_options.txt
> -+++ b/meson_options.txt
> -@@ -254,6 +254,12 @@ option(
> -   value : false,
> -   description : 'Enable GLVND support.'
> - )
> -+option(
> -+  'asm',
> -+  type : 'boolean',
> -+  value : true,
> -+  description : 'Build assembly code if possible'
> -+)
> - option(
> -    'glx-read-only-text',
> -    type : 'boolean',
> diff --git a/meta/recipes-graphics/mesa/mesa.inc b/meta/recipes-graphics/mesa/mesa.inc
> index 7956d95fc1..dba23f586f 100644
> --- a/meta/recipes-graphics/mesa/mesa.inc
> +++ b/meta/recipes-graphics/mesa/mesa.inc
> @@ -17,7 +17,6 @@ PE = "2"
>  SRC_URI = "https://mesa.freedesktop.org/archive/mesa-${PV}.tar.xz \
>             file://0001-meson.build-check-for-all-linux-host_os-combinations.patch \
>             file://0002-meson.build-make-TLS-ELF-optional.patch \
> -           file://0004-Revert-mesa-Enable-asm-unconditionally-now-that-gen_.patch \
>             file://0005-vc4-use-intmax_t-for-formatted-output-of-timespec-me.patch \
>             file://0001-meson-misdetects-64bit-atomics-on-mips-clang.patch \
>             file://0001-futex.h-Define-__NR_futex-if-it-does-not-exist.patch \
> @@ -178,9 +177,6 @@ PACKAGECONFIG[osmesa] = "-Dosmesa=${OSMESA},-Dosmesa=none"
>
>  PACKAGECONFIG[unwind] = "-Dlibunwind=true,-Dlibunwind=false,libunwind"
>
> -# mesa tries to run cross-built gen_matypes on build machine to get struct size information
> -EXTRA_OEMESON_append = " -Dasm=false"
> -
>  # llvmpipe is slow if compiled with -fomit-frame-pointer (e.g. -O2)
>  FULL_OPTIMIZATION_append = " -fno-omit-frame-pointer"
>
> --
> 2.29.2
>
>
> 
>

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

[Alistair Francis]

On Thu, Dec 10, 2020 at 11:37 PM Anuj Mittal <anuj.mittal@intel.com> 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 <anuj.mittal@intel.com>

Reviewed-by: Alistair Francis <alistair.francis@wdc.com>

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 <alistair@alistair23.me>
>
> --
> 2.29.2
>
>
> 
>

Re: [PATCH 2/2] mesa: remove patch disabling asm

[Khem Raj]

On 12/10/20 11:36 PM, Anuj Mittal wrote:
> This was originally added for musl only [1]. Upstream has removed this
> option but we reverted that change during an upgrade to fix certain
> musl specific compile time issues [2].
> 
> I compile tested without this patch on musl-x86 and musl-x86-64 and
> don't see the warnings anymore. Similar textrel issues were fixed
> upstream so perhaps those helped [3].
> 
> [1] https://git.yoctoproject.org/cgit/cgit.cgi/poky/commit?id=68bf128e901e89d5b49a28b15d5083b2bf6b536f
> [2] https://www.openembedded.org/pipermail/openembedded-core/2019-November/289222.html
> [3] https://gitlab.freedesktop.org/mesa/mesa/-/commit/45206d7673adb1484cbdb3eadaf82e0849c9cdcf?merge_request_iid=1974
> 

Please try running/booting core-image-sato after this change for a musl 
based x86 system as well.

> Signed-off-by: Anuj Mittal <anuj.mittal@intel.com>
> ---
>   ...le-asm-unconditionally-now-that-gen_.patch | 147 ------------------
>   meta/recipes-graphics/mesa/mesa.inc           |   4 -
>   2 files changed, 151 deletions(-)
>   delete mode 100644 meta/recipes-graphics/mesa/files/0004-Revert-mesa-Enable-asm-unconditionally-now-that-gen_.patch
> 
> diff --git a/meta/recipes-graphics/mesa/files/0004-Revert-mesa-Enable-asm-unconditionally-now-that-gen_.patch b/meta/recipes-graphics/mesa/files/0004-Revert-mesa-Enable-asm-unconditionally-now-that-gen_.patch
> deleted file mode 100644
> index 833742359f..0000000000
> --- a/meta/recipes-graphics/mesa/files/0004-Revert-mesa-Enable-asm-unconditionally-now-that-gen_.patch
> +++ /dev/null
> @@ -1,147 +0,0 @@
> -From 43d9e40db7357f27e91002b2bb7688b6775ebb43 Mon Sep 17 00:00:00 2001
> -From: Alistair Francis <alistair@alistair23.me>
> -Date: Thu, 14 Nov 2019 09:06:02 -0800
> -Subject: [PATCH] Revert "mesa: Enable asm unconditionally, now that
> - gen_matypes is gone."
> -
> -This reverts commit 20294dceebc23236e33b22578245f7e6f41b6997.
> -
> -Upstream-Status: Inappropriate [configuration]
> -Signed-off-by: Alistair Francis <alistair@alistair23.me>
> -
> ----
> - meson.build       | 94 ++++++++++++++++++++++++++++++-----------------
> - meson_options.txt |  6 +++
> - 2 files changed, 67 insertions(+), 33 deletions(-)
> -
> -diff --git a/meson.build b/meson.build
> -index e7dc599..e2fc934 100644
> ---- a/meson.build
> -+++ b/meson.build
> -@@ -52,6 +52,7 @@ pre_args = [
> - with_vulkan_icd_dir = get_option('vulkan-icd-dir')
> - with_tests = get_option('build-tests')
> - with_aco_tests = get_option('build-aco-tests')
> -+with_asm = get_option('asm')
> - with_glx_read_only_text = get_option('glx-read-only-text')
> - with_glx_direct = get_option('glx-direct')
> - with_osmesa = get_option('osmesa')
> -@@ -1154,41 +1155,68 @@ dep_ws2_32 = cc.find_library('ws2_32', required : with_platform_windows)
> -
> - # TODO: shared/static? Is this even worth doing?
> -
> -+# When cross compiling we generally need to turn off the use of assembly,
> -+# because mesa's assembly relies on building an executable for the host system,
> -+# and running it to get information about struct sizes. There is at least one
> -+# case of cross compiling where we can use asm, and that's x86_64 -> x86 when
> -+# host OS == build OS, since in that case the build machine can run the host's
> -+# binaries.
> -+if with_asm and meson.is_cross_build()
> -+  if build_machine.system() != host_machine.system()
> -+    # TODO: It may be possible to do this with an exe_wrapper (like wine).
> -+    message('Cross compiling from one OS to another, disabling assembly.')
> -+    with_asm = false
> -+  elif not (build_machine.cpu_family().startswith('x86') and host_machine.cpu_family() == 'x86')
> -+    # FIXME: Gentoo always sets -m32 for x86_64 -> x86 builds, resulting in an
> -+    # x86 -> x86 cross compile. We use startswith rather than == to handle this
> -+    # case.
> -+    # TODO: There may be other cases where the 64 bit version of the
> -+    # architecture can run 32 bit binaries (aarch64 and armv7 for example)
> -+    message('''
> -+      Cross compiling to different architectures, and the host cannot run
> -+      the build machine's binaries. Disabling assembly.
> -+    ''')
> -+    with_asm = false
> -+  endif
> -+endif
> -+
> - with_asm_arch = ''
> --if host_machine.cpu_family() == 'x86'
> --  if system_has_kms_drm or host_machine.system() == 'gnu'
> --    with_asm_arch = 'x86'
> --    pre_args += ['-DUSE_X86_ASM', '-DUSE_MMX_ASM', '-DUSE_3DNOW_ASM',
> --                 '-DUSE_SSE_ASM']
> --
> --    if with_glx_read_only_text
> --      pre_args += ['-DGLX_X86_READONLY_TEXT']
> -+if with_asm
> -+  if host_machine.cpu_family() == 'x86'
> -+    if system_has_kms_drm or host_machine.system() == 'gnu'
> -+      with_asm_arch = 'x86'
> -+      pre_args += ['-DUSE_X86_ASM', '-DUSE_MMX_ASM', '-DUSE_3DNOW_ASM',
> -+                   '-DUSE_SSE_ASM']
> -+
> -+      if with_glx_read_only_text
> -+         pre_args += ['-DGLX_X86_READONLY_TEXT']
> -+      endif
> -+    endif
> -+  elif host_machine.cpu_family() == 'x86_64'
> -+    if system_has_kms_drm
> -+      with_asm_arch = 'x86_64'
> -+      pre_args += ['-DUSE_X86_64_ASM']
> -+    endif
> -+  elif host_machine.cpu_family() == 'arm'
> -+    if system_has_kms_drm
> -+      with_asm_arch = 'arm'
> -+      pre_args += ['-DUSE_ARM_ASM']
> -+    endif
> -+  elif host_machine.cpu_family() == 'aarch64'
> -+    if system_has_kms_drm
> -+      with_asm_arch = 'aarch64'
> -+      pre_args += ['-DUSE_AARCH64_ASM']
> -+    endif
> -+  elif host_machine.cpu_family() == 'sparc64'
> -+    if system_has_kms_drm
> -+      with_asm_arch = 'sparc'
> -+      pre_args += ['-DUSE_SPARC_ASM']
> -+    endif
> -+  elif host_machine.cpu_family().startswith('ppc64') and host_machine.endian() == 'little'
> -+    if system_has_kms_drm
> -+      with_asm_arch = 'ppc64le'
> -+      pre_args += ['-DUSE_PPC64LE_ASM']
> -     endif
> --  endif
> --elif host_machine.cpu_family() == 'x86_64'
> --  if system_has_kms_drm
> --    with_asm_arch = 'x86_64'
> --    pre_args += ['-DUSE_X86_64_ASM']
> --  endif
> --elif host_machine.cpu_family() == 'arm'
> --  if system_has_kms_drm
> --    with_asm_arch = 'arm'
> --    pre_args += ['-DUSE_ARM_ASM']
> --  endif
> --elif host_machine.cpu_family() == 'aarch64'
> --  if system_has_kms_drm
> --    with_asm_arch = 'aarch64'
> --    pre_args += ['-DUSE_AARCH64_ASM']
> --  endif
> --elif host_machine.cpu_family() == 'sparc64'
> --  if system_has_kms_drm
> --    with_asm_arch = 'sparc'
> --    pre_args += ['-DUSE_SPARC_ASM']
> --  endif
> --elif host_machine.cpu_family().startswith('ppc64') and host_machine.endian() == 'little'
> --  if system_has_kms_drm
> --    with_asm_arch = 'ppc64le'
> --    pre_args += ['-DUSE_PPC64LE_ASM']
> -   endif
> - endif
> -
> -diff --git a/meson_options.txt b/meson_options.txt
> -index 147cccb..562b059 100644
> ---- a/meson_options.txt
> -+++ b/meson_options.txt
> -@@ -254,6 +254,12 @@ option(
> -   value : false,
> -   description : 'Enable GLVND support.'
> - )
> -+option(
> -+  'asm',
> -+  type : 'boolean',
> -+  value : true,
> -+  description : 'Build assembly code if possible'
> -+)
> - option(
> -    'glx-read-only-text',
> -    type : 'boolean',
> diff --git a/meta/recipes-graphics/mesa/mesa.inc b/meta/recipes-graphics/mesa/mesa.inc
> index 7956d95fc1..dba23f586f 100644
> --- a/meta/recipes-graphics/mesa/mesa.inc
> +++ b/meta/recipes-graphics/mesa/mesa.inc
> @@ -17,7 +17,6 @@ PE = "2"
>   SRC_URI = "https://mesa.freedesktop.org/archive/mesa-${PV}.tar.xz \
>              file://0001-meson.build-check-for-all-linux-host_os-combinations.patch \
>              file://0002-meson.build-make-TLS-ELF-optional.patch \
> -           file://0004-Revert-mesa-Enable-asm-unconditionally-now-that-gen_.patch \
>              file://0005-vc4-use-intmax_t-for-formatted-output-of-timespec-me.patch \
>              file://0001-meson-misdetects-64bit-atomics-on-mips-clang.patch \
>              file://0001-futex.h-Define-__NR_futex-if-it-does-not-exist.patch \
> @@ -178,9 +177,6 @@ PACKAGECONFIG[osmesa] = "-Dosmesa=${OSMESA},-Dosmesa=none"
>   
>   PACKAGECONFIG[unwind] = "-Dlibunwind=true,-Dlibunwind=false,libunwind"
>   
> -# mesa tries to run cross-built gen_matypes on build machine to get struct size information
> -EXTRA_OEMESON_append = " -Dasm=false"
> -
>   # llvmpipe is slow if compiled with -fomit-frame-pointer (e.g. -O2)
>   FULL_OPTIMIZATION_append = " -fno-omit-frame-pointer"
>   
> 

Re: [PATCH 2/2] mesa: remove patch disabling asm

[Anuj Mittal]

On Fri, 2020-12-11 at 08:41 -0800, Khem Raj wrote:
> 
> 
> On 12/10/20 11:36 PM, Anuj Mittal wrote:
> > This was originally added for musl only [1]. Upstream has removed
> > this
> > option but we reverted that change during an upgrade to fix certain
> > musl specific compile time issues [2].
> > 
> > I compile tested without this patch on musl-x86 and musl-x86-64 and
> > don't see the warnings anymore. Similar textrel issues were fixed
> > upstream so perhaps those helped [3].
> > 
> > [1] 
> > https://git.yoctoproject.org/cgit/cgit.cgi/poky/commit?id=68bf128e901e89d5b49a28b15d5083b2bf6b536f
> > [2] 
> > https://www.openembedded.org/pipermail/openembedded-core/2019-November/289222.html
> > [3] 
> > https://gitlab.freedesktop.org/mesa/mesa/-/commit/45206d7673adb1484cbdb3eadaf82e0849c9cdcf?merge_request_iid=1974
> > 
> 
> Please try running/booting core-image-sato after this change for a
> musl 
> based x86 system as well.

Yeah, I had done that and didn't see any problems.

Thanks,

Anuj