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@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
