<p>neels <strong>uploaded patch set #3</strong> to this change.</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 repeated EARFCNs encoding<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. Finally there is a '0' bit to indicate the list end.<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>The final '0' was missing. Add it.<br><br>With these changes, adjust the size calculation in<br>CELL_SEL_IND_AFTER_REL_MAX_BITS to match.<br><br>Also fix CELL_SEL_IND_AFTER_REL_MAX_BYTES by using OSMO_BYTES_FOR_BITS()<br>instead of the inaccurate (n/8)+1.<br><br>A test for this is in I882c5e1f70bcc4833fc837a95c900ce291919cc5<br>(osmo-ttcn3-hacks).<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, 8 insertions(+), 4 deletions(-)<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/3</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: 3 </div>
<div style="display:none"> Gerrit-Owner: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Reviewer: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: newpatchset </div>