[Mesa-dev] [PATCH 6/6] st/mesa: Reset the constant buffers before destroying the pipe context.

Tilman Sauerbeck tilman at code-monkey.de
Tue Nov 2 11:17:47 PDT 2010


Tilman Sauerbeck [2010-10-31 17:38]:
> Signed-off-by: Tilman Sauerbeck <tilman at code-monkey.de>
> ---
>  src/mesa/state_tracker/st_context.c |    4 ++++
>  1 files changed, 4 insertions(+), 0 deletions(-)
> 
> diff --git a/src/mesa/state_tracker/st_context.c b/src/mesa/state_tracker/st_context.c
> index b5ea6d0..c0bfbea 100644
> --- a/src/mesa/state_tracker/st_context.c
> +++ b/src/mesa/state_tracker/st_context.c
> @@ -238,6 +238,10 @@ void st_destroy_context( struct st_context *st )
>  
>     pipe->set_index_buffer(pipe, NULL);
>  
> +   for (i = 0; i < PIPE_SHADER_TYPES; i++) {
> +      pipe_resource_reference(&st->state.constants[PIPE_SHADER_VERTEX], NULL);
> +   }
> +
>     _mesa_delete_program_cache(st->ctx, st->pixel_xfer.cache);
>  
>     _vbo_DestroyContext(st->ctx);

With "st/mesa: unbind constant buffer when not in use"
(8dfafbf0861fe3d254233), we should probably use
pipe->set_constant_buffer() instead.

Regards,
Tilman

-- 
A: Because it messes up the order in which people normally read text.
Q: Why is top-posting such a bad thing?
A: Top-posting.
Q: What is the most annoying thing on usenet and in e-mail?
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20101102/f708bdd0/attachment.pgp>


More information about the mesa-dev mailing list