Attention is currently required from: pespin.

osmith has posted comments on this change by pespin. ( 
https://gerrit.osmocom.org/c/libosmo-sigtran/+/39377?usp=email )

Change subject: AS loadsharing: Initial routing implementation based on 
extended-SLS
......................................................................


Patch Set 4: Code-Review+1

(2 comments)

File include/osmocom/sigtran/osmo_ss7.h:

https://gerrit.osmocom.org/c/libosmo-sigtran/+/39377/comment/a5b4c5a8_87eb182d?usp=email
 :
PS4, Line 129: internal
what do you mean with internal here?


File src/osmo_ss7_combined_linkset.c:

https://gerrit.osmocom.org/c/libosmo-sigtran/+/39377/comment/25c6ae98_9b9bc906?usp=email
 :
PS4, Line 166: struct osmo_ss7_route *current_rt(const struct 
osmo_ss7_combined_linkset *clset, const struct osmo_ss7_esls_entry *eslse)
clset is not uses in this function?



--
To view, visit https://gerrit.osmocom.org/c/libosmo-sigtran/+/39377?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings?usp=email

Gerrit-MessageType: comment
Gerrit-Project: libosmo-sigtran
Gerrit-Branch: master
Gerrit-Change-Id: I0fb4ca4959096f748a23082efa0481300de56436
Gerrit-Change-Number: 39377
Gerrit-PatchSet: 4
Gerrit-Owner: pespin <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: osmith <[email protected]>
Gerrit-Attention: pespin <[email protected]>
Gerrit-Comment-Date: Wed, 26 Feb 2025 10:20:20 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes

Reply via email to