[Mesa-dev] [PATCH] st/mesa: Use correct size for compute CAPs.

Marek Olšák maraeo at gmail.com
Wed Apr 20 14:22:43 UTC 2016


Reviewed-by: Marek Olšák <marek.olsak at amd.com>

Marek

On Wed, Apr 20, 2016 at 3:35 PM, Bas Nieuwenhuizen
<bas at basnieuwenhuizen.nl> wrote:
> Some CAPs are stored as 64-bit value while Mesa stores
> the related constant as 32-bit value.
>
> Signed-off-by: Bas Nieuwenhuizen <bas at basnieuwenhuizen.nl>
> ---
>  src/mesa/state_tracker/st_extensions.c | 8 ++++++--
>  1 file changed, 6 insertions(+), 2 deletions(-)
>
> diff --git a/src/mesa/state_tracker/st_extensions.c b/src/mesa/state_tracker/st_extensions.c
> index 939f15d..3f769b6 100644
> --- a/src/mesa/state_tracker/st_extensions.c
> +++ b/src/mesa/state_tracker/st_extensions.c
> @@ -1152,6 +1152,7 @@ void st_init_extensions(struct pipe_screen *screen,
>                                    PIPE_SHADER_CAP_SUPPORTED_IRS);
>        if (compute_supported_irs & (1 << PIPE_SHADER_IR_TGSI)) {
>           uint64_t grid_size[3], block_size[3];
> +         uint64_t max_local_size, max_threads_per_block;
>
>           screen->get_compute_param(screen, PIPE_SHADER_IR_TGSI,
>                                     PIPE_COMPUTE_CAP_MAX_GRID_SIZE, grid_size);
> @@ -1159,10 +1160,13 @@ void st_init_extensions(struct pipe_screen *screen,
>                                     PIPE_COMPUTE_CAP_MAX_BLOCK_SIZE, block_size);
>           screen->get_compute_param(screen, PIPE_SHADER_IR_TGSI,
>                                     PIPE_COMPUTE_CAP_MAX_THREADS_PER_BLOCK,
> -                                   &consts->MaxComputeWorkGroupInvocations);
> +                                   &max_threads_per_block);
>           screen->get_compute_param(screen, PIPE_SHADER_IR_TGSI,
>                                     PIPE_COMPUTE_CAP_MAX_LOCAL_SIZE,
> -                                   &consts->MaxComputeSharedMemorySize);
> +                                   &max_local_size);
> +
> +         consts->MaxComputeWorkGroupInvocations = max_threads_per_block;
> +         consts->MaxComputeSharedMemorySize = max_local_size;
>
>           for (i = 0; i < 3; i++) {
>              consts->MaxComputeWorkGroupCount[i] = grid_size[i];
> --
> 2.8.0
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev


More information about the mesa-dev mailing list