pespin has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-pcu/+/34425?usp=email )


Change subject: tbf_dl_fsm: Fix assert hit due to EV_MAX_N3105 received in 
ST_RELEASING
......................................................................

tbf_dl_fsm: Fix assert hit due to EV_MAX_N3105 received in ST_RELEASING

That code path was incorrectly removed a few commits back, re-add it.

Fixes: 40a297f3b0c8e1670d46a4974750dd3335bc7885
Change-Id: I27e61dc4b3396360327dcd377d171daa30684d23
---
M src/tbf_dl_fsm.c
1 file changed, 19 insertions(+), 0 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-pcu refs/changes/25/34425/1

diff --git a/src/tbf_dl_fsm.c b/src/tbf_dl_fsm.c
index 8d95d14..cf656bc 100644
--- a/src/tbf_dl_fsm.c
+++ b/src/tbf_dl_fsm.c
@@ -305,6 +305,13 @@
                 * already ACKED since we left ST_FINISHED. This happens due to
                 * fn-advance scheduling several DL blocks in advance. */
                break;
+       case TBF_EV_MAX_N3105:
+               /* Triggered potentially by a poll timeout of PKT UL/DL TBF ASS.
+                * Reached N3105, the MS is not responding, so stop attempting
+                * using the TBF for DL assignment and change to ST_RELEASING in
+                * order to simply wait until resources can be reused (see
+                * st_releasing_on_enter()). */
+               tbf_dl_fsm_state_chg(fi, TBF_ST_RELEASING);
        default:
                OSMO_ASSERT(0);
        }

--
To view, visit https://gerrit.osmocom.org/c/osmo-pcu/+/34425?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit-Change-Id: I27e61dc4b3396360327dcd377d171daa30684d23
Gerrit-Change-Number: 34425
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pes...@sysmocom.de>
Gerrit-MessageType: newchange

Reply via email to