[Bug 728815] New: GstGLMemory mix stride and width

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=728815
  GStreamer | gst-plugins-bad | git

           Summary: GstGLMemory mix stride and width
    Classification: Platform
           Product: GStreamer
           Version: git
        OS/Version: All
            Status: UNCONFIRMED
          Severity: normal
          Priority: Normal
         Component: gst-plugins-bad
        AssignedTo: gstreamer-bugs at lists.freedesktop.org
        ReportedBy: julien.isorce at gmail.com
         QAContact: gstreamer-bugs at lists.freedesktop.org
     GNOME version: ---


I have not investigated too much but worth to open a bug before to forget.

So in gst_gl_memory_setup_buffer we use GST_VIDEO_INFO_PLANE_STRIDE (info, i)
as the stride param of gst_gl_memory_alloc whereas in gstglupload or
gstglconvert we use the width.

And we use this param to compute the maxsize
http://cgit.freedesktop.org/gstreamer/gst-plugins-bad/tree/gst-libs/gst/gl/gstglmemory.c#n485

And maxsize is used for g_malloc
http://cgit.freedesktop.org/gstreamer/gst-plugins-bad/tree/gst-libs/gst/gl/gstglmemory.c#n896

So when doing gst-launch videotestrc ! video/x-raw, format=RGB ! glimagesink

2 textures are allocated with their glmem maxsize set to 960 (stride), and the
texture used for the FBO has its glmem maxsize set to 360.

I tried to replace GST_VIDEO_INFO_PLANE_STRIDE (info, i) by _get_plane_width
(info, i) here
http://cgit.freedesktop.org/gstreamer/gst-plugins-bad/tree/gst-libs/gst/gl/gstglmemory.c#n1087
 and the above pipeline still works.

Another thing is that the texture used for the FBO also has its glmem that does
a g_malloc, is it really useful ? maybe for download ? but worth to allocate
only if we actually do a download.

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.