[PATCH 06/10] glamor: Select VBO path by ARB_mbr extension.

Eric Anholt eric at anholt.net
Fri Mar 21 13:52:36 PDT 2014


Markus Wick <markus at selfnet.de> writes:

> The mbr path was hard coded enabled for desktop gl and disabled for gles.
> But there are both, desktop without mbr and mobiles with mbr.

I've pulled patches 1-9 for my next pull request, but I think in this
patch we should also apply this:

diff --git a/glamor/glamor_vbo.c b/glamor/glamor_vbo.c
index c6ffea8..2731692 100644
--- a/glamor/glamor_vbo.c
+++ b/glamor/glamor_vbo.c
@@ -179,7 +179,7 @@ glamor_fini_vbo(ScreenPtr screen)
     glamor_get_context(glamor_priv);
 
     glDeleteBuffers(1, &glamor_priv->vbo);
-    if (glamor_priv->gl_flavor != GLAMOR_GL_DESKTOP)
+    if (!glamor_priv->has_map_buffer_range)
         free(glamor_priv->vb);
 
     glamor_put_context(glamor_priv);

Sound good?
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 818 bytes
Desc: not available
URL: <http://lists.x.org/archives/xorg-devel/attachments/20140321/8af01d93/attachment.sig>


More information about the xorg-devel mailing list