[Mesa-dev] [PATCH] [v2] i965/bxt: Production thread counts

Jason Ekstrand jason at jlekstrand.net
Sun Feb 14 02:46:28 UTC 2016


Assuming the URB sizes mean the sensible thing (didn't we just clean that
up?):

Reviewed-by: Jason Ekstrand <jason.ekstrand at intel.com>

I'm a little wary of the URB size since the bspec actually gives a range.
Might be better to drop it to 128 just to be safe.  I guess if we find a
problem, we can always push a patch to stable to drop it to 128.  Do what
you'd like.
--Jason

On Fri, Feb 12, 2016 at 9:53 PM, Ben Widawsky <benjamin.widawsky at intel.com>
wrote:

> v2: Forgot to squash in the comment removal
>
> Signed-off-by: Ben Widawsky <benjamin.widawsky at intel.com>
> ---
>  src/mesa/drivers/dri/i965/brw_device_info.c | 19 +++++++++----------
>  1 file changed, 9 insertions(+), 10 deletions(-)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_device_info.c
> b/src/mesa/drivers/dri/i965/brw_device_info.c
> index 7ab70fe..55c4d36 100644
> --- a/src/mesa/drivers/dri/i965/brw_device_info.c
> +++ b/src/mesa/drivers/dri/i965/brw_device_info.c
> @@ -384,20 +384,19 @@ static const struct brw_device_info
> brw_device_info_bxt = {
>     .gt = 1,
>     .has_llc = false,
>
> -   /* XXX: These are preliminary thread counts and URB sizes. */
>     .num_slices = 1,
> -   .max_vs_threads = 56,
> -   .max_hs_threads = 56,
> -   .max_ds_threads = 56,
> -   .max_gs_threads = 56,
> +   .max_vs_threads = 112,
> +   .max_hs_threads = 112,
> +   .max_ds_threads = 112,
> +   .max_gs_threads = 112,
>     .max_wm_threads = 64 * 3,
> -   .max_cs_threads = 28,
> +   .max_cs_threads = 6 * 6,
>     .urb = {
> -      .size = 64,
> +      .size = 192,
>        .min_vs_entries = 34,
> -      .max_vs_entries = 640,
> -      .max_hs_entries = 80,
> -      .max_ds_entries = 80,
> +      .max_vs_entries = 704,
> +      .max_hs_entries = 256,
> +      .max_ds_entries = 416,
>        .max_gs_entries = 256,
>     }
>  };
> --
> 2.7.1
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20160213/3d7c888c/attachment.html>


More information about the mesa-dev mailing list