Mesa (master): anv: Initialize depth_bounds_test_enable when not explicitly set

GitLab Mirror gitlab-mirror at kemper.freedesktop.org
Wed Nov 13 18:14:06 UTC 2019


Module: Mesa
Branch: master
Commit: 0aaf47f7cd50b9c07c1297fe8daff1b2693a6729
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=0aaf47f7cd50b9c07c1297fe8daff1b2693a6729

Author: Caio Marcelo de Oliveira Filho <caio.oliveira at intel.com>
Date:   Tue Nov 12 10:42:09 2019 -0800

anv: Initialize depth_bounds_test_enable when not explicitly set

This was causing uninitialized value to end up propagated to the
3DSTATE_DEPTH_BOUNDS packet, leading to asserts on packet
building due to the value being greater than 1.

Fixes: 939ddccb7a5 ("anv: Add support for depth bounds testing.")
Reviewed-by: Plamena Manolova <plamena.manolova at intel.com>

---

 src/intel/vulkan/genX_pipeline.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/src/intel/vulkan/genX_pipeline.c b/src/intel/vulkan/genX_pipeline.c
index 078b3f2b462..5594726e4a2 100644
--- a/src/intel/vulkan/genX_pipeline.c
+++ b/src/intel/vulkan/genX_pipeline.c
@@ -999,6 +999,7 @@ emit_ds_state(struct anv_pipeline *pipeline,
       pipeline->stencil_test_enable = false;
       pipeline->writes_depth = false;
       pipeline->depth_test_enable = false;
+      pipeline->depth_bounds_test_enable = false;
       memset(depth_stencil_dw, 0, sizeof(depth_stencil_dw));
       return;
    }
@@ -1017,8 +1018,6 @@ emit_ds_state(struct anv_pipeline *pipeline,
    pipeline->depth_test_enable = info.depthTestEnable;
    pipeline->depth_bounds_test_enable = info.depthBoundsTestEnable;
 
-   /* VkBool32 depthBoundsTestEnable; // optional (depth_bounds_test) */
-
 #if GEN_GEN <= 7
    struct GENX(DEPTH_STENCIL_STATE) depth_stencil = {
 #else




More information about the mesa-commit mailing list