[Mesa-dev] [PATCH 06/31] i965/blorp: Use gen6_upload_urb

[Pohjolainen, Topi]

On Fri, Aug 19, 2016 at 09:55:43AM -0700, Jason Ekstrand wrote:
> ---
>  src/mesa/drivers/dri/i965/genX_blorp_exec.c | 6 ++----
>  1 file changed, 2 insertions(+), 4 deletions(-)
> 
> diff --git a/src/mesa/drivers/dri/i965/genX_blorp_exec.c b/src/mesa/drivers/dri/i965/genX_blorp_exec.c
> index a6ac7b0..402ae3f 100644
> --- a/src/mesa/drivers/dri/i965/genX_blorp_exec.c
> +++ b/src/mesa/drivers/dri/i965/genX_blorp_exec.c
> @@ -200,9 +200,9 @@ static void
>  emit_urb_config(struct brw_context *brw,
>                  const struct brw_blorp_params *params)
>  {
> -#if GEN_GEN >= 7
>     const unsigned vs_entry_size = gen7_blorp_get_vs_entry_size(params);

Initially it looked a little odd to use logic marked for gen7 also for gen6.
In practise this looks to work though. I'll read a little further the series
to see how this turns out in the end.

>  
> +#if GEN_GEN >= 7
>     if (!(brw->ctx.NewDriverState & (BRW_NEW_CONTEXT | BRW_NEW_URB_SIZE)) &&
>         brw->urb.vsize >= vs_entry_size)
>        return;
> @@ -211,9 +211,7 @@ emit_urb_config(struct brw_context *brw,
>  
>     gen7_upload_urb(brw, vs_entry_size, false, false);
>  #else
> -   blorp_emit(brw, GENX(3DSTATE_URB), urb) {
> -      urb.VSNumberofURBEntries = brw->urb.max_vs_entries;
> -   }
> +   gen6_upload_urb(brw, vs_entry_size, false, 0);
>  #endif
>  }
>  
> -- 
> 2.5.0.400.gff86faf
> 
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev