On Thu, Apr 19, 2018 at 7:44 AM, Topi Pohjolainen <
[email protected]> wrote:

> 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
>
>
​Verified in BSpec.
​Reviewed-by: Anuj Phogat <[email protected]>
_______________________________________________
mesa-dev mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to