fixeria has uploaded this change for review.

View Change

rlcmac: fix st_new_on_enter(): actually release the TBF

The comment above the invocation of configure_ul_tbf() suggests that
the lower layers need to be notified about the *release* of a tbf_nr.

Change-Id: Iabfe9c7ee23335d6eae8017071651b78a36079bf
Related: OS#5500
---
M src/rlcmac/tbf_ul_fsm.c
1 file changed, 14 insertions(+), 1 deletion(-)

git pull ssh://gerrit.osmocom.org:29418/libosmo-gprs refs/changes/11/32411/1
diff --git a/src/rlcmac/tbf_ul_fsm.c b/src/rlcmac/tbf_ul_fsm.c
index 96adf8b..1e56d18 100644
--- a/src/rlcmac/tbf_ul_fsm.c
+++ b/src/rlcmac/tbf_ul_fsm.c
@@ -127,7 +127,7 @@
/* Mark everything we transmitted so far as NACKed: */
gprs_rlcmac_rlc_ul_window_mark_for_resend(ctx->ul_tbf->ulw);
/* Make sure the lower layers realize this tbf_nr has no longer any assigned resource: */
- configure_ul_tbf(ctx, false);
+ configure_ul_tbf(ctx, true);
}

static void st_new(struct osmo_fsm_inst *fi, uint32_t event, void *data)

To view, visit change 32411. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: libosmo-gprs
Gerrit-Branch: master
Gerrit-Change-Id: Iabfe9c7ee23335d6eae8017071651b78a36079bf
Gerrit-Change-Number: 32411
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <vyanitskiy@sysmocom.de>
Gerrit-MessageType: newchange