Update notes for submitted changes

* fix typo 's/dowlink/downlink/g'
* tbf: Move T3193 to tbf_state FSM
* Simplify tbf::set_polling()
* Move tbf ul_ack_state to osmocom FSM
* tbf.h: Improve documentation on several flags
* tbf_dl: Clarify requirements for DL ACK/NACK
* Move timer X2002 to tbf_fsm
* tbf: Use type bool for upgrade_to_multislot
* Fix typos in comments documenting fsm st chg macro
* tbf: Merge handle_ack_nack() into rcvd_dl_ack()
* Get rid of lots of code only used by tests
* Move timer X2001 to tbf_fsm
* tbf: Drop unuseful flag GPRS_RLCMAC_FLAG_UL_DATA
* tbf: Drop unuseful flag GPRS_RLCMAC_FLAG_TO_DL_ASS
* replace dl_ass_state with osmocom FSM
* tbf: Drop unuseful flag GPRS_RLCMAC_FLAG_TO_UL_ASS
* tbf: Reimplement rlcmac_diag() and make it available from C
* Replace ul_ass_state with osmocom FSM
* tbf_free: Get rid of uneeded tbf_state transition
* Get rid of tbf_dl:abort()
* Move rate_ctr free to tbf subclass destructor
* Remove duplicate call to gprs_rlcmac_lost_rep
* Drop logging last mas report before freeing TBF
* Put dl_tbf::cleanup into destructor
* Drop duplicate log line
* Move T3169 and T3195 to tbf_fsm
* Move RELEASING tbf_state transition to tbf_fsm
* Move WAIT_RELEASE tbf_state transition to tbf_fsm
* Move FINISHED tbf_state transition to tbf_fsm
* tests: tbf: Fix dl_tbf polled for data without being in FLOW state
* Move FLOW tbf_state transition to tbf_fsm.
* Move NULL and ASSIGN tbf_state transition to tbf_fsm
* Revert "Revert "Stop abusing T3169""
* Revert "fix: handle NULL return of as_dl_tbf() and as_ul_tbf()"
* Revert "coverity: fix null deref from recent UL TBF leak fix"
This commit is contained in:
Gerrit Code Review 2021-08-23 16:27:59 +00:00
parent d6ed4c3940
commit faf0a88627
35 changed files with 275 additions and 0 deletions

View File

@ -0,0 +1,8 @@
Verified+1: Jenkins Builder
Code-Review+2: pespin <pespin@sysmocom.de>
Code-Review+1: osmith <osmith@sysmocom.de>
Submitted-by: pespin <pespin@sysmocom.de>
Submitted-at: Mon, 23 Aug 2021 16:27:59 +0000
Reviewed-on: https://gerrit.osmocom.org/c/osmo-pcu/+/25053
Project: osmo-pcu
Branch: refs/heads/master

View File

@ -0,0 +1,8 @@
Verified+1: Jenkins Builder
Code-Review+2: pespin <pespin@sysmocom.de>
Code-Review+1: osmith <osmith@sysmocom.de>
Submitted-by: pespin <pespin@sysmocom.de>
Submitted-at: Mon, 23 Aug 2021 16:27:59 +0000
Reviewed-on: https://gerrit.osmocom.org/c/osmo-pcu/+/25054
Project: osmo-pcu
Branch: refs/heads/master

View File

@ -0,0 +1,8 @@
Verified+1: Jenkins Builder
Code-Review+2: pespin <pespin@sysmocom.de>
Code-Review+1: osmith <osmith@sysmocom.de>
Submitted-by: pespin <pespin@sysmocom.de>
Submitted-at: Mon, 23 Aug 2021 16:27:59 +0000
Reviewed-on: https://gerrit.osmocom.org/c/osmo-pcu/+/25104
Project: osmo-pcu
Branch: refs/heads/master

View File

