Mesa (main): freedreno/a6xx: Emit a null descriptor for unoccupied IBO slots.
GitLab Mirror
gitlab-mirror at kemper.freedesktop.org
Wed Nov 3 20:02:41 UTC 2021
Module: Mesa
Branch: main
Commit: 7b578c12497cfea2249467951e270993462ff4a9
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=7b578c12497cfea2249467951e270993462ff4a9
Author: Emma Anholt <emma at anholt.net>
Date: Mon Oct 25 12:57:15 2021 -0700
freedreno/a6xx: Emit a null descriptor for unoccupied IBO slots.
Fixes a crash in some desktop GL testcases in piglit.
Cc: mesa-stable
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/13635>
---
src/freedreno/ci/freedreno-a630-fails.txt | 8 +++-----
src/gallium/drivers/freedreno/a6xx/fd6_image.c | 6 ++++++
2 files changed, 9 insertions(+), 5 deletions(-)
diff --git a/src/freedreno/ci/freedreno-a630-fails.txt b/src/freedreno/ci/freedreno-a630-fails.txt
index f2671ee4486..7db977aa3b7 100644
--- a/src/freedreno/ci/freedreno-a630-fails.txt
+++ b/src/freedreno/ci/freedreno-a630-fails.txt
@@ -246,9 +246,6 @@ spec at arb_texture_rg@texwrap formats-int bordercolor-swizzled at GL_RG32UI- swizzled
spec at arb_texture_rg@texwrap formats-int bordercolor-swizzled at GL_RG8I- swizzled- border color only,Fail
spec at arb_texture_rg@texwrap formats-int bordercolor-swizzled at GL_RG8UI- swizzled- border color only,Fail
-# segfault on fd6_emit_image_tex() of null image resource
-spec at arb_texture_view@bug-layers-image,Crash
-
spec at arb_texture_view@mipgen,Fail
spec at arb_texture_view@rendering-layers-image,Fail
spec at arb_texture_view@rendering-layers-image at layers rendering of image1DArray,Fail
@@ -508,8 +505,9 @@ spec at nv_copy_depth_to_color@nv_copy_depth_to_color 1 0x223344ff,Crash
spec at nv_copy_depth_to_color@nv_copy_depth_to_color 1 0x76356278,Crash
spec at nv_copy_depth_to_color@nv_copy_depth_to_color,Crash
-# segfault on fd6_emit_image_tex() of null image resource
-spec at oes_egl_image_external_essl3@oes_egl_image_external_essl3,Crash
+spec at oes_egl_image_external_essl3@oes_egl_image_external_essl3 at oes_egl_image_external_essl3_imageLoad,Fail
+spec at oes_egl_image_external_essl3@oes_egl_image_external_essl3 at oes_egl_image_external_essl3_imageStore,Fail
+spec at oes_egl_image_external_essl3@oes_egl_image_external_essl3,Fail
# "MESA: error: unknown vertex shader output name: VARYING_SLOT_EDGE
# gl-2.0-edgeflag: ../src/freedreno/ir3/ir3_context.c:411: ir3_context_error: Assertion `!""' failed."
diff --git a/src/gallium/drivers/freedreno/a6xx/fd6_image.c b/src/gallium/drivers/freedreno/a6xx/fd6_image.c
index 61c6ccbee78..347d83a239a 100644
--- a/src/gallium/drivers/freedreno/a6xx/fd6_image.c
+++ b/src/gallium/drivers/freedreno/a6xx/fd6_image.c
@@ -174,6 +174,12 @@ translate_buf(struct fd6_image *img, const struct pipe_shader_buffer *pimg)
static void
emit_image_tex(struct fd_ringbuffer *ring, struct fd6_image *img)
{
+ if (!img->prsc) {
+ for (int i = 0; i < FDL6_TEX_CONST_DWORDS; i++)
+ OUT_RING(ring, 0);
+ return;
+ }
+
struct fd_resource *rsc = fd_resource(img->prsc);
bool ubwc_enabled = fd_resource_ubwc_enabled(rsc, img->level);
More information about the mesa-commit
mailing list