<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://reviewboard.asterisk.org/r/2873/">https://reviewboard.asterisk.org/r/2873/</a>
</td>
</tr>
</table>
<br />
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://reviewboard.asterisk.org/static/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Asterisk Developers.</div>
<div>By David Lee.</div>
<p style="color: grey;"><i>Updated Sept. 20, 2013, 4:05 p.m.</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Changes</h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Review feedback.
* Remove the memsets from internal_ao2_ref entirely, to help malloc_debug find
fence violations.
* s/i/idx/
* alloc proper size for route tables.
* comments
</pre>
</td>
</tr>
</table>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
Asterisk
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">This patch addresses several performance problems that were found in
the initial performance testing of Asterisk 12.
It turns out that we don't have many subscribers to Stasis topics in
the system. We have lots of caching and forwarding, but those don't
use the threadpool. Since we have such a limited number subscribers,
it made sense to simply use taskprocessors and allocate a thread per
subscriber instead.
The Stasis dispatch object was allocated as an AO2 object, even though
it has a very confined lifecycle. This was replaced with a straight
ast_malloc().
The Stasis message router was spending an inordinate amount of time
searching hash tables. In this case, most of our routers had 6 or
fewer routes in them to begin with. This was replaced with an array
that's searched linearly for the route.
We more heavily rely on AO2 objects in Asterisk 12, and the memset()
in ao2_ref() actually became noticeable on the profile. This was
#ifdef'ed to only run when AO2_DEBUG was enabled.
Finally, since we're more reliant on the taskprocessor, it was
optimized to reduce the number of times it was locking/unlocking the
tps mutex.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Using the setup below, sipp was run against the 12 branch both with
and without my changes.
==
[test-server]$ cat extensions.conf
[performance]
exten => service,1,Answer
exten => service,n,Wait(1)
exten => service,n,Hangup
[test-server]$ cat sip.conf
[perf](!)
type=peer
qualify=no
disallow=all
allow=g722
allow=ulaw
insecure=invite,port
context=performance
[test-client](perf)
host=test-client
Simple sipp scenario:
[test-client]$ sipp -sn uac -l 100 -r 50 tests-server</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> (updated)</h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>/branches/12/configs/stasis.conf.sample <span style="color: grey">(399546)</span></li>
<li>/branches/12/include/asterisk/stasis.h <span style="color: grey">(399546)</span></li>
<li>/branches/12/main/astobj2.c <span style="color: grey">(399546)</span></li>
<li>/branches/12/main/stasis.c <span style="color: grey">(399546)</span></li>
<li>/branches/12/main/stasis_config.c <span style="color: grey">(399546)</span></li>
<li>/branches/12/main/stasis_message_router.c <span style="color: grey">(399546)</span></li>
<li>/branches/12/main/taskprocessor.c <span style="color: grey">(399546)</span></li>
<li>/branches/12/res/res_pjsip/include/res_pjsip_private.h <span style="color: grey">(399546)</span></li>
</ul>
<p><a href="https://reviewboard.asterisk.org/r/2873/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>