@ -0,0 +1,8 @@
Verified+1: Jenkins Builder
Code-Review+1: osmith <osmith@sysmocom.de>
Code-Review+2: laforge <laforge@osmocom.org>
Submitted-by: pespin <pespin@sysmocom.de>
Submitted-at: Mon, 23 Aug 2021 16:27:59 +0000
Reviewed-on: https://gerrit.osmocom.org/c/osmo-pcu/+/25011
Project: osmo-pcu
Branch: refs/heads/master

View File

@ -0,0 +1,8 @@
Verified+1: Jenkins Builder
Code-Review+2: pespin <pespin@sysmocom.de>
Code-Review+1: osmith <osmith@sysmocom.de>
Submitted-by: pespin <pespin@sysmocom.de>
Submitted-at: Mon, 23 Aug 2021 16:27:59 +0000
Reviewed-on: https://gerrit.osmocom.org/c/osmo-pcu/+/25106
Project: osmo-pcu
Branch: refs/heads/master

View File

@ -0,0 +1,8 @@
Verified+1: Jenkins Builder
Code-Review+2: pespin <pespin@sysmocom.de>
Code-Review+1: osmith <osmith@sysmocom.de>
Submitted-by: pespin <pespin@sysmocom.de>
Submitted-at: Mon, 23 Aug 2021 16:27:59 +0000
Reviewed-on: https://gerrit.osmocom.org/c/osmo-pcu/+/25105
Project: osmo-pcu
Branch: refs/heads/master

View File

@ -0,0 +1,8 @@
Verified+1: Jenkins Builder
Code-Review+2: pespin <pespin@sysmocom.de>
Code-Review+1: osmith <osmith@sysmocom.de>
Submitted-by: pespin <pespin@sysmocom.de>
Submitted-at: Mon, 23 Aug 2021 16:27:59 +0000
Reviewed-on: https://gerrit.osmocom.org/c/osmo-pcu/+/25059
Project: osmo-pcu
Branch: refs/heads/master

View File

@ -0,0 +1,7 @@
Verified+1: Jenkins Builder
Code-Review+2: pespin <pespin@sysmocom.de>
Submitted-by: pespin <pespin@sysmocom.de>
Submitted-at: Mon, 23 Aug 2021 16:27:59 +0000
Reviewed-on: https://gerrit.osmocom.org/c/osmo-pcu/+/25060
Project: osmo-pcu
Branch: refs/heads/master

View File

@ -0,0 +1,8 @@
Verified+1: Jenkins Builder
Code-Review+2: pespin <pespin@sysmocom.de>
Code-Review+1: osmith <osmith@sysmocom.de>
Submitted-by: pespin <pespin@sysmocom.de>
Submitted-at: Mon, 23 Aug 2021 16:27:59 +0000
Reviewed-on: https://gerrit.osmocom.org/c/osmo-pcu/+/25047
Project: osmo-pcu
Branch: refs/heads/master

View File

@ -0,0 +1,8 @@
Verified+1: Jenkins Builder
Code-Review+2: pespin <pespin@sysmocom.de>
Code-Review+1: osmith <osmith@sysmocom.de>
Submitted-by: pespin <pespin@sysmocom.de>
Submitted-at: Mon, 23 Aug 2021 16:27:59 +0000
Reviewed-on: https://gerrit.osmocom.org/c/osmo-pcu/+/25061
Project: osmo-pcu
Branch: refs/heads/master

View File

@ -0,0 +1,8 @@
Verified+1: Jenkins Builder
Code-Review+2: pespin <pespin@sysmocom.de>
Code-Review+1: osmith <osmith@sysmocom.de>
Submitted-by: pespin <pespin@sysmocom.de>
Submitted-at: Mon, 23 Aug 2021 16:27:59 +0000
Reviewed-on: https://gerrit.osmocom.org/c/osmo-pcu/+/25095
Project: osmo-pcu
Branch: refs/heads/master

View File

