[Mesa-dev] [PATCH] i965: Use brw->urb.min_vs_urb_entries instead of 32 for BLORP.

Pohjolainen, Topi topi.pohjolainen at intel.com
Thu Mar 31 09:20:09 UTC 2016


On Thu, Mar 31, 2016 at 02:18:18AM -0700, Kenneth Graunke wrote:
> Haswell GT2 and GT3 have a minimum of 64 entries.  Hardcoding 32
> is not legal.

Makes also error state decoder happy :)

Reviewed-by: Topi Pohjolainen <topi.pohjolainen at intel.com>

> 
> Cc: mesa-stable at lists.freedesktop.org
> Signed-off-by: Kenneth Graunke <kenneth at whitecape.org>
> ---
>  src/mesa/drivers/dri/i965/gen7_blorp.cpp | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/mesa/drivers/dri/i965/gen7_blorp.cpp b/src/mesa/drivers/dri/i965/gen7_blorp.cpp
> index 89b73ca..a89d5d7 100644
> --- a/src/mesa/drivers/dri/i965/gen7_blorp.cpp
> +++ b/src/mesa/drivers/dri/i965/gen7_blorp.cpp
> @@ -59,7 +59,7 @@ gen7_blorp_emit_urb_config(struct brw_context *brw)
>      * 1.15:0 "VS Number of URB Entries".
>      */
>     gen7_emit_urb_state(brw,
> -                       32 /* num_vs_entries */,
> +                       brw->urb.min_vs_entries /* num_vs_entries */,
>                         2 /* vs_size */,
>                         2 /* vs_start */,
>                         0 /* num_hs_entries */,
> -- 
> 2.7.4
> 
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev


More information about the mesa-dev mailing list