<p style="white-space: pre-wrap; word-wrap: break-word;">It's really good that you appear to have manged to uncover the mystery of UEs stickin to GSM in combined GSM+LTE setups.  However, some stylistic comments below on bitvec API usage.</p><p>Patch set 1:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #ffd4d4;">Code-Review -1</span></p><p><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/14678">View Change</a></p><p>3 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/14678/1/src/osmo-bsc/rest_octets.c">File src/osmo-bsc/rest_octets.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/14678/1/src/osmo-bsc/rest_octets.c@198">Patch Set #1, Line 198:</a> </p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><pre style="font-family: monospace,monospace; white-space: pre-wrap;">bitvec_set_bit(bv, 0);<br>    bitvec_set_bit(bv, 0);<br>        bitvec_set_bit(bv, 0);<br></pre></blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">I suppose this is a 3bit field encoding an unsigned integer so I would encode it not as 3 individual bits but use bitvec_set_uint(bv, 0, 3);</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/14678/1/src/osmo-bsc/rest_octets.c@203">Patch Set #1, Line 203:</a> </p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><pre style="font-family: monospace,monospace; white-space: pre-wrap;">bitvec_set_bit(bv, 0);<br>     bitvec_set_bit(bv, 0);<br>        bitvec_set_bit(bv, 0);<br>        bitvec_set_bit(bv, 0);<br></pre></blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">same here. it's a 4-bit integer that should use bitvec_set_uint(bv, 0, 4);  The advantage is that later on, if you want to use anything else but '0' you have one actual integer value in the code to change, rather than four individual bits.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/14678/1/src/osmo-bsc/rest_octets.c@208">Patch Set #1, Line 208:</a> </p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><pre style="font-family: monospace,monospace; white-space: pre-wrap;">      /* THRESH_GSM_low */<br>  bitvec_set_bit(bv, 0);<br>        bitvec_set_bit(bv, 0);<br>        bitvec_set_bit(bv, 0);<br>        bitvec_set_bit(bv, 0);<br><br>      /* H_PRIO */<br>  bitvec_set_bit(bv, 0);<br>        bitvec_set_bit(bv, 0);<br><br>      /* T_Reselection */<br>   bitvec_set_bit(bv, 0);<br>        bitvec_set_bit(bv, 1);<br><br></pre></blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">same comment applies for these, I suppose</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bsc/+/14678">change 14678</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/+/14678"/><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: I7eaf7de4386fe8aea404e8a187d8a1f5ed596ead </div>
<div style="display:none"> Gerrit-Change-Number: 14678 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Hoernchen <ewild@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Sun, 07 Jul 2019 08:57:48 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: Yes </div>
<div style="display:none"> Gerrit-MessageType: comment </div>