Attention is currently required from: pespin.
laforge has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40676?usp=email )
Change subject: smlc: Use static sigtran config between osmo-msc and osmo-stp
......................................................................
Patch Set 1: Code-Review+1
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40676?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I008dba252da29f445c52151c7b65428ca7054ed0
Gerrit-Change-Number: 40676
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Tue, 15 Jul 2025 08:23:57 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
lynxis lazus has submitted this change. ( https://gerrit.osmocom.org/c/osmo-sgsn/+/40653?usp=email )
Change subject: gprs_gmm: fix endianess of GSM48_IE_GMM_PDP_CTX_STATUS
......................................................................
gprs_gmm: fix endianess of GSM48_IE_GMM_PDP_CTX_STATUS
The pdp_status is actually encoded as little endian,
but there is no tlvp_val16le yet.
Change-Id: Ib9bee2f8d0b1f89986d15cf3ce6404ad76378c8c
---
M src/sgsn/gprs_gmm.c
1 file changed, 2 insertions(+), 1 deletion(-)
Approvals:
Jenkins Builder: Verified
pespin: Looks good to me, approved
fixeria: Looks good to me, but someone else must approve
daniel: Looks good to me, but someone else must approve
diff --git a/src/sgsn/gprs_gmm.c b/src/sgsn/gprs_gmm.c
index c34c1a1..1fa73b9 100644
--- a/src/sgsn/gprs_gmm.c
+++ b/src/sgsn/gprs_gmm.c
@@ -1975,7 +1975,8 @@
/* Look at PDP Context Status IE and see if MS's view of
* activated/deactivated NSAPIs agrees with our view */
if (TLVP_PRESENT(&tp, GSM48_IE_GMM_PDP_CTX_STATUS)) {
- uint16_t pdp_status = tlvp_val16be(&tp, GSM48_IE_GMM_PDP_CTX_STATUS);
+ /* TODO: Use tlvp_val16le when available */
+ uint16_t pdp_status = osmo_load16le(TLVP_VAL(&tp, GSM48_IE_GMM_PDP_CTX_STATUS));
process_ms_ctx_status(ctx, pdp_status);
--
To view, visit https://gerrit.osmocom.org/c/osmo-sgsn/+/40653?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: merged
Gerrit-Project: osmo-sgsn
Gerrit-Branch: master
Gerrit-Change-Id: Ib9bee2f8d0b1f89986d15cf3ce6404ad76378c8c
Gerrit-Change-Number: 40653
Gerrit-PatchSet: 3
Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: daniel <dwillmann(a)sysmocom.de>
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: lynxis lazus <lynxis(a)fe80.eu>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Hello Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-smlc/+/40665?usp=email
to look at the new patch set (#4).
The following approvals got outdated and were removed:
Verified+1 by Jenkins Builder
Change subject: smlc: Initial implementation of N-PCSTATE.ind
......................................................................
smlc: Initial implementation of N-PCSTATE.ind
Related: OS#5917
Change-Id: Id034a0c4d8bff0647a64658480dfa9b4cea563de
---
M include/osmocom/smlc/lb_peer.h
M src/osmo-smlc/lb_peer.c
M src/osmo-smlc/sccp_lb_inst.c
3 files changed, 155 insertions(+), 2 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-smlc refs/changes/65/40665/4
--
To view, visit https://gerrit.osmocom.org/c/osmo-smlc/+/40665?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: newpatchset
Gerrit-Project: osmo-smlc
Gerrit-Branch: master
Gerrit-Change-Id: Id034a0c4d8bff0647a64658480dfa9b4cea563de
Gerrit-Change-Number: 40665
Gerrit-PatchSet: 4
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Hello Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-smlc/+/40679?usp=email
to look at the new patch set (#2).
The following approvals got outdated and were removed:
Verified+1 by Jenkins Builder
Change subject: Introduce stats msc.ran_peers.{total,active}
......................................................................
Introduce stats msc.ran_peers.{total,active}
osmo_stats_init() was already being called despite no stats were being
used.
Change-Id: Ib01576036f34ac7f21c5bce0155b50932eb9b72a
---
M include/osmocom/smlc/smlc_data.h
M src/osmo-smlc/lb_peer.c
M src/osmo-smlc/smlc_data.c
M src/osmo-smlc/smlc_vty.c
4 files changed, 55 insertions(+), 0 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-smlc refs/changes/79/40679/2
--
To view, visit https://gerrit.osmocom.org/c/osmo-smlc/+/40679?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: newpatchset
Gerrit-Project: osmo-smlc
Gerrit-Branch: master
Gerrit-Change-Id: Ib01576036f34ac7f21c5bce0155b50932eb9b72a
Gerrit-Change-Number: 40679
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
pespin has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-smlc/+/40677?usp=email )
Change subject: cell_locations.h: Fix includes
......................................................................
cell_locations.h: Fix includes
No sigtran header is needed as a dependecy in that header.
struct gsm0808_cell_id is defined in gsm0808_utils.h.
Change-Id: Iceedd7a8c2e050e6631c5f471d2eb71ab8ee2432
---
M include/osmocom/smlc/cell_locations.h
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-smlc refs/changes/77/40677/1
diff --git a/include/osmocom/smlc/cell_locations.h b/include/osmocom/smlc/cell_locations.h
index 9712718..5eb5d7e 100644
--- a/include/osmocom/smlc/cell_locations.h
+++ b/include/osmocom/smlc/cell_locations.h
@@ -23,7 +23,7 @@
#include <stdint.h>
#include <osmocom/core/linuxlist.h>
-#include <osmocom/sigtran/sccp_sap.h>
+#include <osmocom/gsm/gsm0808_utils.h>
struct osmo_gad;
--
To view, visit https://gerrit.osmocom.org/c/osmo-smlc/+/40677?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: newchange
Gerrit-Project: osmo-smlc
Gerrit-Branch: master
Gerrit-Change-Id: Iceedd7a8c2e050e6631c5f471d2eb71ab8ee2432
Gerrit-Change-Number: 40677
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin(a)sysmocom.de>