llvmpipe: make sure to initialize the lp_setup_context slots with the default values
authorCorentin Noël <corentin.noel@collabora.com>
Mon, 11 Sep 2023 14:53:03 +0000 (16:53 +0200)
committerMarge Bot <emma+marge@anholt.net>
Tue, 12 Sep 2023 15:17:39 +0000 (15:17 +0000)
As 0 is an actual valid value, this can lead to issues when we actually want to use them.

This fixes `spec@arb_cull_distance@basic-cull-3` crashes for instance.

Cc: mesa-stable
Signed-off-by: Corentin Noël <corentin.noel@collabora.com>
Reviewed-by: Mike Blumenkrantz <michael.blumenkrantz@gmail.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/25152>

src/gallium/drivers/llvmpipe/ci/llvmpipe-fails.txt
src/gallium/drivers/llvmpipe/lp_setup.c
src/gallium/drivers/virgl/ci/virpipe-gl-fails.txt

index 4793cd3..8b584ca 100644 (file)
@@ -198,6 +198,3 @@ spec@intel_shader_atomic_float_minmax@execution@ssbo-atomiccompswap-float-nan,Cr
 spec@intel_shader_atomic_float_minmax@execution@ssbo-atomiccompswap-float-negative-zero,Crash
 spec@!opengl 1.1@line-flat-clip-color,Fail
 spec@intel_shader_atomic_float_minmax@execution@shared-atomiccompswap-float,Crash
-spec@arb_cull_distance@basic-cull-3,Crash
-spec@arb_cull_distance@basic-cull-4,Crash
-spec@arb_cull_distance@vs-cull-distance-vertex-outside-cull-volume,Crash
index 9b58e71..26081a4 100644 (file)
@@ -1336,6 +1336,11 @@ lp_setup_create(struct pipe_context *pipe,
 
    lp_setup_init_vbuf(setup);
 
+   setup->psize_slot = -1;
+   setup->viewport_index_slot = -1;
+   setup->layer_slot = -1;
+   setup->face_slot = -1;
+
    /* Used only in update_state():
     */
    setup->pipe = pipe;
index ec4661c..47b73dd 100644 (file)
@@ -678,6 +678,3 @@ spec@arb_viewport_array@display-list,Fail
 spec@arb_gl_spirv@execution@ssbo@unsized-array,Fail
 spec@arb_pipeline_statistics_query@arb_pipeline_statistics_query-frag,Fail
 spec@!opengl 1.1@line-flat-clip-color,Fail
-spec@arb_cull_distance@basic-cull-3,Crash
-spec@arb_cull_distance@basic-cull-4,Crash
-spec@arb_cull_distance@vs-cull-distance-vertex-outside-cull-volume,Crash