From 98a1993536add730b7ec29a9e37f62b1cd70ad31 Mon Sep 17 00:00:00 2001 From: Eric Anholt Date: Sun, 18 Oct 2015 19:16:20 -0700 Subject: [PATCH] glamor: No need to glFlush before destroying a pixmap. MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit I assume this was a workaround for an old, broken, closed driver. The driver doesn't get to throw away rendering just because the rendering context's shared-across-processes render target is getting freed from the local address space. If the rendering isn't to a shared render target, then we *do* want to throw away the rendering to it. Signed-off-by: Eric Anholt Reviewed-by: Michel Dänzer --- glamor/glamor_egl.c | 4 ---- 1 file changed, 4 deletions(-) diff --git a/glamor/glamor_egl.c b/glamor/glamor_egl.c index 2e6c7bdbc..cc16b0a75 100644 --- a/glamor/glamor_egl.c +++ b/glamor/glamor_egl.c @@ -609,10 +609,6 @@ glamor_egl_destroy_pixmap_image(PixmapPtr pixmap) struct glamor_egl_screen_private *glamor_egl = glamor_egl_get_screen_private(scrn); - /* Before destroy an image which was attached to - * a texture. we must call glFlush to make sure the - * operation on that texture has been done.*/ - glamor_block_handler(pixmap->drawable.pScreen); eglDestroyImageKHR(glamor_egl->display, pixmap_priv->image); pixmap_priv->image = NULL; }