<p>Richard Mudgett <strong>uploaded patch set #2</strong> to this change.</p><p><a href="https://gerrit.asterisk.org/10189">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">stasis: No need to keep a stasis type ref in a stasis msg or cache object.<br><br>Stasis message types are global ao2 objects and we make stasis messages<br>and cache entries hold references to them.  Since there are currently<br>situations where cache objects are never deleted, the reference count on<br>the types can exceed 100000 and generate a FRACK assertion message.  The<br>stasis message cache could conceivably also have that many messages<br>legitimately on large systems.<br><br>The only down side to not holding the message type ref in the stasis<br>message is it only makes a crash either at shutdown or when manually<br>unloading a busy module slightly more likely.  However, this is more<br>exposing a pre-existing stasis shutdown ordering issue than a problem with<br>not holding a message type ref in stasis messages.<br><br>* Made stasis messages and cache entries no longer hold a ref to the<br>message type.<br><br>Change-Id: Ibaa28efa8d8ad3836f0c65957192424c7f561707<br>---<br>M main/stasis_cache.c<br>M main/stasis_message.c<br>2 files changed, 18 insertions(+), 4 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/89/10189/2</pre><p>To view, visit <a href="https://gerrit.asterisk.org/10189">change 10189</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.asterisk.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.asterisk.org/10189"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 13 </div>
<div style="display:none"> Gerrit-MessageType: newpatchset </div>
<div style="display:none"> Gerrit-Change-Id: Ibaa28efa8d8ad3836f0c65957192424c7f561707 </div>
<div style="display:none"> Gerrit-Change-Number: 10189 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: Richard Mudgett <rmudgett@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>
<div style="display:none"> Gerrit-Reviewer: Richard Mudgett <rmudgett@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Sean Bright <sean.bright@gmail.com> </div>