tbf: Remove TBF chaining (m_new_tbf and m_old_tbf)

Currently a new TBF is chained to an existing older one, either of
the other direction (active or releasing) or of the same direction
(releasing). This does not work properly work if and uplink and a
downlink TBF are being established at the same time while an old TBF
is being released. In that case, one of them is thrown away and the
pending procedure is cancelled.

The chaining is no longer necessary since the GprsMs objects have
been introduced which keep track of the active TBFs.

This commit removes the TBF members m_new_tbf and m_old_tbf and the
related methods and code paths.

Note that a new TBF can replace an older TBF entry of the same
direction within an MS object when it is associated with an MS (e.g.
by TLLI or because it is assigned via another, already associated
TBF). In that case, the old TBF is no longer associated with an MS
object.

Ticket: #1674
Sponsored-by: On-Waves ehf

1 files changed, 0 insertions, 11 deletions

diff --git a/src/tbf.h b/src/tbf.h
index 0820065f..5ea6d4ef 100644
--- a/src/tbf.h
+++ b/src/tbf.h
@@ -150,9 +150,6 @@ struct gprs_rlcmac_tbf {
 	const char *imsi() const;
 	void assign_imsi(const char *imsi);
 
-	void set_new_tbf(gprs_rlcmac_tbf *tbf);
-	gprs_rlcmac_tbf *new_tbf() const;
-
 	time_t created_ts() const;
 
 	/* attempt to make things a bit more fair */
@@ -176,9 +173,6 @@ struct gprs_rlcmac_tbf {
 	enum gprs_rlcmac_tbf_ul_ass_state ul_ass_state;
 	enum gprs_rlcmac_tbf_ul_ack_state ul_ack_state;
 
-	gprs_rlcmac_tbf *m_new_tbf;
-	gprs_rlcmac_tbf *m_old_tbf; /* reverse pointer for m_new_tbf */
-
 	enum gprs_rlcmac_tbf_poll_state poll_state;
 	uint32_t poll_fn; /* frame number to poll */
 
@@ -297,11 +291,6 @@ inline uint8_t gprs_rlcmac_tbf::tfi() const
 	return m_tfi;
 }
 
-inline gprs_rlcmac_tbf *gprs_rlcmac_tbf::new_tbf() const
-{
-	return m_new_tbf;
-}
-
 inline time_t gprs_rlcmac_tbf::created_ts() const
 {
 	return m_created_ts;