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