Mesa (master): anv: Drop I915_EXEC_CONSTANTS_REL_GENERAL from execbuf.

Kenneth Graunke kwg at kemper.freedesktop.org
Mon Feb 12 15:00:51 UTC 2018


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

Author: Kenneth Graunke <kenneth at whitecape.org>
Date:   Sun Feb 11 14:52:27 2018 -0800

anv: Drop I915_EXEC_CONSTANTS_REL_GENERAL from execbuf.

The kernel used to have execbuf parameters to program the INSTPM bit
for whether 3DSTATE_CONSTANT_* should be relative to dynamic state
base address or an absolute address.  However, they never worked in
the presence of hardware contexts, so I deleted them a while back.

It doesn't make sense to set this flag, as it doesn't exist anymore.
It also never did anything anyway - the flag is zero, so |'ing it in
did nothing.  The default is relative anyway.

Reviewed-by: Jason Ekstrand <jason at jlekstrand.net>

---

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

diff --git a/src/intel/vulkan/anv_batch_chain.c b/src/intel/vulkan/anv_batch_chain.c
index 85444e64ce..13d55e0efe 100644
--- a/src/intel/vulkan/anv_batch_chain.c
+++ b/src/intel/vulkan/anv_batch_chain.c
@@ -1375,8 +1375,7 @@ setup_execbuf_for_cmd_buffer(struct anv_execbuf *execbuf,
       .num_cliprects = 0,
       .DR1 = 0,
       .DR4 = 0,
-      .flags = I915_EXEC_HANDLE_LUT | I915_EXEC_RENDER |
-               I915_EXEC_CONSTANTS_REL_GENERAL,
+      .flags = I915_EXEC_HANDLE_LUT | I915_EXEC_RENDER,
       .rsvd1 = cmd_buffer->device->context_id,
       .rsvd2 = 0,
    };




More information about the mesa-commit mailing list