tu: Don't invalidate irrelevant state when changing pipeline
authorConnor Abbott <cwabbott0@gmail.com>
Mon, 13 Jul 2020 10:54:36 +0000 (12:54 +0200)
committerConnor Abbott <cwabbott0@gmail.com>
Tue, 14 Jul 2020 08:23:58 +0000 (10:23 +0200)
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

index 84cb9c465d92edaff37e0246eb72b096173dc082..b081d575e2d4f8d2d89a454728d0642e7f5d1b62 100644 (file)
@@ -469,14 +469,8 @@ tu6_emit_cs_config(struct tu_cs *cs, const struct tu_shader *shader,
                    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 @@ tu6_emit_program(struct tu_cs *cs,
          .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