Split the global registry into its own wl_registry object
authorKristian Høgsberg <krh@bitplanet.net>
Mon, 8 Oct 2012 17:53:47 +0000 (13:53 -0400)
committerKristian Høgsberg <krh@bitplanet.net>
Thu, 11 Oct 2012 00:59:00 +0000 (20:59 -0400)
commit9fe75537ad207c1496e6d9be41a8f5af4b876506
tree418b2b6857f458bddbdf9ca9864310c690f6fa64
parent8872956dfd43d36e4165d15cb50d8ef4f81fbe0d
Split the global registry into its own wl_registry object

The only way to make the global object listener interface thread safe is to
make it its own interface and make different listeners different wl_proxies.
The core of the problem is the callback we do when a global show up or
disappears, which we can't do with a lock held.  On the other hand we can't
iterate the global list or the listener list without a lock held as new
globals or listeners may come and go during the iteration.

Making a copy of the list under the lock and then iterating after dropping
the lock wont work either.  In case of the listener list, once we drop the
lock another thread may unregister a listener and destroy the callbackk
data, which means that when we eventually call that listener we'll pass it
free memory and break everything.

We did already solve the thread-safe callback problem, however.  It's what
we do for all protocol events.  So we can just make the global registry
functionality its own new interface and give each thread its own proxy.
That way, the thread will do its own callbacks (with no locks held) and
destroy the proxy when it's no longer interested in wl_registry events.
protocol/wayland.xml
src/wayland-client.c
src/wayland-client.h
src/wayland-server.c