Skip to content
Commit 5e8af9e6 authored by Tomeu Vizoso's avatar Tomeu Vizoso Committed by Emil Velikov
Browse files

etnaviv: Consolidate buffer references from framebuffers



We were leaking surfaces because the references taken in
etna_set_framebuffer_state weren't being released on context destroy.

Instead of just directly releasing those references in
etna_context_destroy, use the util_copy_framebuffer_state helper.

Take the chance to remove the duplicated buffer references in
compiled_framebuffer_state to avoid confusion.

The leak can be reproduced with a client that continuously creates and
destroys contexts.

Signed-off-by: default avatarTomeu Vizoso <tomeu.vizoso@collabora.com>
Reported-by: default avatarSjoerd Simons <sjoerd.simons@collabora.co.uk>
Reviewed-by: default avatarChristian Gmeiner <christian.gmeiner@gmail.com>
(cherry picked from commit bf1dfcc3)
[Emil: resolve trivial conflict - dummy_rt does not exist in branch]
Signed-off-by: default avatarEmil Velikov <emil.velikov@collabora.com>

Conflicts:
	src/gallium/drivers/etnaviv/etnaviv_context.c
parent f0725855
Loading
Loading
Loading
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment