Project

General

Profile

Feature #1902

unify/refactor dynamic timeslot code

Added by neels over 1 year ago. Updated 2 months ago.

Status:
New
Priority:
Normal
Assignee:
sysmocom
Category:
A-bis RSL
Target version:
-
Start date:
01/05/2017
Due date:
% Done:

0%

Estimated time:
Spec Reference:

Description

The two kinds of dynamic timeslots could use the same data structures.

The IPAC style TCH/F_PDCH was implemented first, using ts->flags.
The Osmo style TCH/F_TCH/H_PDCH needed more detailed state, so ts->dyn was added.

The TCH/F_PDCH could also use ts->dyn.

Also, we had neverending bugs of places that forgot to check for dyn ts.
The code would be less fragile if we kept the actual current mode the
timeslot is in in the actual ts->pchan field, where all the code expects it.
Currently we keep "TCH_F_TCH_H_PDCH" in pchan and store the current real
mode in dyn->pchan_is. So all code needs to be aware of dynamic channels
and has to add checks to find out the current mode.

I assume it would be nicer if the fact that a channel is dynamic is kept
in ts->dyn, and the ts->pchan can be evaluated like with static channels.
ts->pchan should probably also reflect switchover as an invalid pchan.
This way all code can just directly use ts->pchan, and the places that
are concerned with switchover can look it up in ts->dyn.*.


Related issues

Related to OsmoBTS - Bug #3131: dynamic timeslots not tested by BTS_Tests.ttcnResolved2018-04-04

Related to OsmoBSC - Bug #3099: dynamic timeslots not tested by BSC_Tests.ttcnResolved2018-03-22

History

#1 Updated by neels over 1 year ago

  • Description updated (diff)
  • Priority changed from Normal to Low

(accidentally hit enter when there was no description yet, adding description now)

#2 Updated by neels over 1 year ago

  • Assignee set to Osmocom Developers

#3 Updated by laforge 6 months ago

  • Project changed from OpenBSC to OsmoBSC

#4 Updated by laforge 5 months ago

  • Assignee changed from Osmocom Developers to sysmocom
  • Priority changed from Low to Normal

I think this should be done together with the introduction of osmo_fsm for handling a lchan, i.e. one FSM per lchan whihc then tracks all of it's activation/deactivation state.

#5 Updated by laforge 2 months ago

  • Related to Bug #3131: dynamic timeslots not tested by BTS_Tests.ttcn added

#6 Updated by laforge 2 months ago

  • Related to Bug #3099: dynamic timeslots not tested by BSC_Tests.ttcn added

#7 Updated by laforge 2 months ago

I think this should wait until we have proper test cases, e.g. #3099

#8 Updated by laforge 2 months ago

  • Category set to A-bis RSL

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 48.8 MB)