[Mesa-dev] [PATCH] i965: fix MAX_COMPUTE_SHARED_SIZE constant value

Kenneth Graunke kenneth at whitecape.org
Sat Feb 13 21:57:35 UTC 2016


On Saturday, February 13, 2016 10:49:37 PM PST Samuel Pitoiset wrote:
> MAX_COMPUTE_SHARED_SIZE should be set to 32768. This fixes a regression
> introduced in be27f77 (mesa: do not use a constant for
> MAX_COMPUTE_SHARED_SIZE).
> 
> Signed-off-by: Samuel Pitoiset <samuel.pitoiset at gmail.com>
> Cc: Mark Janes <mark.a.janes at intel.com>
> ---
> 
> This patch is untested because I don't have any Intel GPUs.
> 
>  src/mesa/drivers/dri/i965/brw_context.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/src/mesa/drivers/dri/i965/brw_context.c b/src/mesa/drivers/dri/
i965/brw_context.c
> index a5f7a2e..972b62e 100644
> --- a/src/mesa/drivers/dri/i965/brw_context.c
> +++ b/src/mesa/drivers/dri/i965/brw_context.c
> @@ -710,6 +710,7 @@ brw_initialize_cs_context_constants(struct brw_context 
*brw, unsigned max_thread
>     ctx->Const.MaxComputeWorkGroupSize[1] = max_invocations;
>     ctx->Const.MaxComputeWorkGroupSize[2] = max_invocations;
>     ctx->Const.MaxComputeWorkGroupInvocations = max_invocations;
> +   ctx->Const.MaxComputeSharedMemorySize = 32768;
>  }
>  
>  /**
> 

Hah!  Multiple patches at the same time :)  Let's go with yours.

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: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20160213/b921fadd/attachment.sig>


More information about the mesa-dev mailing list