Bug #1759: Wrong calculation of DL window size for DL assignment - OsmoPCU - Open Source Mobile Communications

Bug #1759

Wrong calculation of DL window size for DL assignment

Added by arvind.sirsikar over 4 years ago. Updated 9 months ago.

Status:

Stalled

Priority:

High

Assignee:

Target version:

Start date:

06/28/2016

Due date:

% Done:

30%

Spec Reference:

Description

Hi,

When we create DOWNLINK TBF without existing UPLINK TBF i.e DL assignment on PCH case, The calculation of window size is found to be incorrect.

Description:
4 time slots is configured for DL and osmo-pcu.cfg is configured as window-size 64 104.

When we try to do IPERF in DL direction, PCU allocates window-size as 160 but configures 4 time slots
   as seen by PCU VTY. Below is the result of VTY output.

DL TBFs
   TBF: TFI=0 TLLI=0xf73d2ece (valid) DIR=DL IMSI=901555000001280
   created=1095 state=0000000a 1st_TS=4 1st_cTS=6 ctrl_TS=6 MS_CLASS=0/1
   TS_alloc=4 5 6! 7  CS=MCS-9 WS=160 V(A)=12 V(S)=12 nBSN=138

But we see proper calculation of window-size when DL assignement is done on PACCH. as seen by VTY interface below

DL TBFs
   TBF: TFI=0 TLLI=0xf73d2ece (valid) DIR=DL IMSI=901555000001280
   created=1095 state=0000000a 1st_TS=4 1st_cTS=6 ctrl_TS=6 MS_CLASS=0/1
   TS_alloc=4 5 6! 7  CS=MCS-9 WS=480 V(A)=138 V(S)=139 nBSN=138

Thanks,
Aravind Sirsikar

Related issues

History

#1 Updated by laforge about 3 years ago

#2 Updated by laforge about 3 years ago

Assignee changed from arvind.sirsikar to sysmocom

#3 Updated by laforge about 3 years ago

Assignee changed from sysmocom to msuraev
Priority changed from High to Normal

#4 Updated by laforge about 3 years ago

Priority changed from Normal to High

#5 Updated by msuraev almost 3 years ago

I don't see explicit tests for it - only as part of larger TBF tests. So the first step would to add such a test.

#6 Updated by msuraev almost 3 years ago

Project changed from Cellular Network Infrastructure to OsmoPCU

#7 Updated by msuraev almost 3 years ago

Related to Feature #1533: Separate the window handling from the TBF more clearly added

#8 Updated by msuraev almost 3 years ago

Status changed from New to In Progress
% Done changed from 0 to 20

Related gerrit 5355, 5336-5341 are under review. Remaining parts: clearly differenciate between CCCH and PACCH assignment, figure out whyit makes a difference to window size calculations and test with different window sizes settings and available channels.

#9 Updated by msuraev almost 3 years ago

Status changed from In Progress to Stalled

#10 Updated by msuraev almost 3 years ago

Status changed from Stalled to In Progress
% Done changed from 20 to 30

Patches were merged. Additional gerrit 5786 was sent for review.

#11 Updated by msuraev almost 3 years ago

5786 is merged, needs additional testing.

#12 Updated by msuraev over 2 years ago

The window size parameter in vty defined as follows:

window-size <0-1024> [<0-256>]
Window size configuration (b + N_PDCH * f)
Base value (b)
Factor for number of PDCH (f)

So the configuration window-size 64 104 means that WS(1TS) = 64 + 1 * 104 = 168 and WS(4TS) = 64 + 4 * 104 = 480. By default f = 0.

#13 Updated by msuraev over 2 years ago

Related to Bug #1524: PACCH on the wrong timeslot added

#14 Updated by msuraev over 2 years ago

In addition to difference between PACCH and CCCH, the TBF can be assigned via CCCH but configrmed via PACCH (see rcv_control_ack() function). This and the related #1524 makes it harder to reproduce this reliably.

Related gerrit 6239 is under review.

#15 Updated by laforge over 2 years ago

Hi Max,
On Mon, Feb 05, 2018 at 03:45:50PM +0000, msuraev [REDMINE] wrote:

In addition to difference between PACCH and CCCH, the TBF can be assigned via CCCH but configrmed via PACCH (see rcv_control_ack() function). This and the related #1524 makes it harder to reproduce this reliably.

would it be possible to disable certain variants as a temporary hack?