@ -0,0 +1,8 @@
Verified+1: Jenkins Builder
Code-Review+2: pespin <pespin@sysmocom.de>
Code-Review+1: osmith <osmith@sysmocom.de>
Submitted-by: pespin <pespin@sysmocom.de>
Submitted-at: Mon, 23 Aug 2021 16:27:59 +0000
Reviewed-on: https://gerrit.osmocom.org/c/osmo-pcu/+/25050
Project: osmo-pcu
Branch: refs/heads/master

View File

@ -0,0 +1,8 @@
Verified+1: Jenkins Builder
Code-Review+1: osmith <osmith@sysmocom.de>
Code-Review+2: laforge <laforge@osmocom.org>
Submitted-by: pespin <pespin@sysmocom.de>
Submitted-at: Mon, 23 Aug 2021 16:27:59 +0000
Reviewed-on: https://gerrit.osmocom.org/c/osmo-pcu/+/25045
Project: osmo-pcu
Branch: refs/heads/master

View File

@ -0,0 +1,7 @@
Verified+1: Jenkins Builder
Code-Review+2: pespin <pespin@sysmocom.de>
Submitted-by: pespin <pespin@sysmocom.de>
Submitted-at: Mon, 23 Aug 2021 16:27:59 +0000
Reviewed-on: https://gerrit.osmocom.org/c/osmo-pcu/+/25056
Project: osmo-pcu
Branch: refs/heads/master

View File

@ -0,0 +1,8 @@
Verified+1: Jenkins Builder
Code-Review+1: osmith <osmith@sysmocom.de>
Code-Review+2: laforge <laforge@osmocom.org>
Submitted-by: pespin <pespin@sysmocom.de>
Submitted-at: Mon, 23 Aug 2021 16:27:59 +0000
Reviewed-on: https://gerrit.osmocom.org/c/osmo-pcu/+/25012
Project: osmo-pcu
Branch: refs/heads/master

View File

@ -0,0 +1,7 @@
Verified+1: Jenkins Builder
Code-Review+2: pespin <pespin@sysmocom.de>
Submitted-by: pespin <pespin@sysmocom.de>
Submitted-at: Mon, 23 Aug 2021 16:27:59 +0000
Reviewed-on: https://gerrit.osmocom.org/c/osmo-pcu/+/25223
Project: osmo-pcu
Branch: refs/heads/master

View File

@ -0,0 +1,8 @@
Verified+1: Jenkins Builder
Code-Review+2: pespin <pespin@sysmocom.de>
Code-Review+1: osmith <osmith@sysmocom.de>
Submitted-by: pespin <pespin@sysmocom.de>
Submitted-at: Mon, 23 Aug 2021 16:27:59 +0000
Reviewed-on: https://gerrit.osmocom.org/c/osmo-pcu/+/25051
Project: osmo-pcu
Branch: refs/heads/master

View File

@ -0,0 +1,8 @@
Verified+1: Jenkins Builder
Code-Review+2: pespin <pespin@sysmocom.de>
Code-Review+1: osmith <osmith@sysmocom.de>
Submitted-by: pespin <pespin@sysmocom.de>
Submitted-at: Mon, 23 Aug 2021 16:27:59 +0000
Reviewed-on: https://gerrit.osmocom.org/c/osmo-pcu/+/25102
Project: osmo-pcu
Branch: refs/heads/master

View File

@ -0,0 +1,8 @@
Verified+1: Jenkins Builder
Code-Review+2: pespin <pespin@sysmocom.de>
Code-Review+1: osmith <osmith@sysmocom.de>
Submitted-by: pespin <pespin@sysmocom.de>
Submitted-at: Mon, 23 Aug 2021 16:27:59 +0000
Reviewed-on: https://gerrit.osmocom.org/c/osmo-pcu/+/25092
Project: osmo-pcu
Branch: refs/heads/master

View File

