Attention is currently required from: laforge. neels has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bsc/+/27371 )
Change subject: add counter for inter-BSC incoming Handover Request ...................................................................... Patch Set 2: (2 comments) Patchset: PS2: > in general I find it completely strange that the type of SCCP transport layer > message is part of the […] yes exactly the reason why i was reluctant to complicate things here. I'm not sure where the DT1 came from originally... File src/osmo-bsc/osmo_bsc_msc.c: https://gerrit.osmocom.org/c/osmo-bsc/+/27371/comment/e8066bd3_9601c9a7 PS2, Line 58: [MSC_CTR_BSSMAP_RX_DT1_HANDOVER_RQST] = {"bssmap:rx:dt1:handover:rqst", "Number of received BSSMAP DT1 HANDOVER RQST messages"}, > is the description and naming correct here? Are we really only counting > those handover requests whi […] that's true. i was thinking though, should we really have separate counters for those cases? if i want to see the nr of ho req, do i want to see separate counters for those coming in the CR and those coming juuust after the CR? maybe it should be called _RX_CR_AND_DT1_? same thing for the Perform Location Request in the other patch. There we already have MSC_CTR_BSSMAP_RX_DT1_PERFORM_LOCATION_REQUEST so far accurately only counting the DT1 Perform Loc Req; because we so far don't count the CR ones at all. I guess we don't want to rename that counter. So then is the conclusion to after all do have separate counters: MSC_CTR_BSSMAP_RX_CR_HANDOVER_REQUEST MSC_CTR_BSSMAP_RX_DT1_HANDOVER_REQUEST MSC_CTR_BSSMAP_RX_CR_PERFORM_LOCATION_REQUEST MSC_CTR_BSSMAP_RX_DT1_PERFORM_LOCATION_REQUEST to benefit accuracy but not neccessarily practical use ... ? -- To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/27371 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-bsc Gerrit-Branch: master Gerrit-Change-Id: Icdde2bb339a5e367a4d297802214a1ef3f36eefa Gerrit-Change-Number: 27371 Gerrit-PatchSet: 2 Gerrit-Owner: neels <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <[email protected]> Gerrit-Reviewer: laforge <[email protected]> Gerrit-Attention: laforge <[email protected]> Gerrit-Comment-Date: Thu, 03 Mar 2022 22:10:11 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: laforge <[email protected]> Gerrit-MessageType: comment
