pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/libosmo-sccp/+/15977 )
Change subject: ASP: Stop re-transmitting un-acknowledged messages on SCTP down event ...................................................................... Patch Set 4: (2 comments) https://gerrit.osmocom.org/c/libosmo-sccp/+/15977/3//COMMIT_MSG Commit Message: https://gerrit.osmocom.org/c/libosmo-sccp/+/15977/3//COMMIT_MSG@9 PS3, Line 9: We don't want to attempt to re-transimt if we lots the connection: If we have lots of connections it's fine re-transimting! re-transmit lost https://gerrit.osmocom.org/c/libosmo-sccp/+/15977/3/src/xua_asp_fsm.c File src/xua_asp_fsm.c: https://gerrit.osmocom.org/c/libosmo-sccp/+/15977/3/src/xua_asp_fsm.c@605 PS3, Line 605: osmo_timer_del(&xafp->t_ack.timer); will the timer be armed again when we re-gain connectivity? or are we simply dropping everything on the queue then? Because in that case it makes sense to clean up the queue now... -- To view, visit https://gerrit.osmocom.org/c/libosmo-sccp/+/15977 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: libosmo-sccp Gerrit-Branch: master Gerrit-Change-Id: Ic8da646d22d3120121d9af43a890762a879f9d74 Gerrit-Change-Number: 15977 Gerrit-PatchSet: 4 Gerrit-Owner: laforge <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-CC: pespin <[email protected]> Gerrit-Comment-Date: Tue, 05 Nov 2019 20:05:19 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment
