Project

General

Profile

Actions

Bug #2847

closed

OsmoMSC doesn't support all of TS 08.08 CellID formats

Added by laforge about 6 years ago. Updated about 6 years ago.

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

0%

Resolution:
Spec Reference:

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) */

Actions #1

Updated by stsp about 6 years ago

  • Status changed from New to In Progress
Actions #2

Updated by stsp about 6 years ago

  • Status changed from In Progress to Resolved

Above changes have been merged.

Actions #3

Updated by stsp about 6 years 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.

Actions #4

Updated by stsp about 6 years ago

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

Actions #5

Updated by stsp about 6 years 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

Actions #6

Updated by stsp about 6 years ago

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

I am also planning to write more tests.

Actions #7

Updated by stsp about 6 years 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.

Actions #8

Updated by stsp about 6 years ago

Actions #9

Updated by stsp about 6 years ago

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

Actions #10

Updated by stsp about 6 years ago

Actions #11

Updated by stsp about 6 years 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.

Actions

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 48.8 MB)