<p>neels has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/19210">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">RR Release Cell selection IE: fix presence bit for repeated EARFCNs<br><br>3GPP 44.018 10.5.2.1e defines the EARFCNs encoded in the 'Cell selection<br>indicator after release of all TCH and SDCCH IE' as follows:<br><br>  <Cell Selection Indicator after release of all TCH and SDCCH value part> ::=<br>  [...]<br>  | 011 { 1 <E-UTRAN Description : < E-UTRAN Description struct >> } ** 0<br><br>So after a 3-bit discriminator of '3' there can be multiple E-UTRAN<br>Descriptions, and each of them starts with a '1' bit to indicate that another<br>item follows.<br><br>Before this patch, osmo-bsc only encoded the first '1' bit, and failed to<br>repeat this before each following E-UTRAN Description. Fix that by moving the<br>'1' encoding into the loop.<br><br>Related: SYS#4871 SYS#4872<br>Change-Id: I59e427e4ebb1c6af99b27a15c40fed82457ac8ab<br>---<br>M src/osmo-bsc/gsm_04_08_rr.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-bsc refs/changes/10/19210/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/osmo-bsc/gsm_04_08_rr.c b/src/osmo-bsc/gsm_04_08_rr.c</span><br><span>index f47c31f..635b8a2 100644</span><br><span>--- a/src/osmo-bsc/gsm_04_08_rr.c</span><br><span>+++ b/src/osmo-bsc/gsm_04_08_rr.c</span><br><span>@@ -255,7 +255,6 @@</span><br><span> </span><br><span>       /* E-UTRAN Description */</span><br><span>    bitvec_set_uint(&bv, 3, 3);</span><br><span style="color: hsl(0, 100%, 40%);">- bitvec_set_bit(&bv, 1);</span><br><span> </span><br><span>      for (i = 0; i < MAX_EARFCN_LIST; i++) {</span><br><span>           const struct osmo_earfcn_si2q *e = &bts->si_common.si2quater_neigh_list;</span><br><span>@@ -266,6 +265,7 @@</span><br><span>                        LOGP(DRR, LOGL_NOTICE, "%s: Not enough room to store EARFCN %u in the "</span><br><span>                            "Cell Selection Indicator IE\n", gsm_bts_name(bts), e->arfcn[i]);</span><br><span>               } else {</span><br><span style="color: hsl(120, 100%, 40%);">+                      bitvec_set_bit(&bv, 1);</span><br><span>                  bitvec_set_uint(&bv, e->arfcn[i], 16);</span><br><span>                        /* No "Measurement Bandwidth" */</span><br><span>                   bitvec_set_bit(&bv, 0);</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bsc/+/19210">change 19210</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/c/osmo-bsc/+/19210"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-bsc </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I59e427e4ebb1c6af99b27a15c40fed82457ac8ab </div>
<div style="display:none"> Gerrit-Change-Number: 19210 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>