[Bug 776927] gl: gldownload: convert GstGLMemory to GstDmaBuf

[GStreamer (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=776927

Matt Fischer <mattfischer84 at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #343926|reviewed                    |none
             status|                            |
 Attachment #343926|0                           |1
        is obsolete|                            |

--- Comment #22 from Matt Fischer <mattfischer84 at gmail.com> ---
Created attachment 345062
  --> https://bugzilla.gnome.org/attachment.cgi?id=345062&action=edit
Patch to add dmabuf export to gldownload

Here's a patch which adds the use of the memory:DMABuf feature.  That does seem
like the right way to go, although it will require the VAAPI plugins to expose
that cap feature on their sink pads before this will work correctly.  Victor,
if this patch were accepted, would that be an ok thing to add to the VAAPI
plugins?

Currently it exposes the feature whenever dmabuf support and EGL support is
compiled in.  It does not yet attempt to query the context at runtime to
determine if EGLImage exporting is possible.  I'm not exactly sure how to do
that timing-wise, because we have to announce the caps during negotiation, but
we may not have a context until some time later.  So I'm worried that by the
time we get a context, it will be too late to disable the dmabuf feature if we
discover the context doesn't support it.  Are there any examples of this kind
of thing being done right now that I could go and look at?

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.