https://osmocom.org/https://osmocom.org/favicon.ico?16647414092019-03-21T18:48:11ZOpen Source Mobile CommunicationsOsmoPCU - Bug #3854: OsmoPCU uses wrong CellID in BSSGPhttps://osmocom.org/issues/3854?journal_id=136152019-03-21T18:48:11Zlaforge
<ul><li><strong>Status</strong> changed from <i>New</i> to <i>In Progress</i></li></ul> OsmoPCU - Bug #3854: OsmoPCU uses wrong CellID in BSSGPhttps://osmocom.org/issues/3854?journal_id=136162019-03-21T19:08:19Zlaforge
<ul><li><strong>% Done</strong> changed from <i>0</i> to <i>80</i></li></ul><p>Fix in <a class="external" href="https://gerrit.osmocom.org/#/c/osmo-pcu/+/13369">https://gerrit.osmocom.org/#/c/osmo-pcu/+/13369</a></p> OsmoPCU - Bug #3854: OsmoPCU uses wrong CellID in BSSGPhttps://osmocom.org/issues/3854?journal_id=136812019-03-28T09:33:38Zlaforge
<ul><li><strong>Status</strong> changed from <i>In Progress</i> to <i>Resolved</i></li></ul> OsmoPCU - Bug #3854: OsmoPCU uses wrong CellID in BSSGPhttps://osmocom.org/issues/3854?journal_id=157002019-08-26T13:34:26Zosmith
<ul><li><strong>Status</strong> changed from <i>Resolved</i> to <i>In Progress</i></li><li><strong>Assignee</strong> changed from <i>laforge</i> to <i>osmith</i></li><li><strong>% Done</strong> changed from <i>80</i> to <i>50</i></li></ul><p>Unfortunately, this issue is appearing again. I can see in both wireshark, and in the ttcn3 testsuite logs, that the CellID is endian-swapped again in BSSGP. That is, with osmo-pcu master, and strangely, after reverting your patch that fixed the issue earlier, the CellID is not swapped anymore.</p>
<p>I read through both your fix and bdc55f (Neels' patch linked above), and it is obvious that the swap was introduced into osmo-pcu in bdc55f and the fix does what it should.</p>
<p>This means, that some other component in the "osmo-bsc -- osmo-bts -- osmo-pcu" chain is mixing up the CellID. Looking at the osmo-pcu.log confirms, that osmo-pcu is already receiving the wrong cell ID from the BTS socket. I've looked at the RSL traces between osmo-bsc and osmo-bts, and there the right CellID is present. So it seems that the error happens in osmo-bts.</p>
<p>Then I searched through the code for mentions of cell_id in osmo-bts, but found only two places, and with debug prints added, they always showed the wrong ID.</p>
<p>My current guess is, that this is a bug in libosmocore, possibly related to the "struct osmo_cell_global_id", that internally combines mcc, mnc, lac and cell_id to pass all of them at once to various functions.</p>
<p>I can't simply try old versions or bisect libosmocore for this problem, because the various components won't build against much older libosmocore versions.</p> OsmoPCU - Bug #3854: OsmoPCU uses wrong CellID in BSSGPhttps://osmocom.org/issues/3854?journal_id=157032019-08-26T13:37:23Zosmith
<ul><li><strong>Related to</strong> <i><a class="issue tracker-1 status-5 priority-3 priority-high3 closed" href="/issues/3925">Bug #3925</a>: Missing PCU_Tests.ttcn UL TBF tests</i> added</li></ul> OsmoPCU - Bug #3854: OsmoPCU uses wrong CellID in BSSGPhttps://osmocom.org/issues/3854?journal_id=157392019-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/4179">Bug #4179</a>: Race condition: OsmoBTS sends empty INFO_ind to PCU socket, if not all SI arrived from BSC via RSL</i> added</li></ul> OsmoPCU - Bug #3854: OsmoPCU uses wrong CellID in BSSGPhttps://osmocom.org/issues/3854?journal_id=157422019-08-29T12:39:09Zosmith
<ul><li><strong>% Done</strong> changed from <i>50</i> to <i>90</i></li></ul><p>Fix and TTCN3 test submitted: <a class="external" href="https://gerrit.osmocom.org/q/topic:cellid-fix">https://gerrit.osmocom.org/q/topic:cellid-fix</a></p> OsmoPCU - Bug #3854: OsmoPCU uses wrong CellID in BSSGPhttps://osmocom.org/issues/3854?journal_id=157562019-08-30T06:14:42Zosmith
<ul><li><strong>Status</strong> changed from <i>In Progress</i> to <i>Resolved</i></li><li><strong>% Done</strong> changed from <i>90</i> to <i>100</i></li></ul>