[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 #5 from Matthew Waters <ystreet00 at gmail.com> 2014-04-09 23:19:46 UTC ---
(In reply to comment #4)
> > If said intermediate plugin does not support the GstVideoGLTextureUploadMeta,
> > then you negotiate sysmem (or fail).  I can't think of a case for supporting
> > mapping buffers and using upload meta by the same element.
> 
> What if a tee element is used, one branch goes to a video sink which supports
> that meta, and another branch goes to deinterlacer+videoresize+rtp
> payloader+udpsink elements for example?

Then negotiation will fail based on caps features.  Either both sides of the
tee will support the meta or will not.

> Changing the format in the caps just because a meta is added just feels wrong
> to me. The caps should describe what is really inside. Additional mechanisms
> like this meta are placed on top of that. They do not replace anything.

Also, there is no guarantee that mapping the buffer is supported when the meta
exists, I would even suggest that mapping a buffer with the meta would be
undefined behaviour.  The meta provides a way to get the data from a buffer to
some GL texture/s of a specified format without downstream explicitly mapping
the buffer.

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