<p>George Joseph <strong>uploaded patch set #2</strong> to this change.</p><p><a href="https://gerrit.asterisk.org/5987">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">res_musiconhold:  Add kill_escalation_delay, kill_method to class<br><br>By default, when res_musiconhold reloads or unloads, it sends a HUP<br>signal to custom applications (and all descendants), waits 100ms,<br>then sends a TERM signal, waits 100ms, then finally sends a KILL<br>signal.  An application which is interacting with an external<br>device and/or spawns children of its own may not be able to exit<br>cleanly in the default times, expecially if sent a KILL signal, or<br>if it's children are getting signals directly from<br>res_musiconhoild.<br><br>* To allow extra time, the 'kill_escalation_delay'<br>  class option can be used to set the number of milliseconds<br>  res_musiconhold waits before escalating kill signals, with the<br>  default being the current 100ms.<br><br>* To control to whom the signals are sent, the "kill_method" class<br>  option can be set to "process_group" (the default, existing<br>  behavior), which sends signals to the application and its<br>  descendants directly, or "process" which sends signals only to the<br>  application itself.<br><br>Change-Id: Iff70a1a9405685a9021a68416830c0db5158603b<br>---<br>M CHANGES<br>M configs/samples/musiconhold.conf.sample<br>M res/res_musiconhold.c<br>3 files changed, 115 insertions(+), 49 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/87/5987/2</pre><p>To view, visit <a href="https://gerrit.asterisk.org/5987">change 5987</a>. To unsubscribe, 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/5987"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: certified/13.13 </div>
<div style="display:none"> Gerrit-MessageType: newpatchset </div>
<div style="display:none"> Gerrit-Change-Id: Iff70a1a9405685a9021a68416830c0db5158603b </div>
<div style="display:none"> Gerrit-Change-Number: 5987 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>
<div style="display:none"> Gerrit-Reviewer: Matthew Fredrickson <creslin@digium.com> </div>