[Mesa-dev] [PATCH 01/13] vbo: Declare the index range invalid for DrawIndirect

Ian Romanick idr at freedesktop.org
Thu May 19 17:10:28 UTC 2016


On 05/19/2016 12:20 AM, Jason Ekstrand wrote:
> Right now, we're just setting the range to [0, MAX_UINT32] which, while
> correct isn't helpful.  With DrawIndirect, you can't really know what the
> actual range is so we may as well flag it as being an invalid range.  This
> is what we do for draws with index buffer which is similar (the indices
> aren't statically known) if a bit simpler.
> 
> Cc: "10.2" <mesa-stable at lists.freedesktop.org>

I don't think this is the stable tagging that you intended. :)  These
are definitely candidates for as far back as Emil is cherry-picking
patches.  Maybe "11.0 11.1 11.2" ?

In any case, as discussed on IRC, this patch is

Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>

> ---
>  src/mesa/vbo/vbo_context.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/mesa/vbo/vbo_context.c b/src/mesa/vbo/vbo_context.c
> index 9f807a1..ae5d265 100644
> --- a/src/mesa/vbo/vbo_context.c
> +++ b/src/mesa/vbo/vbo_context.c
> @@ -170,7 +170,7 @@ vbo_draw_indirect_prims(struct gl_context *ctx,
>     }
>  
>     vbo->draw_prims(ctx, prim, draw_count,
> -                   ib, GL_TRUE, 0, ~0,
> +                   ib, false, ~0, ~0,

I'd be in favor of a follow-up patch that converts this GLboolean to
bool everywhere. :)

>                     NULL, 0,
>                     ctx->DrawIndirectBuffer);
>  



More information about the mesa-dev mailing list