https://projects.osmocom.org/https://projects.osmocom.org/favicon.ico?16647414092016-02-22T21:07:37ZOpen Source Mobile CommunicationsOsmoPCU - Feature #1533: Separate the window handling from the TBF more clearlyhttps://projects.osmocom.org/issues/1533?journal_id=10172016-02-22T21:07:37Zzecke
<ul></ul><p>From Jacob:</p>
<p>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.<br />The RBB layout is fully abstracted in decoding.cpp now, only a bitvec and the first BSN are passed around.</p> OsmoPCU - Feature #1533: Separate the window handling from the TBF more clearlyhttps://projects.osmocom.org/issues/1533?journal_id=10232016-02-22T21:32:14Zzecke
<ul><li><strong>Priority</strong> changed from <i>Normal</i> to <i>Low</i></li></ul> OsmoPCU - Feature #1533: Separate the window handling from the TBF more clearlyhttps://projects.osmocom.org/issues/1533?journal_id=44692017-07-10T12:10:42Zmsuraev
<ul><li><strong>Status</strong> changed from <i>New</i> to <i>Feedback</i></li></ul><p>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.</p> OsmoPCU - Feature #1533: Separate the window handling from the TBF more clearlyhttps://projects.osmocom.org/issues/1533?journal_id=46672017-07-21T14:46:59Zmsuraev
<ul><li><strong>Assignee</strong> changed from <i>msuraev</i> to <i>zecke</i></li></ul> OsmoPCU - Feature #1533: Separate the window handling from the TBF more clearlyhttps://projects.osmocom.org/issues/1533?journal_id=67792017-12-13T13:49:14Zmsuraev
<ul><li><strong>Related to</strong> <i><a class="issue tracker-1 status-7 priority-3 priority-high3" href="/issues/1759">Bug #1759</a>: Wrong calculation of DL window size for DL assignment</i> added</li></ul>