[Mesa-dev] [PATCH] i965/skl: Fix the maximum thread count format for the PS
Kenneth Graunke
kenneth at whitecape.org
Mon Mar 16 11:40:39 PDT 2015
On Friday, February 27, 2015 07:20:37 PM Neil Roberts wrote:
> According to the bspec for some reason the format of the maximum
> number of threads field has changed from U8-2 to U8-1 for the PS.
>
> I've run this through Piglit and it doesn't cause any regressions.
> ---
> src/mesa/drivers/dri/i965/gen8_ps_state.c | 7 ++++++-
> 1 file changed, 6 insertions(+), 1 deletion(-)
>
> diff --git a/src/mesa/drivers/dri/i965/gen8_ps_state.c b/src/mesa/drivers/dri/i965/gen8_ps_state.c
> index d4a58e4..5f39e12 100644
> --- a/src/mesa/drivers/dri/i965/gen8_ps_state.c
> +++ b/src/mesa/drivers/dri/i965/gen8_ps_state.c
> @@ -146,8 +146,13 @@ upload_ps_state(struct brw_context *brw)
>
> /* 3DSTATE_PS expects the number of threads per PSD, which is always 64;
> * it implicitly scales for different GT levels (which have some # of PSDs).
> + *
> + * In Gen8 the format is U8-2 whereas in Gen9 it is U8-1.
> */
> - dw6 |= (64 - 2) << HSW_PS_MAX_THREADS_SHIFT;
> + if (brw->gen >= 9)
> + dw6 |= (64 - 1) << HSW_PS_MAX_THREADS_SHIFT;
> + else
> + dw6 |= (64 - 2) << HSW_PS_MAX_THREADS_SHIFT;
>
> if (prog_data->base.nr_params > 0)
> dw6 |= GEN7_PS_PUSH_CONSTANT_ENABLE;
>
Right...the -2 thing was a Broadwell oddity.
Reviewed-by: Kenneth Graunke <kenneth at whitecape.org>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: This is a digitally signed message part.
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20150316/86e8f643/attachment.sig>
More information about the mesa-dev
mailing list