Module: Mesa
Branch: main
Commit: 13dcbdcdf341ec269e4d2c53ceb5a1007169fb01
URL:    
http://cgit.freedesktop.org/mesa/mesa/commit/?id=13dcbdcdf341ec269e4d2c53ceb5a1007169fb01

Author: Samuel Pitoiset <[email protected]>
Date:   Thu Nov 30 17:53:19 2023 +0100

radv: make sure to reset the stipple line state when it's disabled

In a scenario like:
CmdBindPipeline(stipple line enabled)
CmdDraw()
CmdBindPipeline(stipple line disabled)
CmdDraw()

The second draw wasn't resetting the stipple line state and this might
have caused issues, though it's uncovered by VK CTS.

Signed-off-by: Samuel Pitoiset <[email protected]>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/26429>

---

 src/amd/vulkan/radv_pipeline_graphics.c | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/src/amd/vulkan/radv_pipeline_graphics.c 
b/src/amd/vulkan/radv_pipeline_graphics.c
index d63792e8ad9..bbc94d9d3d7 100644
--- a/src/amd/vulkan/radv_pipeline_graphics.c
+++ b/src/amd/vulkan/radv_pipeline_graphics.c
@@ -557,9 +557,6 @@ radv_pipeline_needed_dynamic_state(const struct radv_device 
*device, const struc
        (!state->ms || !state->ms->sample_locations_enable))
       states &= ~RADV_DYNAMIC_SAMPLE_LOCATIONS;
 
-   if (!(pipeline->dynamic_states & RADV_DYNAMIC_LINE_STIPPLE_ENABLE) && 
!state->rs->line.stipple.enable)
-      states &= ~RADV_DYNAMIC_LINE_STIPPLE;
-
    if (!has_color_att || !radv_pipeline_is_blend_enabled(pipeline, state->cb))
       states &= ~RADV_DYNAMIC_BLEND_CONSTANTS;
 

Reply via email to