pespin submitted this change.

View Change

Approvals: Jenkins Builder: Verified fixeria: Looks good to me, approved
cosmetic: xua_snm: Add spec reference regarding no SSNM tx

Change-Id: I69912d5b7032c12e06648e5f5a700384182bb1f9
---
M src/xua_snm.c
1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/xua_snm.c b/src/xua_snm.c
index 916f6c3..a2e4fde 100644
--- a/src/xua_snm.c
+++ b/src/xua_snm.c
@@ -236,7 +236,7 @@

/* inform remote ASPs via DUNA/DAVA */
llist_for_each_entry(asp, &s7i->asp_list, list) {
- /* SSNM is only permitted for ASPs in ACTIVE state */
+ /* SSNM is only permitted for ASPs in ACTIVE state (RFC4666 4.3.1) */
if (!osmo_ss7_asp_active(asp))
continue;

@@ -278,7 +278,7 @@

/* inform remote SUA ASPs via DUNA/DAVA */
llist_for_each_entry(asp, &s7i->asp_list, list) {
- /* SSNM is only permitted for ASPs in ACTIVE state */
+ /* SSNM is only permitted for ASPs in ACTIVE state (RFC4666 4.3.1) */
if (!osmo_ss7_asp_active(asp))
continue;

@@ -317,7 +317,7 @@

/* inform remote ASPs via DUPU */
llist_for_each_entry(asp, &s7i->asp_list, list) {
- /* SSNM is only permitted for ASPs in ACTIVE state */
+ /* SSNM is only permitted for ASPs in ACTIVE state (RFC4666 4.3.1) */
if (!osmo_ss7_asp_active(asp))
continue;

@@ -386,7 +386,7 @@
* procedures of the relevant MTP3 standard."
* ie. inform remote ASPs via SCON: */
llist_for_each_entry(asp, &s7i->asp_list, list) {
- /* SSNM is only permitted for ASPs in ACTIVE state */
+ /* SSNM is only permitted for ASPs in ACTIVE state (RFC4666 4.3.1) */
if (!osmo_ss7_asp_active(asp))
continue;


To view, visit change 42505. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-MessageType: merged
Gerrit-Project: libosmo-sigtran
Gerrit-Branch: master
Gerrit-Change-Id: I69912d5b7032c12e06648e5f5a700384182bb1f9
Gerrit-Change-Number: 42505
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <pespin@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>