[Mesa-dev] [PATCH 04/10] egl/android: Stop leaking DRI images
eric at anholt.net
Fri Jul 15 18:56:44 UTC 2016
Tomasz Figa <tfiga at chromium.org> writes:
> Hi Eric,
> On Sat, Jul 16, 2016 at 3:05 AM, Eric Anholt <eric at anholt.net> wrote:
>> Tomasz Figa <tfiga at chromium.org> writes:
>>> Current implementation of the DRI image loader does not free the images
>>> created in get_back_bo() and so leaks memory. Moreover, it creates a new
>>> image every time the DRI driver queries for buffers, even if the backing
>>> native buffer has not changed. leaking memory again.
>>> This patch adds missing call to destroyImage() in droid_enqueue_buffer()
>>> and a check if image is already created to get_back_bo() to fix the
>> This patch is:
>> Reviewed-by: Eric Anholt <eric at anholt.net>
>> But I noticed in review, it looks like droid_destroy_surface() could
>> also use an image destroy.
> Thanks for review!
> It already calls droid_window_cancel_buffer(), which in turns calls
> droid_window_enqueue_buffer() (typo in commit message...). Or you mean
> something else I didn't notice?
Ah, I missed that path. Excellent.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 818 bytes
Desc: not available
More information about the mesa-dev