[Bug 727886] GstVideoGLTextureUploadMeta assumes no color conversion is done during upload

[GStreamer (bugzilla.gnome.org)]

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

--- Comment #28 from Carlos Rafael Giani <dv at pseudoterminal.org> 2014-04-16 12:58:35 UTC ---
(In reply to comment #27)
> Sorry, I'm not very clear on the cropping issue. I do remember then in big GL
> we could query the texture size. What I'm not too clear is how is that going to
> solve your issue. Would it be possible to elaborate on that. We need to make
> sure this won't also be an issue for other stack, that this solution is not
> Vivante specific.

The main detail to keep in mind here is that the Vivante extension does not
allow you to specify plane offsets, strides, padding etc. explicitely. It
assumes width = stride*bytesperpixel, and that planes are tightly packed
together, that is, after the last row of plane 1 comes the first row of plane 2
etc. it is a bit more flexible if you actually upload the pixels, but if you
specify a physical address to a DMA buffer, you have zero freedom.

So, the trick was to include padding etc. into the texture width & height. The
actual frame would then be a subregion of this texture.

I agree that we must make sure this does not become too Vivante specific.
However, I have seen this limitation in several other APIs (that is, no fields
for stride, plane offsets etc.) so I believe this is still generic enough.

-- 
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.