[Mesa-dev] [PATCH v2 1/2] mesa: Update DispatchComputeIndirect errors for indirect parameter

[Iago Toral]

the changes look reasonable. For both patches:

Reviewed-by: Iago Toral Quiroga <itoral at igalia.com>

On Mon, 2015-11-02 at 00:35 -0800, Jordan Justen wrote:
> There is some discrepancy between the return values for some error
> cases for the DispatchComputeIndirect call in the ARB_compute_shader
> specification. Regarding the indirect parameter, in one place the
> extension spec lists that the error returned for invalid values should
> be INVALID_OPERATION, while later it specifies INVALID_VALUE.
> 
> The OpenGL 4.3 and OpenGLES 3.1 specifications appear to be consistent
> in requiring the INVALID_VALUE error return in this case.
> 
> Here we update the code to match the main specifications, and update
> the citations use the main specification rather than the extension
> specification.
> 
> v2:
>  * Updates based on review from Iago
> 
> Signed-off-by: Jordan Justen <jordan.l.justen at intel.com>
> Cc: Iago Toral Quiroga <itoral at igalia.com>
> Cc: Marta Lofstedt <marta.lofstedt at intel.com>
> ---
>  src/mesa/main/api_validate.c | 11 +++++------
>  1 file changed, 5 insertions(+), 6 deletions(-)
> 
> diff --git a/src/mesa/main/api_validate.c b/src/mesa/main/api_validate.c
> index 1f729e8..a3ee8c0 100644
> --- a/src/mesa/main/api_validate.c
> +++ b/src/mesa/main/api_validate.c
> @@ -960,20 +960,19 @@ valid_dispatch_indirect(struct gl_context *ctx,
>     if (!check_valid_to_compute(ctx, name))
>        return GL_FALSE;
>  
> -   /* From the ARB_compute_shader specification:
> +   /* From the OpenGL 4.3 Core Specification, Chapter 19, Compute Shaders:
>      *
> -    * "An INVALID_OPERATION error is generated [...] if <indirect> is less
> -    *  than zero or not a multiple of the size, in basic machine units, of
> -    *  uint."
> +    * "An INVALID_VALUE error is generated if indirect is negative or is not a
> +    *  multiple of four."
>      */
>     if ((GLintptr)indirect & (sizeof(GLuint) - 1)) {
> -      _mesa_error(ctx, GL_INVALID_OPERATION,
> +      _mesa_error(ctx, GL_INVALID_VALUE,
>                    "%s(indirect is not aligned)", name);
>        return GL_FALSE;
>     }
>  
>     if ((GLintptr)indirect < 0) {
> -      _mesa_error(ctx, GL_INVALID_OPERATION,
> +      _mesa_error(ctx, GL_INVALID_VALUE,
>                    "%s(indirect is less than zero)", name);
>        return GL_FALSE;
>     }