@ -0,0 +1,8 @@
Verified+1: Jenkins Builder
Code-Review+2: pespin <pespin@sysmocom.de>
Code-Review+1: osmith <osmith@sysmocom.de>
Submitted-by: pespin <pespin@sysmocom.de>
Submitted-at: Mon, 23 Aug 2021 16:27:59 +0000
Reviewed-on: https://gerrit.osmocom.org/c/osmo-pcu/+/25052
Project: osmo-pcu
Branch: refs/heads/master

View File

@ -0,0 +1,8 @@
Verified+1: Jenkins Builder
Code-Review+2: pespin <pespin@sysmocom.de>
Code-Review+1: osmith <osmith@sysmocom.de>
Submitted-by: pespin <pespin@sysmocom.de>
Submitted-at: Mon, 23 Aug 2021 16:27:59 +0000
Reviewed-on: https://gerrit.osmocom.org/c/osmo-pcu/+/25111
Project: osmo-pcu
Branch: refs/heads/master

View File

@ -0,0 +1,8 @@
Verified+1: Jenkins Builder
Code-Review+2: pespin <pespin@sysmocom.de>
Code-Review+1: osmith <osmith@sysmocom.de>
Submitted-by: pespin <pespin@sysmocom.de>
Submitted-at: Mon, 23 Aug 2021 16:27:59 +0000
Reviewed-on: https://gerrit.osmocom.org/c/osmo-pcu/+/25058
Project: osmo-pcu
Branch: refs/heads/master

View File

@ -0,0 +1,9 @@
Verified+1: Jenkins Builder
Code-Review+1: osmith <osmith@sysmocom.de>
Code-Review+1: dexter <pmaier@sysmocom.de>
Code-Review+2: laforge <laforge@osmocom.org>
Submitted-by: pespin <pespin@sysmocom.de>
Submitted-at: Mon, 23 Aug 2021 16:27:59 +0000
Reviewed-on: https://gerrit.osmocom.org/c/osmo-pcu/+/25093
Project: osmo-pcu
Branch: refs/heads/master

View File

@ -0,0 +1,7 @@
Verified+1: Jenkins Builder
Code-Review+2: pespin <pespin@sysmocom.de>
Submitted-by: pespin <pespin@sysmocom.de>
Submitted-at: Mon, 23 Aug 2021 16:27:59 +0000
Reviewed-on: https://gerrit.osmocom.org/c/osmo-pcu/+/25055
Project: osmo-pcu
Branch: refs/heads/master

View File

@ -0,0 +1,7 @@
Verified+1: Jenkins Builder
Code-Review+2: pespin <pespin@sysmocom.de>
Submitted-by: pespin <pespin@sysmocom.de>
Submitted-at: Mon, 23 Aug 2021 16:27:59 +0000
Reviewed-on: https://gerrit.osmocom.org/c/osmo-pcu/+/25222
Project: osmo-pcu
Branch: refs/heads/master

View File

@ -0,0 +1,7 @@
Verified+1: Jenkins Builder
Code-Review+2: pespin <pespin@sysmocom.de>
Submitted-by: pespin <pespin@sysmocom.de>
Submitted-at: Mon, 23 Aug 2021 16:27:59 +0000
Reviewed-on: https://gerrit.osmocom.org/c/osmo-pcu/+/25221
Project: osmo-pcu
Branch: refs/heads/master

View File

@ -0,0 +1,8 @@
Verified+1: Jenkins Builder
Code-Review+2: pespin <pespin@sysmocom.de>
Code-Review+1: osmith <osmith@sysmocom.de>
Submitted-by: pespin <pespin@sysmocom.de>
Submitted-at: Mon, 23 Aug 2021 16:27:59 +0000
Reviewed-on: https://gerrit.osmocom.org/c/osmo-pcu/+/25034
Project: osmo-pcu
Branch: refs/heads/master

View File

