diff options
| author | Brian Paul <[email protected]> | 2015-06-22 14:04:09 -0600 |
|---|---|---|
| committer | Brian Paul <[email protected]> | 2015-06-24 07:14:56 -0600 |
| commit | c1de7df6d4086070e63369ab0af3950f53a03592 (patch) | |
| tree | 782a3ff9eab4bb2b9aa722876fd986caf3025cd4 | |
| parent | a552c897caea31bbff3f16d2af8f5028a58bd344 (diff) | |
st/mesa: remove unneeded pipe_surface_release() in st_render_texture()
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().
Reviewed-by: Marek Olšák <[email protected]>
Reviewed-by: Jose Fonseca <[email protected]>
| -rw-r--r-- | src/mesa/state_tracker/st_cb_fbo.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/src/mesa/state_tracker/st_cb_fbo.c b/src/mesa/state_tracker/st_cb_fbo.c index 0399eef7204..57075904450 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); |