neels has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/33473 )
Change subject: hnbgw: drop dead code from MSC_UnitdataCallback ...................................................................... Patch Set 1: (1 comment) Patchset: PS1: OK, I remember now. a) not used: osmo-hnbgw has only recently started sending RANAP UnitData by its own initiative. Until very recently, osmo-hnbgw has only responded to receiving a RESET, with an ACK. It has never sent a RESET message to the peer, here titan. Also on RANAP, the HNBGW_Tests.ttcn so far conveniently ignored RANAP RESET handling. In short, this UnitDataCallback so far never happened. b) makes no sense: The UnitDataCallback happens when osmo-hnbgw sends a RANAP UnitData to titan: so currently, in response to receiving any UnitData message, the code responds with a RESET. That is not logical at all. (What would make sense: when receiving a RESET, respond with RESET ACK. But: Proper handling of RANAP RESET in ttcn actually happens in RAN_Emulation.ttcnpp) -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/33473 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-Change-Id: Ie7b9022e991b63b945c7ec6e5c9f7c4eb5da4d7e Gerrit-Change-Number: 33473 Gerrit-PatchSet: 1 Gerrit-Owner: neels <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-CC: laforge <[email protected]> Gerrit-Comment-Date: Tue, 04 Jul 2023 02:24:28 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment
