[asterisk-dev] [Code Review] Optimize chan_sip.c check_rtp_timeout() function

Rob Gagnon reviewboard at asterisk.org
Tue Aug 23 14:53:25 CDT 2011


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

(Updated Aug. 23, 2011, 2:53 p.m.)


Review request for Asterisk Developers, Paul Belanger, rmudgett, and Rob Gagnon.


Changes
-------

Removed inline declaration per coding guidelines.  Put them to the top unintialized.


Summary
-------

Copying from original post on Jira:

While reviewing how "rtptimeout" and "rtpholdtimeout" operate in code, I found that the check_rtp_timeout() function could be optimized a little to speed up asterisk performance.

Currently, three integers are fetched from the rtp instance multiple times apiece (causing of course, multiple stack operations)

In the worst-case scenario:
 - ast_rtp_instance_get_timeout() is called 4 times.
 - ast_rtp_instance_get_hold_timeout() is called 4 times.
 - ast_rtp_instance_get_keepalive() is called 3 times.
 
With this patch, each function will be called only once, thus removing up to 9 stack push/pops during runtime
Description posted on jira would be the same


This addresses bug ASTERISK-18319.
    https://issues.asterisk.org/jira/browse/ASTERISK-18319


Diffs (updated)
-----

  /trunk/channels/chan_sip.c 333066 

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


Testing
-------

Compiled code with patch, ran on test system without any problems.  Turned logging way up, and verifiied that calls that were silent/disconnected by accident were still being hung-up on.


Thanks,

Rob

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


More information about the asterisk-dev mailing list