<p>fixeria <strong>merged</strong> this change.</p><p><a href="https://gerrit.osmocom.org/c/osmo-bts/+/14817">View Change</a></p><div style="white-space:pre-wrap">Approvals:
  fixeria: Looks good to me, approved
  pespin: Looks good to me, but someone else must approve
  Jenkins Builder: Verified

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">common/rsl.c: fix: properly handle SI3 Rest Octets<br><br>It was noticed with old Sony Ericsson phones (like W595 and K510i)<br>that the service provided by Osmocom network becomes unreliable<br>from time to time. The RSSI indicator on those phones shows that<br>the signal is lost, so neither CS nor PS services are working.<br><br>As it then turned out, System Information 3 broadcasted on the<br>Um interface is different than the one received from the BSC.<br>In particular, the content of SI3 Rest Octets IE is different.<br><br>Among with the 'GPRS Indicator', which is actually expected to<br>indicate whether the PCU is connected or not, SI3 Rest Octets<br>on the Um interface contain both 'Optional Power Offset' and<br>'Scheduling if and where' IEs, which are not present in the<br>original messages from the BSC.<br><br>Moreover, as soon as the PCU is connected, 'GPRS Indicator' IE<br>contains different 'GPRS RA Colour' value, and informs the MS<br>that System Information 13 is sent on extended BCCH, which<br>is not even supported by OsmoBTS!<br><br>The culprit is in rsl_rx_bcch_info(), where we pass a pointer<br>to osmo_gsm48_rest_octets_si3_decode(). Instead of passing a<br>pointer to the beginning of SI3 buffer, we actually need to<br>shift it to the beginning of the SI3 Rest Octets IE.<br><br>This change makes my Sony Ericsson phones happy ;)<br><br>Change-Id: Ia962cf21903ba674057cf52746996dd3254bc1c6<br>---<br>M src/common/rsl.c<br>1 file changed, 3 insertions(+), 1 deletion(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/common/rsl.c b/src/common/rsl.c</span><br><span>index 0a6a4e5..d09dc4a 100644</span><br><span>--- a/src/common/rsl.c</span><br><span>+++ b/src/common/rsl.c</span><br><span>@@ -368,7 +368,9 @@</span><br><span>                                       LCHAN_REL_ACT_REACT;</span><br><span>                         }</span><br><span>                    /* decode original SI3 Rest Octets as sent by BSC */</span><br><span style="color: hsl(0, 100%, 40%);">-                    osmo_gsm48_rest_octets_si3_decode(&bts->si3_ro_decoded, GSM_BTS_SI(bts, osmo_si));</span><br><span style="color: hsl(120, 100%, 40%);">+                     const uint8_t *si3_ro_buf = (uint8_t *) GSM_BTS_SI(bts, osmo_si);</span><br><span style="color: hsl(120, 100%, 40%);">+                     si3_ro_buf += offsetof(struct gsm48_system_information_type_3, rest_octets);</span><br><span style="color: hsl(120, 100%, 40%);">+                  osmo_gsm48_rest_octets_si3_decode(&bts->si3_ro_decoded, si3_ro_buf);</span><br><span>                  /* patch out GPRS indicator from binary if PCU is not connected; will be enabled</span><br><span>                      * after PCU connects */</span><br><span>                     regenerate_si3_restoctets(bts);</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bts/+/14817">change 14817</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-bts/+/14817"/><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-Change-Id: Ia962cf21903ba674057cf52746996dd3254bc1c6 </div>
<div style="display:none"> Gerrit-Change-Number: 14817 </div>
<div style="display:none"> Gerrit-PatchSet: 4 </div>
<div style="display:none"> Gerrit-Owner: fixeria <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: fixeria <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: tnt <tnt@246tNt.com> </div>
<div style="display:none"> Gerrit-MessageType: merged </div>