Project

General

Profile

Feature #1539

Remove knowledge of the TBF from the PDCH object inside PCU

Added by zecke almost 4 years ago. Updated 5 days 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 structureStalled02/22/2016

History

#1 Updated by laforge almost 4 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

#2 Updated by msuraev over 2 years ago

  • Status changed from New to In Progress

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

#3 Updated by msuraev over 2 years ago

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

#4 Updated by msuraev over 2 years ago

  • % Done changed from 10 to 20

Gerrit 3155, 3156 merged.

#5 Updated by msuraev about 2 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.

#6 Updated by msuraev almost 2 years ago

  • Status changed from In Progress to Stalled

#7 Updated by msuraev almost 2 years ago

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

#8 Updated by msuraev almost 2 years ago

  • % Done changed from 30 to 50

Related gerrit 6609 is sent for review.

#9 Updated by msuraev almost 2 years ago

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

#10 Updated by laforge almost 2 years ago

  • Assignee deleted (msuraev)

#11 Updated by pespin 6 days ago

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

#12 Updated by pespin 6 days 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 ?

#13 Updated by laforge 5 days ago

Feel free to close as you see fit

#14 Updated by pespin 5 days ago

  • Status changed from Stalled to Closed

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 48.8 MB)