[Bug 704806] Implement GL context sharing

[GStreamer (bugzilla.gnome.org)]

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

--- Comment #5 from Matthew Waters <ystreet00 at gmail.com> 2013-08-29 13:16:29 UTC ---
(In reply to comment #3)
> Review of attachment 253481 [details]:
> 
> ::: gst-libs/gst/gl/egl/gstglcontext_egl.c
> @@ +24,3 @@
> 
> +/* FIXME: Sharing contexts requires the EGLDisplay & EGLConfig to be the same
> + * may need to box it.
> 
> The config should always be the same if the display is the same. A display can
> only be initialized to one config iirc

So it does :).  I'll fix that.

(In reply to comment #4)
> Additional to that it would now make sense to change all these external-context
> GObject properties to "other-context" for consistency and make them of type
> GstGLContext instead of guintptr/guint64. Can you do that? :)

Can do.

> Also, does this really already fix the case where we have two contexts. One
> producing a texture, one wanting to render it?

With WGL, GLX and EGL, yes.  I can't test this on a real Cocoa environment and
GNUstep apparently doesn't like shared contexts :(.  Also, that's exactly what
the test case does and what I used to test this.  One thread draws to the fbo
(with a texture as an attachment) as fast it can, the other renders the texture
to the window).  Try it :).

The only thing that needs to change later is for the moving of the vtable from
GstGLDisplay to GstGLContext.

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