diff options
author | Daniel Willmann <dwillmann@sysmocom.de> | 2014-05-30 17:58:00 +0200 |
---|---|---|
committer | Holger Hans Peter Freyther <holger@moiji-mobile.com> | 2014-05-30 18:21:00 +0200 |
commit | 73191a443f4be0a6a6ae7058625912150e07e3c7 (patch) | |
tree | fb26e942b8be2a25d6c717b7cd5fffec996baa15 /src/tbf.cpp | |
parent | fc03bbe0780002dd1a55969528c071ce25e8ce6d (diff) |
tbf/bts, encoding: Keep track of WAIT_RELEASE state for DL assignment
The current code does not properly distinguish between DL assignments to
reuse a tbf (after it was put in state WAIT_RELEASE) and DL assignments
for an active tbf to change the allocation of the PDCH timeslots.
This patch introduces a new variable was_releasing which remembers if
trigger_dl_ass() was called with a tbf in state WAIT_RELEASE. In that
case we have to set the CONTROL_ACK field in the download assignment.
This should allow us to send DL assignments to change PDCH TS allocation
of a tbf before we enter FLOW state.
Diffstat (limited to 'src/tbf.cpp')
-rw-r--r-- | src/tbf.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/tbf.cpp b/src/tbf.cpp index b20d0fc7..dd1f0fb3 100644 --- a/src/tbf.cpp +++ b/src/tbf.cpp @@ -1405,6 +1405,8 @@ int gprs_rlcmac_tbf::maybe_start_new_window() /* report all outstanding packets as received */ gprs_rlcmac_received_lost(this, received, 0); + tbf_new_state(this, GPRS_RLCMAC_WAIT_RELEASE); + /* check for LLC PDU in the LLC Queue */ msg = llc_dequeue(gprs_bssgp_pcu_current_bctx()); if (!msg) { @@ -1414,7 +1416,6 @@ int gprs_rlcmac_tbf::maybe_start_new_window() tbf_timer_start(this, 3193, bts_data()->t3193_msec / 1000, (bts_data()->t3193_msec % 1000) * 1000); - tbf_new_state(this, GPRS_RLCMAC_WAIT_RELEASE); return 0; } |