@ -0,0 +1,8 @@
Verified+1: Jenkins Builder
Code-Review+2: pespin <pespin@sysmocom.de>
Code-Review+1: osmith <osmith@sysmocom.de>
Submitted-by: pespin <pespin@sysmocom.de>
Submitted-at: Mon, 23 Aug 2021 16:27:59 +0000
Reviewed-on: https://gerrit.osmocom.org/c/osmo-pcu/+/25090
Project: osmo-pcu
Branch: refs/heads/master

View File

@ -0,0 +1,8 @@
Verified+1: Jenkins Builder
Code-Review+2: pespin <pespin@sysmocom.de>
Code-Review+1: osmith <osmith@sysmocom.de>
Submitted-by: pespin <pespin@sysmocom.de>
Submitted-at: Mon, 23 Aug 2021 16:27:59 +0000
Reviewed-on: https://gerrit.osmocom.org/c/osmo-pcu/+/25048
Project: osmo-pcu
Branch: refs/heads/master

View File

@ -0,0 +1,8 @@
Verified+1: Jenkins Builder
Code-Review+2: dexter <pmaier@sysmocom.de>
Code-Review+1: osmith <osmith@sysmocom.de>
Submitted-by: pespin <pespin@sysmocom.de>
Submitted-at: Mon, 23 Aug 2021 16:27:59 +0000
Reviewed-on: https://gerrit.osmocom.org/c/osmo-pcu/+/25113
Project: osmo-pcu
Branch: refs/heads/master

View File

@ -0,0 +1,8 @@
Verified+1: Jenkins Builder
Code-Review+2: pespin <pespin@sysmocom.de>
Code-Review+1: osmith <osmith@sysmocom.de>
Submitted-by: pespin <pespin@sysmocom.de>
Submitted-at: Mon, 23 Aug 2021 16:27:59 +0000
Reviewed-on: https://gerrit.osmocom.org/c/osmo-pcu/+/25108
Project: osmo-pcu
Branch: refs/heads/master

View File

@ -0,0 +1,8 @@
Verified+1: Jenkins Builder
Code-Review+2: pespin <pespin@sysmocom.de>
Code-Review+1: osmith <osmith@sysmocom.de>
Submitted-by: pespin <pespin@sysmocom.de>
Submitted-at: Mon, 23 Aug 2021 16:27:59 +0000
Reviewed-on: https://gerrit.osmocom.org/c/osmo-pcu/+/25046
Project: osmo-pcu
Branch: refs/heads/master

View File

@ -0,0 +1,8 @@
Verified+1: Jenkins Builder
Code-Review+2: pespin <pespin@sysmocom.de>
Code-Review+1: osmith <osmith@sysmocom.de>
Submitted-by: pespin <pespin@sysmocom.de>
Submitted-at: Mon, 23 Aug 2021 16:27:59 +0000
Reviewed-on: https://gerrit.osmocom.org/c/osmo-pcu/+/25049
Project: osmo-pcu
Branch: refs/heads/master

View File

@ -0,0 +1,8 @@
Verified+1: Jenkins Builder
Code-Review+2: pespin <pespin@sysmocom.de>
Code-Review+1: osmith <osmith@sysmocom.de>
Submitted-by: pespin <pespin@sysmocom.de>
Submitted-at: Mon, 23 Aug 2021 16:27:59 +0000
Reviewed-on: https://gerrit.osmocom.org/c/osmo-pcu/+/25112
Project: osmo-pcu
Branch: refs/heads/master

View File

@ -0,0 +1,8 @@
Verified+1: Jenkins Builder
Code-Review+2: pespin <pespin@sysmocom.de>
Code-Review+1: osmith <osmith@sysmocom.de>
Submitted-by: pespin <pespin@sysmocom.de>
Submitted-at: Mon, 23 Aug 2021 16:27:59 +0000
Reviewed-on: https://gerrit.osmocom.org/c/osmo-pcu/+/25094
Project: osmo-pcu
Branch: refs/heads/master