Project

General

Profile

Actions

Feature #1533

open

Separate the window handling from the TBF more clearly

Added by zecke about 8 years ago. Updated over 6 years ago.

Status:
Feedback
Priority:
Low
Assignee:
Target version:
-
Start date:
02/22/2016
Due date:
% Done:

0%

Spec Reference:

Description

**The window handling can be more clearly separated from the TBF. This includes clean-ups to the ::assemble_forward_llc. The general multitude of calling dir.dl.window.m_v_b anddir.dl.window.increment_send.


Related issues

Related to OsmoPCU - Bug #1759: Wrong calculation of DL window size for DL assignmentStalled06/28/2016

Actions
Actions #1

Updated by zecke about 8 years ago

From Jacob:

The situation has much improved with the edge related update of the window handling, but the update functions in the TBF and RLC window class haven't changed really.
The RBB layout is fully abstracted in decoding.cpp now, only a bitvec and the first BSN are passed around.

Actions #2

Updated by zecke about 8 years ago

  • Priority changed from Normal to Low
Actions #3

Updated by msuraev over 6 years ago

  • Status changed from New to Feedback

I'm not quite sure what else can be done in here. The "window.m_v_b" is used in tbf_dl.cpp only and half of the time is in analyse_errors() functions to obtain stats data, increment_send() is used only once. It's also unclear which are the "update functions in the TBF" referred to in comments. I think we should either have much clearer description with actionable steps or close this altogether.

Actions #4

Updated by msuraev over 6 years ago

  • Assignee changed from msuraev to zecke
Actions #5

Updated by msuraev over 6 years ago

  • Related to Bug #1759: Wrong calculation of DL window size for DL assignment added
Actions

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 48.8 MB)