<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Aug 28, 2016 at 10:29 PM, Pohjolainen, Topi <span dir="ltr"><<a href="mailto:topi.pohjolainen@gmail.com" target="_blank">topi.pohjolainen@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Fri, Aug 19, 2016 at 09:55:43AM -0700, Jason Ekstrand wrote:<br>
</span><span class="">> ---<br>
>  src/mesa/drivers/dri/i965/<wbr>genX_blorp_exec.c | 6 ++----<br>
>  1 file changed, 2 insertions(+), 4 deletions(-)<br>
><br>
> diff --git a/src/mesa/drivers/dri/i965/<wbr>genX_blorp_exec.c b/src/mesa/drivers/dri/i965/<wbr>genX_blorp_exec.c<br>
> index a6ac7b0..402ae3f 100644<br>
> --- a/src/mesa/drivers/dri/i965/<wbr>genX_blorp_exec.c<br>
> +++ b/src/mesa/drivers/dri/i965/<wbr>genX_blorp_exec.c<br>
> @@ -200,9 +200,9 @@ static void<br>
>  emit_urb_config(struct brw_context *brw,<br>
>                  const struct brw_blorp_params *params)<br>
>  {<br>
> -#if GEN_GEN >= 7<br>
>     const unsigned vs_entry_size = gen7_blorp_get_vs_entry_size(<wbr>params);<br>
<br>
</span>About using logic marked as gen7 also for gen6 further down: The name of<br>
function is misleading, there is nothing gen7 specific in the way the entry<br>
size is calculated (earlier gens like to have the size as 64 byte chunks as<br>
well).<br>
<br>
It looks that you would need to do unnecessary rebasing just to change the<br>
name. Perhaps do it as a follow up?<span class=""><br></span></blockquote><div><br></div><div>Patch incoming<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
><br>
> +#if GEN_GEN >= 7<br>
>     if (!(brw->ctx.NewDriverState & (BRW_NEW_CONTEXT | BRW_NEW_URB_SIZE)) &&<br>
>         brw->urb.vsize >= vs_entry_size)<br>
>        return;<br>
> @@ -211,9 +211,7 @@ emit_urb_config(struct brw_context *brw,<br>
><br>
>     gen7_upload_urb(brw, vs_entry_size, false, false);<br>
>  #else<br>
> -   blorp_emit(brw, GENX(3DSTATE_URB), urb) {<br>
> -      urb.VSNumberofURBEntries = brw->urb.max_vs_entries;<br>
> -   }<br>
<br>
</span>I wonder how correct this was before. Actual entry size<br>
(VSURBEntryAllocationSize) was left to zero. Now we start actually setting it.<br></blockquote><div><br></div><div>Yeah, I was a bit disturbed by that as well. :/<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Reviewed-by: Topi Pohjolainen <<a href="mailto:topi.pohjolainen@intel.com">topi.pohjolainen@intel.com</a>><span class="im HOEnZb"><br></span></blockquote><div><br></div><div>Thanks!<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="im HOEnZb">
> +   gen6_upload_urb(brw, vs_entry_size, false, 0);<br>
>  #endif<br>
>  }<br>
><br>
> --<br>
> 2.5.0.400.gff86faf<br>
><br>
</span><div class="HOEnZb"><div class="h5">> ______________________________<wbr>_________________<br>
> mesa-dev mailing list<br>
> <a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a><br>
> <a href="https://lists.freedesktop.org/mailman/listinfo/mesa-dev" rel="noreferrer" target="_blank">https://lists.freedesktop.org/<wbr>mailman/listinfo/mesa-dev</a><br>
</div></div></blockquote></div><br></div></div>