After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 712287 - Use GL_TEXTURE_2D (npot) instead of GL_TEXTURE_RECTANGLE
Use GL_TEXTURE_2D (npot) instead of GL_TEXTURE_RECTANGLE
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-gl
git master
Other Linux
: Normal enhancement
: 1.2.0
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2013-11-14 11:30 UTC by Matthew Waters (ystreet00)
Modified: 2013-11-26 11:17 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
use GL_TEXTURE_2D instead of GL_TEXTURE_RECTANGLE (147.58 KB, patch)
2013-11-15 07:39 UTC, Matthew Waters (ystreet00)
none Details | Review
use GL_TEXTURE_2D instead of GL_TEXTURE_RECTANGLE (147.89 KB, patch)
2013-11-15 10:20 UTC, Matthew Waters (ystreet00)
committed Details | Review

Description Matthew Waters (ystreet00) 2013-11-14 11:30:50 UTC
In my quest for vaapi integration.  I came across the following problem.

We create our textures (in Desktop GL) with GL_TEXTURE_RECTANGLE, vaapi attempts to bind our texture to GL_TEXTURE_2D which throws a GL_INVALID_OPERATION error and as thus, no video.

By moving exclusively to GL_TEXTURE_2D and the npot extension we also remove a difference between the Desktop GL and GLES2 code.
Comment 1 Matthew Waters (ystreet00) 2013-11-15 07:39:24 UTC
Created attachment 259857 [details] [review]
use GL_TEXTURE_2D instead of GL_TEXTURE_RECTANGLE
Comment 2 Julien Isorce 2013-11-15 08:36:22 UTC
+1 :)
Comment 3 Tim-Philipp Müller 2013-11-15 10:00:41 UTC
It would be nice to have the rationale in the commit message as well :)
Comment 4 Matthew Waters (ystreet00) 2013-11-15 10:20:57 UTC
Created attachment 259861 [details] [review]
use GL_TEXTURE_2D instead of GL_TEXTURE_RECTANGLE
Comment 5 Sebastian Dröge (slomo) 2013-11-26 11:17:29 UTC
commit fffdcbdd3dde437e10cf6e1b5e40c6958d41e6c1
Author: Matthew Waters <ystreet00@gmail.com>
Date:   Fri Nov 15 18:28:49 2013 +1100

    gl: Use GL_TEXTURE_2D instead of GL_TEXTURE_RECTANGLE
    
    We create our textures (in Desktop GL) with GL_TEXTURE_RECTANGLE,
    vaapi attempts to bind our texture to GL_TEXTURE_2D which throws a
    GL_INVALID_OPERATION error and as thus, no video.
    
    Also, by moving exclusively to GL_TEXTURE_2D and the npot extension
    we also remove a difference between the Desktop GL and GLES2 code.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=712287