Mesa (master): i965: Drop BRW_NEW_PUSH_CONSTANT_ALLOCATION from CS packets.

Kenneth Graunke kwg at kemper.freedesktop.org
Sat May 20 07:23:46 UTC 2017


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

Author: Kenneth Graunke <kenneth at whitecape.org>
Date:   Tue Nov 29 01:30:18 2016 -0800

i965: Drop BRW_NEW_PUSH_CONSTANT_ALLOCATION from CS packets.

I don't think CS push constant uploading uses the section of L3
controlled by 3DSTATE_PUSH_CONSTANT_ALLOC_XS.  So I don't think
it needs to be re-emitted when that space is reallocated.

The programming note in gen7_allocate_push_constants doesn't
indicate this is necessary, at least.

Reviewed-by: Chris Forbes <chrisforbes at google.com>

---

 src/mesa/drivers/dri/i965/gen7_cs_state.c     | 3 +--
 src/mesa/drivers/dri/i965/genX_state_upload.c | 1 -
 2 files changed, 1 insertion(+), 3 deletions(-)

diff --git a/src/mesa/drivers/dri/i965/gen7_cs_state.c b/src/mesa/drivers/dri/i965/gen7_cs_state.c
index cbb6bb1ef2..26e4264018 100644
--- a/src/mesa/drivers/dri/i965/gen7_cs_state.c
+++ b/src/mesa/drivers/dri/i965/gen7_cs_state.c
@@ -133,8 +133,7 @@ const struct brw_tracked_state gen7_cs_push_constants = {
       .brw = BRW_NEW_BATCH |
              BRW_NEW_BLORP |
              BRW_NEW_COMPUTE_PROGRAM |
-             BRW_NEW_CS_PROG_DATA |
-             BRW_NEW_PUSH_CONSTANT_ALLOCATION,
+             BRW_NEW_CS_PROG_DATA,
    },
    .emit = gen7_upload_cs_push_constants,
 };
diff --git a/src/mesa/drivers/dri/i965/genX_state_upload.c b/src/mesa/drivers/dri/i965/genX_state_upload.c
index 3921a6f2d4..64ba357738 100644
--- a/src/mesa/drivers/dri/i965/genX_state_upload.c
+++ b/src/mesa/drivers/dri/i965/genX_state_upload.c
@@ -3735,7 +3735,6 @@ static const struct brw_tracked_state genX(cs_state) = {
       .brw = BRW_NEW_BATCH |
              BRW_NEW_BLORP |
              BRW_NEW_CS_PROG_DATA |
-             BRW_NEW_PUSH_CONSTANT_ALLOCATION |
              BRW_NEW_SAMPLER_STATE_TABLE |
              BRW_NEW_SURFACES,
    },




More information about the mesa-commit mailing list