diff options
authorDerek Foreman <derekf@osg.samsung.com>2018-01-30 16:29:32 -0600
committerDerek Foreman <derekf@osg.samsung.com>2018-01-30 17:09:51 -0600
commit9632c7b56cc41cef3c1811b745e207599d427607 (patch)
parentecore_wl2: Add new event for windows not displayed on any output (diff)
wl2_surface_dmabuf: improve surface flush
We used to abandon all buffers even if they were locked. This can't actually free them until they're all released by the compositor. Instead just free any buffer the compositor doesn't have locked, so we can still re-use them when they're released without needing a full redraw. There's probably room for additional cleverness here. If we get a new frame event before the buffer release we may want to keep it, and if we get the release first we may want to abandon it.
1 files changed, 8 insertions, 1 deletions
diff --git a/src/lib/ecore_wl2/ecore_wl2_surface_module_dmabuf.c b/src/lib/ecore_wl2/ecore_wl2_surface_module_dmabuf.c
index a2ddb9119a..2be8ded399 100644
--- a/src/lib/ecore_wl2/ecore_wl2_surface_module_dmabuf.c
+++ b/src/lib/ecore_wl2/ecore_wl2_surface_module_dmabuf.c
@@ -193,7 +193,14 @@ _evas_dmabuf_surface_flush(Ecore_Wl2_Surface *surface EINA_UNUSED, void *priv_da
p = priv_data;
EINA_LIST_FREE(p->buffers, b)
- ecore_wl2_buffer_destroy(b);
+ {
+ if (!ecore_wl2_buffer_busy_get(b))
+ {
+ if (p->current == b)
+ p->current = NULL;
+ ecore_wl2_buffer_destroy(b);
+ }
+ }
static Ecore_Wl2_Surface_Interface dmabuf_smanager =