<p>Pau Espin Pedrol has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.osmocom.org/9514">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">compact AGCH queue: Drop too msg diff than IMM_ASS_REJ<br><br>We saw in a recent prod setup a BSC with saturated channels. Further<br>investigation lead to a AGCH queue of 1000 (previous hard_limit)<br>messages, most of them being regular IMM ASSIGN (non REJ). Hence, we<br>also want to get rid of other messages in the AGCH queue (like regular<br>IMM ASSIGN).<br><br>Furthermore, In this scenario, sending IMM ASS REJ is as important as other<br>messages given than nowadays we support dynamic wait indicatior<br>(calculated based on chan load), which means if we reach the MS, we can<br>tell it to wait for a long time to reach us again, which is desirable.<br><br>Change-Id: I022b8948da8be13fb8f4bc36e7c9dab11c35fddb<br>---<br>M src/common/bts.c<br>1 file changed, 0 insertions(+), 5 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/osmo-bts refs/changes/14/9514/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 1315a0a..69acad2 100644</span><br><span>--- a/src/common/bts.c</span><br><span>+++ b/src/common/bts.c</span><br><span>@@ -615,11 +615,6 @@</span><br><span>             struct gsm48_imm_ass *imm_ass_cmd = msgb_l3(msg);</span><br><span>            int p_drop;</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-         if (imm_ass_cmd->msg_type != GSM48_MT_RR_IMM_ASS_REJ)</span><br><span style="color: hsl(0, 100%, 40%);">-                        return;</span><br><span style="color: hsl(0, 100%, 40%);">-</span><br><span style="color: hsl(0, 100%, 40%);">-         /* IMMEDIATE ASSIGN REJECT */</span><br><span style="color: hsl(0, 100%, 40%);">-</span><br><span>                p_drop = (bts->agch_queue.length - offs) * slope / max_len;</span><br><span> </span><br><span>           if ((random() & 0xffff) >= p_drop)</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/9514">change 9514</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/9514"/><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: I022b8948da8be13fb8f4bc36e7c9dab11c35fddb </div>
<div style="display:none"> Gerrit-Change-Number: 9514 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Pau Espin Pedrol <pespin@sysmocom.de> </div>