[Nice] API thoughts

[Olivier Crête]

On Thu, 2008-04-17 at 09:53 +0300, Kai.Vehmanen at nokia.com wrote:
> Hi,
> 
> On 15 April 2008,  Dafydd Harries wrote:
> >> Could we somehow support both models? Or is there a way to run a set 
> >> of GMainContexts in a single GMainloop (without writing a custom 
> >
> >Oh, perhaps I wasn't clear. I absolutely want to have a good 
> >API for single-threaded applications too.
> 
> aa, ok, then we are on the same page! :)
> 
> >> Hmm, yes, this should be doable. Although the locking has to be done 
> >> very carefully (inbound STUN messages affect agent-wide state 
> >> in many ways, see conncheck.c:conn_check_handle_inbound_stun()).
> >
> >Well, I was thinking that the initial approach could just be 
> >to make any handling of inbound STUN packets mutually 
> >exclusive. We can make the locking finer later, or perhaps 
> >look at using some kind of lock-free update algorithm.
> 
> Yes, but you also need to lock the state in all public API 
> functions (e.g. in case remote peer restarts the ICE
> negotiation), plus in the timer callbacks (which are running
> in the "shared context"). But agreed, still very much doable.

Youness already did a branch to do that, its on the monkey.

-- 
Olivier Crête
olivier.crete at collabora.co.uk
Collabora Ltd
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part
Url : http://lists.freedesktop.org/archives/nice/attachments/20080417/67158c2c/attachment.pgp