[asterisk-dev] [Code Review] Lower level SIP serialization

Mark Michelson reviewboard at asterisk.org
Mon Mar 4 09:36:28 CST 2013


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/2362/#review7975
-----------------------------------------------------------

Ship it!


+28394798234792984729874398298743982374

- Mark


On March 2, 2013, 12:31 p.m., Joshua Colp wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/2362/
> -----------------------------------------------------------
> 
> (Updated March 2, 2013, 12:31 p.m.)
> 
> 
> Review request for Asterisk Developers and Mark Michelson.
> 
> 
> Summary
> -------
> 
> While writing testsuite tests for the new SIP work I came across some weird crash issues. After examining them I determined that the issue was pjsip code continuing to modify state after we queue tasks, potentially even deallocating some resources we need. The attached change moves serialization to a much lower level so that serialization occurs before invoking any pjsip modules. This means that for most cases we no longer need to do serialization ourselves in pjsip callbacks. This can be set on a per-dialog basis so it is not limited to just the session work.
> 
> 
> Diffs
> -----
> 
>   /team/group/pimp_my_sip/include/asterisk/res_sip.h 382201 
>   /team/group/pimp_my_sip/res/res_sip.exports.in 382201 
>   /team/group/pimp_my_sip/res/res_sip/sip_distributor.c 382201 
>   /team/group/pimp_my_sip/res/res_sip_session.c 382201 
> 
> Diff: https://reviewboard.asterisk.org/r/2362/diff
> 
> 
> Testing
> -------
> 
> Ran normal calls through to confirm functionality works as it previously did. Ran against testsuite tests manually under heavy cps with obscure scenarios to confirm no issues.
> 
> 
> Thanks,
> 
> Joshua
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20130304/0da88cce/attachment.htm>


More information about the asterisk-dev mailing list