Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH] package/mesa3d: fix glxinfo crash when gles1 is disabled
@ 2023-05-18 21:30 Romain Naour
  2023-05-19  3:35 ` Baruch Siach via buildroot
  2023-05-21 17:51 ` Yann E. MORIN
  0 siblings, 2 replies; 4+ messages in thread
From: Romain Naour @ 2023-05-18 21:30 UTC (permalink / raw)
  To: buildroot; +Cc: Bernd Kuhls, Romain Naour

mesa3d 23.1 introduced a regression crashing glxinfo (and some piglit tests)
when gles1 is disabled.
See: https://gitlab.freedesktop.org/mesa/mesa/-/issues/9038

Fixes:
https://gitlab.com/buildroot.org/buildroot/-/jobs/4287983490

Signed-off-by: Romain Naour <romain.naour@gmail.com>
Cc: Bernd Kuhls <bernd.kuhls@t-online.de>
---
 .checkpackageignore                           |  1 +
 ...arly-when-trying-to-create-an-unsupp.patch | 50 +++++++++++++++++++
 2 files changed, 51 insertions(+)
 create mode 100644 package/mesa3d/0005-mesa-main-Exit-early-when-trying-to-create-an-unsupp.patch

diff --git a/.checkpackageignore b/.checkpackageignore
index 947f4606aa..eb572de9f1 100644
--- a/.checkpackageignore
+++ b/.checkpackageignore
@@ -995,6 +995,7 @@ package/mender-connect/S43mender-connect Shellcheck
 package/menu-cache/0001-Support-gcc10-compilation.patch Upstream
 package/mesa3d-demos/0001-demos-makes-opengl-an-optional-component.patch Upstream
 package/mesa3d/0001-meson-Set-proper-value-for-LIBCLC_INCLUDEDIR.patch Upstream
+package/mesa3d/0005-mesa-main-Exit-early-when-trying-to-create-an-unsupp.patch Upstream
 package/meson-tools/0001-amlbootenc-gxl-remove-non-std-C-convention-in-for.patch Upstream
 package/meson/0001-Prefer-ext-static-libs-when-default-library-static.patch Upstream
 package/meson/0002-mesonbuild-dependencies-base.py-add-pkg_config_stati.patch Upstream
