diff options
| author | Pau Espin Pedrol <pespin@sysmocom.de> | 2018-06-08 16:29:51 +0200 |
|---|---|---|
| committer | Pau Espin Pedrol <pespin@sysmocom.de> | 2018-06-08 17:03:06 +0200 |
| commit | 868c67fed34d3a5f0d9d1e5150073fee3d71c26c (patch) | |
| tree | d09d12a1b025600ba4d1e85a8be801b9c2f58855 /src | |
| parent | a1a2556c2ae56399a0d24ff19c521c2a182cd31c (diff) | |
compact AGCH queue: Drop too msg diff than IMM_ASS_REJ
We saw in a recent prod setup a BSC with saturated channels. Further
investigation lead to a AGCH queue of 1000 (previous hard_limit)
messages, most of them being regular IMM ASSIGN (non REJ). Hence, we
also want to get rid of other messages in the AGCH queue (like regular
IMM ASSIGN).
Furthermore, In this scenario, sending IMM ASS REJ is as important as other
messages given than nowadays we support dynamic wait indicatior
(calculated based on chan load), which means if we reach the MS, we can
tell it to wait for a long time to reach us again, which is desirable.
Change-Id: I022b8948da8be13fb8f4bc36e7c9dab11c35fddb
Diffstat (limited to 'src')
| -rw-r--r-- | src/common/bts.c | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/src/common/bts.c b/src/common/bts.c index 4fa916ce..ae75b2c3 100644 --- a/src/common/bts.c +++ b/src/common/bts.c @@ -615,11 +615,6 @@ static void compact_agch_queue(struct gsm_bts *bts) struct gsm48_imm_ass *imm_ass_cmd = msgb_l3(msg); int p_drop; - if (imm_ass_cmd->msg_type != GSM48_MT_RR_IMM_ASS_REJ) - return; - - /* IMMEDIATE ASSIGN REJECT */ - p_drop = (bts->agch_queue.length - offs) * slope / max_len; if ((random() & 0xffff) >= p_drop) |