https://projects.osmocom.org/https://projects.osmocom.org/favicon.ico?16647414092019-08-29T11:54:33ZOpen Source Mobile CommunicationsOsmoBTS - Bug #4179: Race condition: OsmoBTS sends empty INFO_ind to PCU socket, if not all SI arrived from BSC via RSLhttps://projects.osmocom.org/issues/4179?journal_id=157382019-08-29T11:54:33Zosmith
<ul><li><strong>Related to</strong> <i><a class="issue tracker-1 status-1 priority-2 priority-default" href="/issues/3000">Bug #3000</a>: OsmoBSC issues "RSL Connect" command too early -> race conditions</i> added</li></ul> OsmoBTS - Bug #4179: Race condition: OsmoBTS sends empty INFO_ind to PCU socket, if not all SI arrived from BSC via RSLhttps://projects.osmocom.org/issues/4179?journal_id=157402019-08-29T11:54:39Zosmith
<ul><li><strong>Related to</strong> <i><a class="issue tracker-1 status-3 priority-2 priority-default closed" href="/issues/3854">Bug #3854</a>: OsmoPCU uses wrong CellID in BSSGP</i> added</li></ul> OsmoBTS - Bug #4179: Race condition: OsmoBTS sends empty INFO_ind to PCU socket, if not all SI arrived from BSC via RSLhttps://projects.osmocom.org/issues/4179?journal_id=157412019-08-29T12:38:19Zosmith
<ul></ul><ul>
<li>Sleep patch: <a class="external" href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/15327">https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/15327</a></li>
<li>Test that reproduces the problem (with reverted sleep patch): <a class="external" href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/15328">https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/15328</a></li>
</ul> OsmoBTS - Bug #4179: Race condition: OsmoBTS sends empty INFO_ind to PCU socket, if not all SI arrived from BSC via RSLhttps://projects.osmocom.org/issues/4179?journal_id=158422019-09-04T09:29:14Zosmith
<ul></ul><p>As noted in the meeting yesterday, a few ttcn3-bts-test and ttcn3-bts-test-latest tests started to fail lately:</p>
<ul>
<li>TC_pcu_socket_connect_multi (Unexpected unix domain connect result)</li>
<li>TC_pcu_socket_connect_si3gprs (SI3 indicates no GPRS despite PCU socket connected)</li>
<li>TC_si_sched_13_2bis_2ter_2quater (Error: Insufficient SI in array)</li>
</ul>
<p>(The new test TC_pcu_socket_verify_info_ind is only failing in latest, and this is expected.)</p>
<p>I found, that they started to fail since the sleep commit mentioned above:<br /><a class="external" href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/15327">https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/15327</a> ("bts: f_init_pcu: sleep before connect")</p>
<p>As I ran TC_si_sched_13_2bis_2ter_2quater, I found several log messages like this one in osmo-bts.log:<br /><pre>
20190904083328552 DL1P <0007> sysinfo.c:160 PH-RTS-IND: Unable to determine actual BS_AG_BLKS_RES value as SI3 is not available yet, fallback to 1
20190904083328552 DL1P <0007> sysinfo.c:160 GSMTAP: Unable to determine actual BS_AG_BLKS_RES value as SI3 is not available yet, fallback to 1
</pre></p>
<p>I think, that this is another race condition bug in the testsuite or OsmoBTS: the additional sleep introduced by the patch should not make anything fail, if everything correctly waited for messages indicating that information is available.</p>
<p>So... in order not investing too much time into this (I'm really getting side-tracked from <a class="issue tracker-1 status-5 priority-3 priority-high3 closed" title="Bug: Missing PCU_Tests.ttcn UL TBF tests (Closed)" href="https://projects.osmocom.org/issues/3925">#3925</a>, which I'm originally working on), I'm going to revert that sleep commit for now. This makes TC_pcu_socket_verify_info_ind fail in master, until this issue (<a class="issue tracker-1 status-3 priority-2 priority-default closed" title="Bug: Race condition: OsmoBTS sends empty INFO_ind to PCU socket, if not all SI arrived from BSC via RSL (Resolved)" href="https://projects.osmocom.org/issues/4179">#4179</a>) is resolved.</p>
<p>Revert: <a class="external" href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/15401">https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/15401</a></p> OsmoBTS - Bug #4179: Race condition: OsmoBTS sends empty INFO_ind to PCU socket, if not all SI arrived from BSC via RSLhttps://projects.osmocom.org/issues/4179?journal_id=264362023-03-21T00:37:42Zfixeria
<ul><li><strong>Status</strong> changed from <i>New</i> to <i>Feedback</i></li></ul><p>osmith wrote in <a href="#note-4">#note-4</a>:</p>
<blockquote>
<p>So... in order not investing too much time into this (I'm really getting side-tracked from <a class="issue tracker-1 status-5 priority-3 priority-high3 closed" title="Bug: Missing PCU_Tests.ttcn UL TBF tests (Closed)" href="https://projects.osmocom.org/issues/3925">#3925</a>, which I'm originally working on), I'm going to revert that sleep commit for now. This makes TC_pcu_socket_verify_info_ind fail in master, until this issue (<a class="issue tracker-1 status-3 priority-2 priority-default closed" title="Bug: Race condition: OsmoBTS sends empty INFO_ind to PCU socket, if not all SI arrived from BSC via RSL (Resolved)" href="https://projects.osmocom.org/issues/4179">#4179</a>) is resolved.</p>
</blockquote>
<p>AFAICS, <code>TC_pcu_socket_verify_info_ind</code> is passing. <a class="user active" href="https://projects.osmocom.org/users/301771">osmith</a> does this mean the problem is resolved and this ticket can be closed?</p> OsmoBTS - Bug #4179: Race condition: OsmoBTS sends empty INFO_ind to PCU socket, if not all SI arrived from BSC via RSLhttps://projects.osmocom.org/issues/4179?journal_id=264372023-03-21T10:37:51Zosmith
<ul><li><strong>Status</strong> changed from <i>Feedback</i> to <i>Resolved</i></li><li><strong>% Done</strong> changed from <i>0</i> to <i>100</i></li></ul><p>Yes, thanks!</p>