[Mesa-dev] [PATCH] st/mesa: remove unneeded pipe_surface_release() in st_render_texture()

[Marek Olšák]

Reviewed-by: Marek Olšák <marek.olsak at amd.com>

Marek

On Tue, Jun 23, 2015 at 6:30 PM, Brian Paul <brianp at vmware.com> wrote:
> This caused us to always free the pipe_surface for the renderbuffer.
> The subsequent call to st_update_renderbuffer_surface() would typically
> just recreate it.  Remove the call to pipe_surface_release() and let
> st_update_renderbuffer_surface() take care of freeing the old surface
> if it needs to be replaced (because of change to mipmap level, etc).
>
> This can save quite a few calls to pipe_context::create_surface() and
> surface_destroy().
> ---
>  src/mesa/state_tracker/st_cb_fbo.c | 2 --
>  1 file changed, 2 deletions(-)
>
> diff --git a/src/mesa/state_tracker/st_cb_fbo.c b/src/mesa/state_tracker/st_cb_fbo.c
> index 0399eef..5707590 100644
> --- a/src/mesa/state_tracker/st_cb_fbo.c
> +++ b/src/mesa/state_tracker/st_cb_fbo.c
> @@ -511,8 +511,6 @@ st_render_texture(struct gl_context *ctx,
>     strb->rtt_layered = att->Layered;
>     pipe_resource_reference(&strb->texture, pt);
>
> -   pipe_surface_release(pipe, &strb->surface);
> -
>     st_update_renderbuffer_surface(st, strb);
>
>     strb->Base.Format = st_pipe_format_to_mesa_format(pt->format);
> --
> 1.9.1
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev