For patches 8, 9:
Reviewed-by: Marek Olšák <marek.ol...@amd.com>
Patch 10 won't work for us, because radeonsi (and presumably softpipe
as well) don't support this feature at the moment. Also, I would
prefer a PIPE_CAP for MaxComputeVariableGroupInvocations and the
extension can be exposed based on that CAP.
Marek
On Thu, Sep 8, 2016 at 10:31 PM, Samuel Pitoiset
<samuel.pitoi...@gmail.com> wrote:
> This extension is only exposed if the underlying driver supports
> ARB_compute_shader.
>
> Signed-off-by: Samuel Pitoiset <samuel.pitoi...@gmail.com>
> ---
> src/mesa/state_tracker/st_extensions.c | 13 +++++++++++++
> 1 file changed, 13 insertions(+)
>
> diff --git a/src/mesa/state_tracker/st_extensions.c
> b/src/mesa/state_tracker/st_extensions.c
> index 807fbfb..dc2e60a 100644
> --- a/src/mesa/state_tracker/st_extensions.c
> +++ b/src/mesa/state_tracker/st_extensions.c
> @@ -1196,6 +1196,19 @@ void st_init_extensions(struct pipe_screen *screen,
> extensions->ARB_compute_shader =
>
> extensions->ARB_shader_image_load_store &&
> extensions->ARB_shader_atomic_counters;
> +
> + if (extensions->ARB_compute_shader) {
> + /* Because the minimum values required by
> + * ARB_compute_variable_group_size are less (or equal) than the
> + * ones defined by ARB_compute_shader we can re-use them. */
> + for (i = 0; i < 3; i++) {
> + consts->MaxComputeVariableGroupSize[i] =
> + consts->MaxComputeWorkGroupSize[i];
> + }
> + consts->MaxComputeVariableGroupInvocations =
> + consts->MaxComputeWorkGroupInvocations;
> + extensions->ARB_compute_variable_group_size = true;
> + }
> }
> }
>
> --
> 2.9.3
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
