[Mesa-dev] [PATCH 15/23] meta: Use internal functions for buffer object and VAO access

Matt Turner mattst88 at gmail.com
Sun Nov 29 11:20:23 PST 2015


On Mon, Nov 9, 2015 at 4:56 PM, Ian Romanick <idr at freedesktop.org> wrote:
> From: Ian Romanick <ian.d.romanick at intel.com>
>
> Signed-off-by: Ian Romanick <ian.d.romanick at intel.com>
> ---
>  src/mesa/drivers/common/meta.c | 76 ++++++++++++++++++++++++------------------
>  1 file changed, 43 insertions(+), 33 deletions(-)
>
> diff --git a/src/mesa/drivers/common/meta.c b/src/mesa/drivers/common/meta.c
> index 8cf0bf9..3507927 100644
> --- a/src/mesa/drivers/common/meta.c
> +++ b/src/mesa/drivers/common/meta.c
> @@ -342,12 +342,16 @@ _mesa_meta_setup_vertex_objects(struct gl_context *ctx,
>     GLuint VBO;
>
>     if (*VAO == 0) {
> +      struct gl_vertex_array_object *array_obj;
>        assert(*buf_obj == NULL);
>
>        /* create vertex array object */
>        _mesa_GenVertexArrays(1, VAO);
>        _mesa_BindVertexArray(*VAO);
>
> +      array_obj = _mesa_lookup_vao(ctx, *VAO);
> +      assert(array_obj != NULL);
> +
>        /* create vertex array buffer */
>        _mesa_CreateBuffers(1, &VBO);
>        *buf_obj = _mesa_lookup_bufferobj(ctx, VBO);
> @@ -360,8 +364,8 @@ _mesa_meta_setup_vertex_objects(struct gl_context *ctx,
>        assert(*buf_obj != NULL && (*buf_obj)->Name == VBO);
>        assert(*buf_obj != ctx->Array.ArrayBufferObj);
>
> -      _mesa_NamedBufferData(VBO, 4 * sizeof(struct vertex), NULL,
> -                            GL_DYNAMIC_DRAW);
> +      _mesa_buffer_data(ctx, *buf_obj, GL_NONE, 4 * sizeof(struct vertex), NULL,
> +                        GL_DYNAMIC_DRAW, __func__);
>
>        assert((*buf_obj)->Size == 4 * sizeof(struct vertex));
>
> @@ -369,45 +373,51 @@ _mesa_meta_setup_vertex_objects(struct gl_context *ctx,
>        if (use_generic_attributes) {
>           assert(color_size == 0);
>
> -         _mesa_VertexArrayAttribFormat(*VAO, 0, vertex_size, GL_FLOAT,
> -                                       GL_FALSE, OFFSET(x));
> -         _mesa_VertexArrayVertexBuffer(*VAO, 0, VBO, 0,
> -                                       sizeof(struct vertex));
> -         _mesa_EnableVertexAttribArray(0);
> -
> +         _mesa_update_array_format(ctx, array_obj, VERT_ATTRIB_GENERIC(0),
> +                                   vertex_size, GL_FLOAT, GL_RGBA, GL_FALSE,
> +                                   GL_FALSE, GL_FALSE,
> +                                   offsetof(struct vertex, x), true);
> +         _mesa_bind_vertex_buffer(ctx, array_obj, VERT_ATTRIB_GENERIC(0),
> +                                  *buf_obj, 0, sizeof(struct vertex));
> +         _mesa_enable_vertex_array_attrib(ctx, array_obj,
> +                                          VERT_ATTRIB_GENERIC(0));
>           if (texcoord_size > 0) {
> -            _mesa_VertexArrayAttribFormat(*VAO, 1, texcoord_size, GL_FLOAT,
> -                                          GL_FALSE, OFFSET(tex));
> -            _mesa_VertexArrayVertexBuffer(*VAO, 1, VBO, 0,
> -                                          sizeof(struct vertex));
> -            _mesa_EnableVertexAttribArray(1);
> +            _mesa_update_array_format(ctx, array_obj, VERT_ATTRIB_GENERIC(1),
> +                                      texcoord_size, GL_FLOAT, GL_RGBA,
> +                                      GL_FALSE, GL_FALSE, GL_FALSE,
> +                                      offsetof(struct vertex, tex), false);
> +            _mesa_bind_vertex_buffer(ctx, array_obj, VERT_ATTRIB_GENERIC(1),
> +                                     *buf_obj, 0, sizeof(struct vertex));
> +            _mesa_enable_vertex_array_attrib(ctx, array_obj,
> +                                             VERT_ATTRIB_GENERIC(1));
>           }
>        } else {
> -         _mesa_BindBuffer(GL_ARRAY_BUFFER, VBO);
> -         _mesa_VertexPointer(vertex_size, GL_FLOAT, sizeof(struct vertex),
> -                             OFFSET(x));
> -         _mesa_EnableClientState(GL_VERTEX_ARRAY);
> +         _mesa_update_array_format(ctx, array_obj, VERT_ATTRIB_POS,
> +                                   vertex_size, GL_FLOAT, GL_RGBA, GL_FALSE,
> +                                   GL_FALSE, GL_FALSE,
> +                                   offsetof(struct vertex, x), true);
> +         _mesa_bind_vertex_buffer(ctx, array_obj, VERT_ATTRIB_POS,
> +                                  *buf_obj, 0, sizeof(struct vertex));
> +         _mesa_enable_vertex_array_attrib(ctx, array_obj, VERT_ATTRIB_POS);
>
>           if (texcoord_size > 0) {
> -            _mesa_TexCoordPointer(texcoord_size, GL_FLOAT,
> -                                  sizeof(struct vertex), OFFSET(tex));
> -            _mesa_EnableClientState(GL_TEXTURE_COORD_ARRAY);
> +            _mesa_update_array_format(ctx, array_obj, VERT_ATTRIB_TEX(0),
> +                                      vertex_size, GL_FLOAT, GL_RGBA, GL_FALSE,

In the similar block above, you pass texcoord_size instead of
vertex_size. Is this difference here intentional?

Coverity suggests there might be a copy-n-paste mistake, but it looks
to me like this^ might be the problem rather than what Coverity says.

*** CID 1341051:  Incorrect expression  (COPY_PASTE_ERROR)
/src/mesa/drivers/common/meta.c: 395 in _mesa_meta_setup_vertex_objects()
389                                        GL_FALSE, GL_FALSE,
390                                        offsetof(struct vertex, x), true);
391              _mesa_bind_vertex_buffer(ctx, array_obj, VERT_ATTRIB_POS,
392                                       *buf_obj, 0, sizeof(struct vertex));
393              _mesa_enable_vertex_array_attrib(ctx, array_obj,
VERT_ATTRIB_POS);
394
>>>     CID 1341051:  Incorrect expression  (COPY_PASTE_ERROR)
>>>     "texcoord_size" in "texcoord_size > 0U" looks like a copy-paste error.
395              if (texcoord_size > 0) {
396                 _mesa_update_array_format(ctx, array_obj,
VERT_ATTRIB_TEX(0),
397                                           vertex_size, GL_FLOAT,
GL_RGBA, GL_FALSE,
398                                           GL_FALSE, GL_FALSE,
399                                           offsetof(struct vertex,
tex), false);
400                 _mesa_bind_vertex_buffer(ctx, array_obj, VERT_ATTRIB_TEX(0),


> +                                      GL_FALSE, GL_FALSE,
> +                                      offsetof(struct vertex, tex), false);
> +            _mesa_bind_vertex_buffer(ctx, array_obj, VERT_ATTRIB_TEX(0),
> +                                     *buf_obj, 0, sizeof(struct vertex));
> +            _mesa_enable_vertex_array_attrib(ctx, array_obj, VERT_ATTRIB_TEX(0));
>           }
>
>           if (color_size > 0) {
> -            _mesa_ColorPointer(color_size, GL_FLOAT,
> -                               sizeof(struct vertex), OFFSET(r));
> -            _mesa_EnableClientState(GL_COLOR_ARRAY);
> -         }
> -
> -         /* Restore the old VBO.  This is done because we don't want the new VBO
> -          * to be bound on exit.  It would be nicer to use DSA type functions,
> -          * but there are no DSA functions to bind a VBO to a VAO for
> -          * fixed-function vertex attributes.
> -          */
> -         {
> -            struct save_state *save = &ctx->Meta->Save[ctx->Meta->SaveStackDepth - 1];
> -            _mesa_BindBuffer(GL_ARRAY_BUFFER, save->ArrayBufferObj->Name);
> +            _mesa_update_array_format(ctx, array_obj, VERT_ATTRIB_COLOR0,
> +                                      vertex_size, GL_FLOAT, GL_RGBA, GL_FALSE,
> +                                      GL_FALSE, GL_FALSE,
> +                                      offsetof(struct vertex, r), false);
> +            _mesa_bind_vertex_buffer(ctx, array_obj, VERT_ATTRIB_COLOR0,
> +                                     *buf_obj, 0, sizeof(struct vertex));
> +            _mesa_enable_vertex_array_attrib(ctx, array_obj, VERT_ATTRIB_COLOR0);
>           }
>        }
>     } else {
> --
> 2.1.0
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev


More information about the mesa-dev mailing list