[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

George Kiagiadakis <kiagiadakis.george> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|Normal                      |High
           Severity|normal                      |critical

--- Comment #34 from George Kiagiadakis <kiagiadakis.george at gmail.com> 2014-04-23 07:50:53 UTC ---
(In reply to comment #33)
> > As far as I have seen, the number of textures equals the number of planes of
> > the given format. At least eglglessink used to assert this and glimagesink only
> > supports RGBA (and asserts that n_textures == 1), so I have no example of
> > another use case. Do you know one?
> 
> I don't. I've asked because I've remember that in some test we notice a
> performance hit when doing YUV in 3 planes compare to doing the same YUV in 1
> plane (was on Mali). But thinking of it, it might not be possible to represent
> YUV planes correctly without having separate textures.

Ok, I think that if there ever is any need for doing all planes in one texture,
then we could have a field in the "features" structure indicating this anomaly.
Even if we keep the n_textures field, it is impossible for a sink to somehow
infer the semantics of each different combination of textures number and planes
number.

This is why I like the idea of introducing the "features" structure, because we
can guarantee future compatibility with all kinds of weird devices without
breaking our API again. I can understand that maybe more research is needed to
find and document other hw anomalies, but I think it's a good idea to have it
even if there is no other weird hw out there atm, just to ensure that we won't
have to break our API if something comes up in the future.

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