i965: Use brw->urb.min_vs_urb_entries instead of 32 for BLORP.
authorKenneth Graunke <kenneth@whitecape.org>
Thu, 31 Mar 2016 07:53:21 +0000 (00:53 -0700)
committerKenneth Graunke <kenneth@whitecape.org>
Thu, 31 Mar 2016 23:45:07 +0000 (16:45 -0700)
Haswell GT2 and GT3 have a minimum of 64 entries.  Hardcoding 32
is not legal.

v2: Delete stale comment (caught by Alejandro).

Cc: mesa-stable@lists.freedesktop.org
Signed-off-by: Kenneth Graunke <kenneth@whitecape.org>
Reviewed-by: Topi Pohjolainen <topi.pohjolainen@intel.com>
src/mesa/drivers/dri/i965/gen7_blorp.cpp

index 89b73ca7519d7661b551165929b16f943ca2c742..eae1e30e150f486133654376a141c00d116edc6d 100644 (file)
@@ -55,11 +55,8 @@ gen7_blorp_emit_urb_config(struct brw_context *brw)
                                  0 /* gs_size */,
                                  urb_size / 2 /* fs_size */);
 
-   /* The minimum valid number of VS entries is 32. See 3DSTATE_URB_VS, Dword
-    * 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 */,