[PATCH xserver 01/11] glamor: No need to glFlush before destroying a pixmap.
Eric Anholt
eric at anholt.net
Fri Nov 6 12:51:55 PST 2015
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 <eric at anholt.net>
---
glamor/glamor_egl.c | 4 ----
1 file changed, 4 deletions(-)
diff --git a/glamor/glamor_egl.c b/glamor/glamor_egl.c
index 2e6c7bd..cc16b0a 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;
}
--
2.6.2
More information about the xorg-devel
mailing list