[Bug 774518] gl/egl: request to revert commit c6326530 - don't install the gstglcontext_egl.h header

[GStreamer (GNOME Bugzilla)]

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

--- Comment #13 from Julien Isorce <julien.isorce at gmail.com> ---
(In reply to Matthew Waters (ystreet00) from comment #12)
> gst_egl_image_from_dmabuf should use gst_gl_context_get_proc_address() then.

Good idea in general. But it will return null for me, also since it is done
internally no need to change that part. In my chromium gst backend I do not
call ctx_egl->eglCreateImageKHR neither, this is not the problem :)

This is really how the gpu process works, I think you see it like a real gl api
with shared libraries and everything. This is not the case. It is an IPC and
only for GL, not for EGL. So I implemented IPC for eglCreateImage/Destroy but
it does not make sense for eglGetDisplay/eglInitialize, also because it is not
in the form ctx_egl->Initialize for example.
It means that gst_gl_context_egl_create_context cannot be called for the client
ipc that talks to the gpu process.

I need to split gst_gl_context_egl_create_context to extract the
egl->eglCreateImage feeding block. Do you think I can move it to
gst_gl_context_egl_get_proc_address ? making sure it is called (maybe by doing
the change you suggested in gst_egl_image_from_dmabuf )

I will still need to solve ->egl_exts, ->gl_api, ->egl_major, ->egl_minor
though.

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