Attention is currently required from: neels.
laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-hnbgw/+/31429 )
Change subject: sccp_sap_up(): ignore PCSTATE.ind
......................................................................
Patch Set 4:
(1 comment)
File src/osmo-hnbgw/hnbgw_cn.c:
https://gerrit.osmocom.org/c/osmo-hnbgw/+/31429/comment/6140cb06_7a96369b
PS4, Line 434: LOGP(DMAIN, LOGL_DEBUG, "Ignoring: SCCP USER SAP: %s\n", osmo_scu_prim_hdr_name_c(OTC_SELECT, oph));
this is worded a bit strange, IMHO. We're not ignoring the SAP, but we're ignoring something received from the SAP, see the log line below. I'd suggest to align the string with the one below, i.e. "Ignoring prim %s from SCCP USER SAP"
--
To view, visit https://gerrit.osmocom.org/c/osmo-hnbgw/+/31429
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-hnbgw
Gerrit-Branch: master
Gerrit-Change-Id: I547387a5cc14ccb506be04ac785e6807fc4e6a96
Gerrit-Change-Number: 31429
Gerrit-PatchSet: 4
Gerrit-Owner: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: laforge <laforge(a)osmocom.org>
Gerrit-Attention: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Comment-Date: Tue, 21 Feb 2023 09:46:11 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Attention is currently required from: neels.
laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-hnbgw/+/31432 )
Change subject: ranap_rab_ass_test: fix use of deprecated ranap_cn_rx_co_decode()
......................................................................
Patch Set 4: Code-Review+1
--
To view, visit https://gerrit.osmocom.org/c/osmo-hnbgw/+/31432
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-hnbgw
Gerrit-Branch: master
Gerrit-Change-Id: Iea41b027d047b2da52d0733fce4c9d75bb74fe48
Gerrit-Change-Number: 31432
Gerrit-PatchSet: 4
Gerrit-Owner: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Attention: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Comment-Date: Tue, 21 Feb 2023 09:44:38 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Attention is currently required from: neels.
laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-hnbgw/+/31428 )
Change subject: log tweak
......................................................................
Patch Set 1:
(1 comment)
Patchset:
PS1:
not sure we want that 'id-' prefix. That's just an artefact of auto-generated asn.1 decoders that want to tell you it's the message ID that's decoded here.
--
To view, visit https://gerrit.osmocom.org/c/osmo-hnbgw/+/31428
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-hnbgw
Gerrit-Branch: master
Gerrit-Change-Id: Ib6f8c46355d4748089e59c80fc1c8c736c887a26
Gerrit-Change-Number: 31428
Gerrit-PatchSet: 1
Gerrit-Owner: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: laforge <laforge(a)osmocom.org>
Gerrit-Attention: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Comment-Date: Tue, 21 Feb 2023 09:43:45 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment