Project

General

Profile

Actions

Feature #1539

closed

Remove knowledge of the TBF from the PDCH object inside PCU

Added by zecke about 8 years ago. Updated about 4 years ago.

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

50%

Spec Reference:

Description

This can be reached by doing many small patches. E.g.:

gprs_rlcmac_pdch::rcv_control_ack plays with the internal state of the TBF.
gprs_rlcmac_pdch::rcv_control_dl_ack_nack is a similar case


Related issues

Related to OsmoPCU - Feature #1541: Move TBF list from BTS to the TRX structureClosed02/22/2016

Actions
Actions #1

Updated by laforge about 8 years ago

  • Subject changed from Remove knowledge of the TBF from the PDCH to Remove knowledge of the TBF from the PDCH object inside PCU
Actions #2

Updated by msuraev almost 7 years ago

  • Status changed from New to In Progress

Gerrit 3154 has been merged, 3155, 3156 are under review.

Actions #3

Updated by msuraev over 6 years ago

  • Status changed from In Progress to Stalled
  • % Done changed from 0 to 10
Actions #4

Updated by msuraev over 6 years ago

  • % Done changed from 10 to 20

Gerrit 3155, 3156 merged.

Actions #5

Updated by msuraev about 6 years ago

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

Related gerrit 5862-5864, 6053, 6100, 6195, 6193 were sent for review.

Actions #6

Updated by msuraev about 6 years ago

  • Status changed from In Progress to Stalled
Actions #7

Updated by msuraev about 6 years ago

Above-mentioned patches were merged, PDCH now lives in separate files.

Actions #8

Updated by msuraev about 6 years ago

  • % Done changed from 30 to 50

Related gerrit 6609 is sent for review.

Actions #9

Updated by msuraev about 6 years ago

Gerrit 6609 is merged, we should double-check if code in pdch.cpp still access TBF internals.

Actions #10

Updated by laforge about 6 years ago

  • Assignee deleted (msuraev)
Actions #11

Updated by pespin about 4 years ago

  • Related to Feature #1541: Move TBF list from BTS to the TRX structure added
Actions #12

Updated by pespin about 4 years ago

A lot of cleanup was done already during patches explained above, and the aim or final state of this ticket is not really clear, so I vote for closing it unless someone defines more specific targets. zecke ? laforge ?

Actions #13

Updated by laforge about 4 years ago

Feel free to close as you see fit

Actions #14

Updated by pespin about 4 years ago

  • Status changed from Stalled to Closed
Actions

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 48.8 MB)