Project

General

Profile

Bug #2847

OsmoMSC doesn't support all of TS 08.08 CellID formats

Added by laforge 4 months ago. Updated 2 months ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
A interface (AoIP)
Target version:
-
Start date:
01/20/2018
Due date:
% Done:

0%

Resolution:

Description

See a_iface_bssmap.c:

»·······/* Parse Cell ID element */
»·······/* FIXME: Encapsulate this in a parser/generator function inside
»······· * libosmocore, add support for all specified cell identification
»······· * discriminators (see 3GPP ts 3.2.2.17 Cell Identifier) */

History

#1 Updated by stsp 3 months ago

  • Status changed from New to In Progress

#2 Updated by stsp 2 months ago

  • Status changed from In Progress to Resolved

Above changes have been merged.

#3 Updated by stsp 2 months ago

  • Status changed from Resolved to In Progress

Re-open. I forgot that osmo-msc still needs to be adapted to make use of the new APIs.

#4 Updated by stsp 2 months ago

osmo-msc starts using the new API with https://gerrit.osmocom.org/#/c/7281/

#5 Updated by stsp 2 months ago

A few more related changes:

Follow-up fix in libosmocore: https://gerrit.osmocom.org/7288

Fix for ttcn3 paging tests: https://gerrit.osmocom.org/7287

Replace the old parsing code in osmo-bsc paging: https://gerrit.osmocom.org/7289

#6 Updated by stsp 2 months ago

This needed another follow-up fix: https://gerrit.osmocom.org/#/c/7307/

I am also planning to write more tests.

#7 Updated by stsp 2 months ago

The next test I wrote uncovered a problem with 3-digit MNCs, support for which was added very recently.

The patch at https://gerrit.osmocom.org/#/c/7309/ allows us to decode 3-digit MNCs correctly when they occur in cell ID lists.

#8 Updated by stsp 2 months ago

#9 Updated by stsp 2 months ago

Writing packet parsers is hard: https://gerrit.osmocom.org/7318

#10 Updated by stsp 2 months ago

#11 Updated by stsp 2 months ago

  • Status changed from In Progress to Resolved

All above changes have been merged.
Closing this now that encoding/decoding works and is tested properly for all cell identifier types where it matters.

Also available in: Atom PDF