[asterisk-dev] [Code Review] chan_local locking cleanup

David Vossel reviewboard at asterisk.org
Wed May 25 09:06:26 CDT 2011


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

(Updated 2011-05-25 09:06:26.224151)


Review request for Asterisk Developers.


Changes
-------

Address's Alex's comments.  I tested this under load now and everything appears in order for be at least.  Alex's problem was a result of a typo, I reversed the chan and owner channels going into the awesome_locking function in local_call.


Summary
-------

Introduces the awesome_locking() function and cleans up all the unnecessary deadlock avoidance junk we do.


Diffs (updated)
-----

  /trunk/channels/chan_local.c 320715 

Diff: https://reviewboard.asterisk.org/r/1231/diff


Testing
-------

I made a call. Obviously more testing is required under load.


Thanks,

David

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


More information about the asterisk-dev mailing list