ipa: Move automatic route add/del from ASP to AS level

SS7 routes operate on AS level, not ASP level.  However, the
automatic SS7 route creation/destruction for IPA was implemented
at the ASP level.  This works for single-connection ASs, but
obviously fails in load-share situations:  We attempt to add the
same route several times, and we delete it at the first ASP
disconnect, even while other ASPs still exist.

This patch moves the IPA route creation/deletion from the ASP level
to the AS level.  When the AS becomes active, the route is added;
when it goes to DOWN state, it is removed.

Change-Id: Idb602beae3e9bc19f7bd96355c02ec8dfd9c5d6c

1 files changed, 0 insertions, 55 deletions

diff --git a/src/xua_asp_fsm.c b/src/xua_asp_fsm.c
index f8683d5..ec85d21 100644
--- a/src/xua_asp_fsm.c
+++ b/src/xua_asp_fsm.c
@@ -939,23 +939,11 @@ out_err:
 /* Server: We're waiting for an ID ACK */
 static void ipa_asp_fsm_wait_id_ack2(struct osmo_fsm_inst *fi, uint32_t event, void *data)
 {
-	struct ipa_asp_fsm_priv *iafp = fi->priv;
-	struct osmo_ss7_asp *asp = iafp->asp;
-	struct osmo_ss7_instance *inst = asp->inst;
-	struct osmo_ss7_as *as;
-
-	xua_find_as_for_asp(&as, asp, NULL);
-	OSMO_ASSERT(as);
-
 	switch (event) {
 	case IPA_ASP_E_ID_ACK:
 		/* ACK received, we can go to active state now.  The
 		 * ACTIVE onenter function will inform the AS */
 		osmo_fsm_inst_state_chg(fi, IPA_ASP_S_ACTIVE, 0, 0);
-		/* As opposed to M3UA, there is no RKM and we have to implicitly automatically add
-		 * a route once an IPA connection has come up */
-		osmo_ss7_route_create(inst->rtable_system, as->cfg.routing_key.pc, 0xffffff,
-				      as->cfg.name);
 		break;
 	}
 }
@@ -1011,48 +999,12 @@ static void ipa_asp_fsm_wait_id_ack(struct osmo_fsm_inst *fi, uint32_t event, vo
 	}
 }
 
-static void ipa_asp_fsm_del_route(struct osmo_fsm_inst *fi)
-{
-	struct ipa_asp_fsm_priv *iafp = fi->priv;
-	struct osmo_ss7_asp *asp = iafp->asp;
-	struct osmo_ss7_instance *inst = asp->inst;
-	struct osmo_ss7_as *as;
-	struct osmo_ss7_route *rt;
-
-	xua_find_as_for_asp(&as, asp, NULL);
-	OSMO_ASSERT(as);
-
-	/* find the route which we have created if we ever reached ipa_asp_fsm_wait_id_ack2 */
-	rt = osmo_ss7_route_find_dpc_mask(inst->rtable_system, as->cfg.routing_key.pc, 0xffffff);
-	/* no route found, bail out */
-	if (!rt) {
-		LOGPFSML(fi, LOGL_NOTICE, "Attempting to delete route for this IPA ASP, but cannot "
-			 "find route for DPC %s. Did you manually delete it?\n",
-			 osmo_ss7_pointcode_print(inst, as->cfg.routing_key.pc));
-		return;
-	}
-
-	/* route points to different AS, bail out */
-	if (rt->dest.as != as) {
-		LOGPFSML(fi, LOGL_NOTICE, "Attempting to delete route for this IPA ASP, but found "
-			 "route for DPC %s points to different AS (%s)\n",
-			 osmo_ss7_pointcode_print(inst, as->cfg.routing_key.pc), rt->dest.as->cfg.name);
-		return;
-	}
-
-	osmo_ss7_route_destroy(rt);
-	/* FIXME: Why don't we also delete this timer if we return early above?
-	 * FIXME: Where is this timer even scheduled? */
-	osmo_timer_del(&iafp->pong_timer);
-}
-
 /* Server + Client: We're actively transmitting user data */
 static void ipa_asp_fsm_active(struct osmo_fsm_inst *fi, uint32_t event, void *data)
 {
 	switch (event) {
 	case XUA_ASP_E_M_ASP_DOWN_REQ:
 	case XUA_ASP_E_M_ASP_INACTIVE_REQ:
-		ipa_asp_fsm_del_route(fi);
 		osmo_fsm_inst_state_chg(fi, IPA_ASP_S_DOWN, 0, 0);
 		break;
 	}
@@ -1062,7 +1014,6 @@ static void ipa_asp_fsm_inactive(struct osmo_fsm_inst *fi, uint32_t event, void
 {
 	switch (event) {
 	case XUA_ASP_E_M_ASP_DOWN_REQ:
-		ipa_asp_fsm_del_route(fi);
 		osmo_fsm_inst_state_chg(fi, IPA_ASP_S_DOWN, 0, 0);
 		break;
 	}
@@ -1187,11 +1138,6 @@ static const struct osmo_fsm_state ipa_asp_states[] = {
 	},
 };
 
-static void ipa_asp_fsm_cleanup(struct osmo_fsm_inst *fi, enum osmo_fsm_term_cause cause)
-{
-	ipa_asp_fsm_del_route(fi);
-}
-
 struct osmo_fsm ipa_asp_fsm = {
 	.name = "IPA_ASP",
 	.states = ipa_asp_states,
@@ -1204,7 +1150,6 @@ struct osmo_fsm ipa_asp_fsm = {
 			       S(XUA_ASP_E_ASPSM_BEAT) |
 			       S(XUA_ASP_E_ASPSM_BEAT_ACK),
 	.allstate_action = ipa_asp_allstate,
-	.cleanup = ipa_asp_fsm_cleanup,
 };