This didn't actually help the failing tests I'm looking at
but hopefully has teeth elsewhere.

CC: Jason Ekstrand <[email protected]>
CC: Jordan Justen <[email protected]>
CC: Anuj Phogat <[email protected]>
Signed-off-by: Topi Pohjolainen <[email protected]>
---
 src/mesa/drivers/dri/i965/gen7_urb.c | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/src/mesa/drivers/dri/i965/gen7_urb.c 
b/src/mesa/drivers/dri/i965/gen7_urb.c
index 2e5f8e6..9e12657 100644
--- a/src/mesa/drivers/dri/i965/gen7_urb.c
+++ b/src/mesa/drivers/dri/i965/gen7_urb.c
@@ -145,8 +145,15 @@ gen7_emit_push_constant_state(struct brw_context *brw, 
unsigned vs_size,
     *     in the ring after this instruction.
     *
     * No such restriction exists for Haswell or Baytrail.
+    *
+    * From the CNL Bspec, Windower -
+    * 3DSTATE_PUSH_CONSTANT_ALLOC_PS/VS/GS/DS/HS:
+    * 
+    *     This command must be followed by a PIPE_CONTROL with CS Stall bit
+    *     set.
     */
-   if (devinfo->gen < 8 && !devinfo->is_haswell && !devinfo->is_baytrail)
+   if ((devinfo->gen < 8 && !devinfo->is_haswell && !devinfo->is_baytrail) ||
+       devinfo->gen >= 10)
       gen7_emit_cs_stall_flush(brw);
 }
 
-- 
2.7.4

_______________________________________________
mesa-dev mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to