commit bf1376aba0e9967317bd5284b6bf324b5cea1b59
author Connor Abbott <cwabbott0@gmail.com> Mon Jul 13 12:54:36 2020 +0200
committer Connor Abbott <cwabbott0@gmail.com> Tue Jul 14 10:23:58 2020 +0200
tree 51ed3496aeaf83797a41dda04609578e2469217c
parent a16136796ff6e7dc31f3daa7c39762a787fbf169

tu: Don't invalidate irrelevant state when changing pipeline

At least in the future this could let us avoid re-emitting gfx/cs
constants when the other changes. This also matches what the blob does.

Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/5877>

src/freedreno/vulkan/tu_pipeline.c

diff --git a/src/freedreno/vulkan/tu_pipeline.c b/src/freedreno/vulkan/tu_pipeline.c
index 84cb9c4..b081d57 100644
--- a/src/freedreno/vulkan/tu_pipeline.c
+++ b/src/freedreno/vulkan/tu_pipeline.c
@@ -469,14 +469,8 @@
                    uint32_t binary_iova)
 {
    tu_cs_emit_regs(cs, A6XX_HLSQ_INVALIDATE_CMD(
-         .vs_state = true,
-         .hs_state = true,
-         .ds_state = true,
-         .gs_state = true,
-         .fs_state = true,
          .cs_state = true,
-         .cs_ibo = true,
-         .gfx_ibo = true));
+         .cs_ibo = true));
 
    tu6_emit_xs_config(cs, MESA_SHADER_COMPUTE, v, binary_iova);
 
@@ -1368,8 +1362,6 @@
          .ds_state = true,
          .gs_state = true,
          .fs_state = true,
-         .cs_state = true,
-         .cs_ibo = true,
          .gfx_ibo = true));
 
   /* Don't use the binning pass variant when GS is present because we don't