diff options
author | Jacob Erlbeck <jerlbeck@sysmocom.de> | 2016-01-29 18:51:04 +0100 |
---|---|---|
committer | Jacob Erlbeck <jerlbeck@sysmocom.de> | 2016-02-08 00:45:38 +0100 |
commit | 9876a3ba5dd1f51bb56d340d709b0d2db7891a26 (patch) | |
tree | 0912557294e643a11b5efb8c92630a1ad469e04b /src/tbf.cpp | |
parent | 6b356a58d1269fe4ad449bc868cbc734c6d2a28e (diff) |
tbf: Don't change type from CCCH to PACCH without ack
Currently the CCCH flag is cleared and the PACCH flag is set when a
multislot upgrade is scheduled for a downlink TBF, even if the MS has
never confirmed in any way that the PACCH really exists. This can
happen if the MS did not receive the DL IMM.ASS. Since the CCCH flags
gets cleared in that case, the IMM.ASSS is never retried and all
subsequent PACKET DOWNLINK ASSIGNMENTS will fail.
This commit delays the update of these flags until the MS has
responded with a corresponding CONTROL ACK.
Sponsored-by: On-Waves ehf
Diffstat (limited to 'src/tbf.cpp')
-rw-r--r-- | src/tbf.cpp | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/src/tbf.cpp b/src/tbf.cpp index 04c72724..c07f379e 100644 --- a/src/tbf.cpp +++ b/src/tbf.cpp @@ -851,7 +851,6 @@ void gprs_rlcmac_tbf::handle_timeout() /* keep to flags */ dl_tbf->state_flags &= GPRS_RLCMAC_FLAG_TO_MASK; - dl_tbf->state_flags &= ~(1 << GPRS_RLCMAC_FLAG_CCCH); dl_tbf->update(); |