diff --git a/package/mesa3d/0005-mesa-main-Exit-early-when-trying-to-create-an-unsupp.patch b/package/mesa3d/0005-mesa-main-Exit-early-when-trying-to-create-an-unsupp.patch
new file mode 100644
index 0000000000..6080a551cc
--- /dev/null
+++ b/package/mesa3d/0005-mesa-main-Exit-early-when-trying-to-create-an-unsupp.patch
@@ -0,0 +1,50 @@
+From 2a6908ff4c94284b39c3cd4c97e1069876720eb7 Mon Sep 17 00:00:00 2001
+From: Jordan Justen <jordan.l.justen@intel.com>
+Date: Tue, 16 May 2023 18:46:50 -0700
+Subject: [PATCH] mesa/main: Exit early when trying to create an unsupported
+ context API
+
+Fixes: adbe8b6c17a ("mesa: optimize out _mesa_is_desktop_gl*() and _mesa_is_gles*() calls when not built")
+Closes: https://gitlab.freedesktop.org/mesa/mesa/-/issues/9038
+Signed-off-by: Jordan Justen <jordan.l.justen@intel.com>
+Reviewed-by: Eric Engestrom <eric@engestrom.ch>
+Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/23068>
+(cherry picked from commit 8bb1ecaa02177720758255bdd7ec34a5d15feca4)
+[Romain: backport to 23.1]
+Signed-off-by: Romain Naour <romain.naour@gmail.com>
+---
+ src/mesa/main/context.c | 18 ++++++++++++++++++
+ 1 file changed, 18 insertions(+)
+
+diff --git a/src/mesa/main/context.c b/src/mesa/main/context.c
+index d8eea2ea867..2b810b0d863 100644
+--- a/src/mesa/main/context.c
++++ b/src/mesa/main/context.c
+@@ -996,6 +996,24 @@ _mesa_initialize_context(struct gl_context *ctx,
+    struct gl_shared_state *shared;
+    int i;
+ 
++   switch (api) {
++   case API_OPENGL_COMPAT:
++   case API_OPENGL_CORE:
++      if (!HAVE_OPENGL)
++         return GL_FALSE;
++      break;
++   case API_OPENGLES2:
++      if (!HAVE_OPENGL_ES_2)
++         return GL_FALSE;
++      break;
++   case API_OPENGLES:
++      if (!HAVE_OPENGL_ES_1)
++         return GL_FALSE;
++      break;
++   default:
++      return GL_FALSE;
++   }
++
+    ctx->API = api;
+    ctx->DrawBuffer = NULL;
+    ctx->ReadBuffer = NULL;
+-- 
+2.40.1
+
-- 
2.40.1

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply related	[flat|nested] 4+ messages in thread
* Re: [Buildroot] [PATCH] package/mesa3d: fix glxinfo crash when gles1 is disabled
  2023-05-18 21:30 [Buildroot] [PATCH] package/mesa3d: fix glxinfo crash when gles1 is disabled Romain Naour
@ 2023-05-19  3:35 ` Baruch Siach via buildroot
  2023-05-19 21:13   ` Romain Naour
  2023-05-21 17:51 ` Yann E. MORIN
  1 sibling, 1 reply; 4+ messages in thread
From: Baruch Siach via buildroot @ 2023-05-19  3:35 UTC (permalink / raw)
  To: Romain Naour; +Cc: Bernd Kuhls, buildroot

Hi Romain,

On Thu, May 18 2023, Romain Naour wrote:
> mesa3d 23.1 introduced a regression crashing glxinfo (and some piglit tests)
> when gles1 is disabled.
> See: https://gitlab.freedesktop.org/mesa/mesa/-/issues/9038
>
> Fixes:
> https://gitlab.com/buildroot.org/buildroot/-/jobs/4287983490
>
> Signed-off-by: Romain Naour <romain.naour@gmail.com>
> Cc: Bernd Kuhls <bernd.kuhls@t-online.de>
> ---
>  .checkpackageignore                           |  1 +
>  ...arly-when-trying-to-create-an-unsupp.patch | 50 +++++++++++++++++++
>  2 files changed, 51 insertions(+)
>  create mode 100644 package/mesa3d/0005-mesa-main-Exit-early-when-trying-to-create-an-unsupp.patch
>
> diff --git a/.checkpackageignore b/.checkpackageignore
> index 947f4606aa..eb572de9f1 100644
> --- a/.checkpackageignore
> +++ b/.checkpackageignore
> @@ -995,6 +995,7 @@ package/mender-connect/S43mender-connect Shellcheck
>  package/menu-cache/0001-Support-gcc10-compilation.patch Upstream
>  package/mesa3d-demos/0001-demos-makes-opengl-an-optional-component.patch Upstream
>  package/mesa3d/0001-meson-Set-proper-value-for-LIBCLC_INCLUDEDIR.patch Upstream
> +package/mesa3d/0005-mesa-main-Exit-early-when-trying-to-create-an-unsupp.patch Upstream

I think the patch should have proper Upstream tag as described in the
manual (docs/manual/patch-policy.txt). .checkpackageignore is only for
existing non conforming patches.

baruch

>  package/meson-tools/0001-amlbootenc-gxl-remove-non-std-C-convention-in-for.patch Upstream
>  package/meson/0001-Prefer-ext-static-libs-when-default-library-static.patch Upstream
>  package/meson/0002-mesonbuild-dependencies-base.py-add-pkg_config_stati.patch Upstream
> diff --git
> a/package/mesa3d/0005-mesa-main-Exit-early-when-trying-to-create-an-unsupp.patch
> b/package/mesa3d/0005-mesa-main-Exit-early-when-trying-to-create-an-unsupp.patch
> new file mode 100644
> index 0000000000..6080a551cc
> --- /dev/null
> +++ b/package/mesa3d/0005-mesa-main-Exit-early-when-trying-to-create-an-unsupp.patch
> @@ -0,0 +1,50 @@
> +From 2a6908ff4c94284b39c3cd4c97e1069876720eb7 Mon Sep 17 00:00:00 2001
> +From: Jordan Justen <jordan.l.justen@intel.com>
> +Date: Tue, 16 May 2023 18:46:50 -0700
> +Subject: [PATCH] mesa/main: Exit early when trying to create an unsupported
> + context API
> +
> +Fixes: adbe8b6c17a ("mesa: optimize out _mesa_is_desktop_gl*() and _mesa_is_gles*() calls when not built")
> +Closes: https://gitlab.freedesktop.org/mesa/mesa/-/issues/9038
> +Signed-off-by: Jordan Justen <jordan.l.justen@intel.com>
> +Reviewed-by: Eric Engestrom <eric@engestrom.ch>
> +Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/23068>
> +(cherry picked from commit 8bb1ecaa02177720758255bdd7ec34a5d15feca4)
> +[Romain: backport to 23.1]
> +Signed-off-by: Romain Naour <romain.naour@gmail.com>
> +---
> + src/mesa/main/context.c | 18 ++++++++++++++++++
> + 1 file changed, 18 insertions(+)
> +
> +diff --git a/src/mesa/main/context.c b/src/mesa/main/context.c
> +index d8eea2ea867..2b810b0d863 100644
> +--- a/src/mesa/main/context.c
> ++++ b/src/mesa/main/context.c
> +@@ -996,6 +996,24 @@ _mesa_initialize_context(struct gl_context *ctx,
> +    struct gl_shared_state *shared;
> +    int i;
> + 
> ++   switch (api) {
> ++   case API_OPENGL_COMPAT:
> ++   case API_OPENGL_CORE:
> ++      if (!HAVE_OPENGL)
> ++         return GL_FALSE;
> ++      break;
> ++   case API_OPENGLES2:
> ++      if (!HAVE_OPENGL_ES_2)
> ++         return GL_FALSE;
> ++      break;
> ++   case API_OPENGLES:
> ++      if (!HAVE_OPENGL_ES_1)
> ++         return GL_FALSE;
> ++      break;
> ++   default:
> ++      return GL_FALSE;
> ++   }
> ++
> +    ctx->API = api;
> +    ctx->DrawBuffer = NULL;
> +    ctx->ReadBuffer = NULL;
> +-- 
> +2.40.1
> +


-- 
                                                     ~. .~   Tk Open Systems
=}------------------------------------------------ooO--U--Ooo------------{=
   - baruch@tkos.co.il - tel: +972.52.368.4656, http://www.tkos.co.il -
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 4+ messages in thread
* Re: [Buildroot] [PATCH] package/mesa3d: fix glxinfo crash when gles1 is disabled
  2023-05-19  3:35 ` Baruch Siach via buildroot
@ 2023-05-19 21:13   ` Romain Naour
  0 siblings, 0 replies; 4+ messages in thread
From: Romain Naour @ 2023-05-19 21:13 UTC (permalink / raw)
  To: Baruch Siach; +Cc: Bernd Kuhls, buildroot

Hi Baruch,

Le 19/05/2023 à 05:35, Baruch Siach a écrit :
> Hi Romain,
> 
> On Thu, May 18 2023, Romain Naour wrote:
>> mesa3d 23.1 introduced a regression crashing glxinfo (and some piglit tests)
>> when gles1 is disabled.
>> See: https://gitlab.freedesktop.org/mesa/mesa/-/issues/9038
>>
>> Fixes:
>> https://gitlab.com/buildroot.org/buildroot/-/jobs/4287983490
>>
>> Signed-off-by: Romain Naour <romain.naour@gmail.com>
>> Cc: Bernd Kuhls <bernd.kuhls@t-online.de>
>> ---
>>  .checkpackageignore                           |  1 +
>>  ...arly-when-trying-to-create-an-unsupp.patch | 50 +++++++++++++++++++
>>  2 files changed, 51 insertions(+)
>>  create mode 100644 package/mesa3d/0005-mesa-main-Exit-early-when-trying-to-create-an-unsupp.patch
>>
>> diff --git a/.checkpackageignore b/.checkpackageignore
>> index 947f4606aa..eb572de9f1 100644
>> --- a/.checkpackageignore
>> +++ b/.checkpackageignore
>> @@ -995,6 +995,7 @@ package/mender-connect/S43mender-connect Shellcheck
>>  package/menu-cache/0001-Support-gcc10-compilation.patch Upstream
>>  package/mesa3d-demos/0001-demos-makes-opengl-an-optional-component.patch Upstream
>>  package/mesa3d/0001-meson-Set-proper-value-for-LIBCLC_INCLUDEDIR.patch Upstream
>> +package/mesa3d/0005-mesa-main-Exit-early-when-trying-to-create-an-unsupp.patch Upstream
> 
> I think the patch should have proper Upstream tag as described in the
> manual (docs/manual/patch-policy.txt). .checkpackageignore is only for
> existing non conforming patches.

Ok thanks, I haven't closely followed recent changes about patch policy.
I thought that it was enough with "(cherry picked from commit ..." (generated by
git cherry-pick -x) to avoid the checkpackage warning.

Best regards,
Romain


> 
> baruch
> 
>>  package/meson-tools/0001-amlbootenc-gxl-remove-non-std-C-convention-in-for.patch Upstream
>>  package/meson/0001-Prefer-ext-static-libs-when-default-library-static.patch Upstream
>>  package/meson/0002-mesonbuild-dependencies-base.py-add-pkg_config_stati.patch Upstream
>> diff --git
>> a/package/mesa3d/0005-mesa-main-Exit-early-when-trying-to-create-an-unsupp.patch
>> b/package/mesa3d/0005-mesa-main-Exit-early-when-trying-to-create-an-unsupp.patch
>> new file mode 100644
>> index 0000000000..6080a551cc
>> --- /dev/null
>> +++ b/package/mesa3d/0005-mesa-main-Exit-early-when-trying-to-create-an-unsupp.patch
>> @@ -0,0 +1,50 @@
>> +From 2a6908ff4c94284b39c3cd4c97e1069876720eb7 Mon Sep 17 00:00:00 2001
>> +From: Jordan Justen <jordan.l.justen@intel.com>
>> +Date: Tue, 16 May 2023 18:46:50 -0700
>> +Subject: [PATCH] mesa/main: Exit early when trying to create an unsupported
>> + context API
>> +
>> +Fixes: adbe8b6c17a ("mesa: optimize out _mesa_is_desktop_gl*() and _mesa_is_gles*() calls when not built")
>> +Closes: https://gitlab.freedesktop.org/mesa/mesa/-/issues/9038
>> +Signed-off-by: Jordan Justen <jordan.l.justen@intel.com>
>> +Reviewed-by: Eric Engestrom <eric@engestrom.ch>
>> +Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/23068>
>> +(cherry picked from commit 8bb1ecaa02177720758255bdd7ec34a5d15feca4)
>> +[Romain: backport to 23.1]
>> +Signed-off-by: Romain Naour <romain.naour@gmail.com>
>> +---
>> + src/mesa/main/context.c | 18 ++++++++++++++++++
>> + 1 file changed, 18 insertions(+)
>> +
>> +diff --git a/src/mesa/main/context.c b/src/mesa/main/context.c
>> +index d8eea2ea867..2b810b0d863 100644
>> +--- a/src/mesa/main/context.c
>> ++++ b/src/mesa/main/context.c
>> +@@ -996,6 +996,24 @@ _mesa_initialize_context(struct gl_context *ctx,
>> +    struct gl_shared_state *shared;
>> +    int i;
>> + 
>> ++   switch (api) {
>> ++   case API_OPENGL_COMPAT:
>> ++   case API_OPENGL_CORE:
>> ++      if (!HAVE_OPENGL)
>> ++         return GL_FALSE;
>> ++      break;
>> ++   case API_OPENGLES2:
>> ++      if (!HAVE_OPENGL_ES_2)
>> ++         return GL_FALSE;
>> ++      break;
>> ++   case API_OPENGLES:
>> ++      if (!HAVE_OPENGL_ES_1)
>> ++         return GL_FALSE;
>> ++      break;
>> ++   default:
>> ++      return GL_FALSE;
>> ++   }
>> ++
>> +    ctx->API = api;
>> +    ctx->DrawBuffer = NULL;
>> +    ctx->ReadBuffer = NULL;
>> +-- 
>> +2.40.1
>> +
> 
> 

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 4+ messages in thread
* Re: [Buildroot] [PATCH] package/mesa3d: fix glxinfo crash when gles1 is disabled
  2023-05-18 21:30 [Buildroot] [PATCH] package/mesa3d: fix glxinfo crash when gles1 is disabled Romain Naour
  2023-05-19  3:35 ` Baruch Siach via buildroot
@ 2023-05-21 17:51 ` Yann E. MORIN
  1 sibling, 0 replies; 4+ messages in thread
From: Yann E. MORIN @ 2023-05-21 17:51 UTC (permalink / raw)
  To: Romain Naour; +Cc: Bernd Kuhls, buildroot

Romain, All,

On 2023-05-18 23:30 +0200, Romain Naour spake thusly:
> mesa3d 23.1 introduced a regression crashing glxinfo (and some piglit tests)
> when gles1 is disabled.
> See: https://gitlab.freedesktop.org/mesa/mesa/-/issues/9038
> 
> Fixes:
> https://gitlab.com/buildroot.org/buildroot/-/jobs/4287983490
> 
> Signed-off-by: Romain Naour <romain.naour@gmail.com>
> Cc: Bernd Kuhls <bernd.kuhls@t-online.de>
> ---
>  .checkpackageignore                           |  1 +
>  ...arly-when-trying-to-create-an-unsupp.patch | 50 +++++++++++++++++++
>  2 files changed, 51 insertions(+)
>  create mode 100644 package/mesa3d/0005-mesa-main-Exit-early-when-trying-to-create-an-unsupp.patch
> 
> diff --git a/.checkpackageignore b/.checkpackageignore
> index 947f4606aa..eb572de9f1 100644
> --- a/.checkpackageignore
> +++ b/.checkpackageignore
> @@ -995,6 +995,7 @@ package/mender-connect/S43mender-connect Shellcheck
>  package/menu-cache/0001-Support-gcc10-compilation.patch Upstream
>  package/mesa3d-demos/0001-demos-makes-opengl-an-optional-component.patch Upstream
>  package/mesa3d/0001-meson-Set-proper-value-for-LIBCLC_INCLUDEDIR.patch Upstream
> +package/mesa3d/0005-mesa-main-Exit-early-when-trying-to-create-an-unsupp.patch Upstream

Do not add a (patch) entry to .checkpackagignore; this ignore list is
only for exsiting files, while new fiels must adhere to codign style
(unless dully explained and justified in the commit log).

In this case, as Baruch noticed, the issue was just about having a
proper "Upstream:" tag, see below..

>  package/meson-tools/0001-amlbootenc-gxl-remove-non-std-C-convention-in-for.patch Upstream
>  package/meson/0001-Prefer-ext-static-libs-when-default-library-static.patch Upstream
>  package/meson/0002-mesonbuild-dependencies-base.py-add-pkg_config_stati.patch Upstream
> diff --git a/package/mesa3d/0005-mesa-main-Exit-early-when-trying-to-create-an-unsupp.patch b/package/mesa3d/0005-mesa-main-Exit-early-when-trying-to-create-an-unsupp.patch
> new file mode 100644
> index 0000000000..6080a551cc
> --- /dev/null
> +++ b/package/mesa3d/0005-mesa-main-Exit-early-when-trying-to-create-an-unsupp.patch
> @@ -0,0 +1,50 @@
> +From 2a6908ff4c94284b39c3cd4c97e1069876720eb7 Mon Sep 17 00:00:00 2001
> +From: Jordan Justen <jordan.l.justen@intel.com>
> +Date: Tue, 16 May 2023 18:46:50 -0700
> +Subject: [PATCH] mesa/main: Exit early when trying to create an unsupported
> + context API
> +
> +Fixes: adbe8b6c17a ("mesa: optimize out _mesa_is_desktop_gl*() and _mesa_is_gles*() calls when not built")
> +Closes: https://gitlab.freedesktop.org/mesa/mesa/-/issues/9038
> +Signed-off-by: Jordan Justen <jordan.l.justen@intel.com>
> +Reviewed-by: Eric Engestrom <eric@engestrom.ch>
> +Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/23068>
> +(cherry picked from commit 8bb1ecaa02177720758255bdd7ec34a5d15feca4)

The recently added rule is to have the upstream status denoted with an
"Upstream:" tag:

    https://buildroot.org/downloads/manual/manual.html#_additional_patch_documentation

So I fixed that, and applied to master, thanks.

Regards,
Yann E. MORIN.

> +[Romain: backport to 23.1]
> +Signed-off-by: Romain Naour <romain.naour@gmail.com>
> +---
> + src/mesa/main/context.c | 18 ++++++++++++++++++
> + 1 file changed, 18 insertions(+)
> +
> +diff --git a/src/mesa/main/context.c b/src/mesa/main/context.c
> +index d8eea2ea867..2b810b0d863 100644
> +--- a/src/mesa/main/context.c
> ++++ b/src/mesa/main/context.c
> +@@ -996,6 +996,24 @@ _mesa_initialize_context(struct gl_context *ctx,
> +    struct gl_shared_state *shared;
> +    int i;
> + 
> ++   switch (api) {
> ++   case API_OPENGL_COMPAT:
> ++   case API_OPENGL_CORE:
> ++      if (!HAVE_OPENGL)
> ++         return GL_FALSE;
> ++      break;
> ++   case API_OPENGLES2:
> ++      if (!HAVE_OPENGL_ES_2)
> ++         return GL_FALSE;
> ++      break;
> ++   case API_OPENGLES:
> ++      if (!HAVE_OPENGL_ES_1)
> ++         return GL_FALSE;
> ++      break;
> ++   default:
> ++      return GL_FALSE;
> ++   }
> ++
> +    ctx->API = api;
> +    ctx->DrawBuffer = NULL;
> +    ctx->ReadBuffer = NULL;
> +-- 
> +2.40.1
> +
> -- 
> 2.40.1
> 
> _______________________________________________
> buildroot mailing list
> buildroot@buildroot.org
> https://lists.buildroot.org/mailman/listinfo/buildroot

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 561 099 427 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 4+ messages in thread
end of thread, other threads:[~2023-05-21 17:52 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-05-18 21:30 [Buildroot] [PATCH] package/mesa3d: fix glxinfo crash when gles1 is disabled Romain Naour
2023-05-19  3:35 ` Baruch Siach via buildroot
2023-05-19 21:13   ` Romain Naour
2023-05-21 17:51 ` Yann E. MORIN

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox