<p>Pau Espin Pedrol has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.osmocom.org/9513">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">bts_agch_enqueue: Decrease queue len hard_limit from 1000 to 50<br><br>In a prod setup, complete channel saturation at the bsc was detected,<br>and Immediate Assignments were not being answered by the MS once sent by<br>the BTS.<br>Further investigation showed that the BTS was all the time printing<br>messages like this:<br>"bts.c:540 AGCH: too many messages in queue, refusing message type 0x3f, length = 1001/10"<br><br>So it seems the AGCH queue was becoming incredibly full (1000, hard<br>limit triggered the log), while acgch_queue.max_length was set to 10.<br><br>As a result, most probably the Immediate Assignments being sent to the<br>MS are super old in time, and the MS doesn't known about them anymore<br>once they are receivied, so no answer is sent back.<br><br>Let's try to avoid that by decreasing the hard limit so we never reach<br>that big queue_len scenario.<br><br>Related: SYS#2695<br>Change-Id: I272798c959abec123776d2fa8dad5685ec512fbd<br>---<br>M src/common/bts.c<br>1 file changed, 1 insertion(+), 1 deletion(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/osmo-bts refs/changes/13/9513/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/common/bts.c b/src/common/bts.c</span><br><span>index 55f4b94..1315a0a 100644</span><br><span>--- a/src/common/bts.c</span><br><span>+++ b/src/common/bts.c</span><br><span>@@ -533,7 +533,7 @@</span><br><span> </span><br><span> int bts_agch_enqueue(struct gsm_bts *bts, struct msgb *msg)</span><br><span> {</span><br><span style="color: hsl(0, 100%, 40%);">-     int hard_limit = 1000;</span><br><span style="color: hsl(120, 100%, 40%);">+        int hard_limit = 50;</span><br><span>         struct gsm48_imm_ass_rej *imm_ass_cmd = msgb_l3(msg);</span><br><span> </span><br><span>    if (bts->agch_queue.length > hard_limit) {</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/9513">change 9513</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.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.osmocom.org/9513"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-bts </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>
<div style="display:none"> Gerrit-Change-Id: I272798c959abec123776d2fa8dad5685ec512fbd </div>
<div style="display:none"> Gerrit-Change-Number: 9513 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Pau Espin Pedrol <pespin@sysmocom.de> </div>