April 2023 - gerrit-log - lists.osmocom.org

[S] Change in libosmocore[master]: gsm_04_08_gprs.h: Add missing GMM IEs for T3302 and T3346
by pespin 13 Apr '23

13 Apr '23

pespin has submitted this change. ( https://gerrit.osmocom.org/c/libosmocore/+/32288 ) Change subject: gsm_04_08_gprs.h: Add missing GMM IEs for T3302 and T3346 ...................................................................... gsm_04_08_gprs.h: Add missing GMM IEs for T3302 and T3346 As specified in TS 24.008 9.4.4 "Attach reject" Change-Id: I2d36d76ee6fe8ed1a36e37a7d74fbbdc9c27c2c7 --- M include/osmocom/gsm/protocol/gsm_04_08_gprs.h 1 file changed, 13 insertions(+), 0 deletions(-) Approvals: Jenkins Builder: Verified fixeria: Looks good to me, approved diff --git a/include/osmocom/gsm/protocol/gsm_04_08_gprs.h b/include/osmocom/gsm/protocol/gsm_04_08_gprs.h index 39f3b3e..3d18243 100644 --- a/include/osmocom/gsm/protocol/gsm_04_08_gprs.h +++ b/include/osmocom/gsm/protocol/gsm_04_08_gprs.h @@ -92,11 +92,13 @@ GSM48_IE_GMM_DRX_PARAM = 0x27, /* 10.5.5.6 */ GSM48_IE_GMM_AUTN = 0x28, /* 10.5.3.1.1 */ GSM48_IE_GMM_AUTH_RES_EXT = 0x29, /* 10.5.3.2.1 */ + GSM48_IE_GMM_TIMER_T3302 = 0x2A, /* 10.5.7.4 */ GSM48_IE_GMM_AUTH_FAIL_PAR = 0x30, /* 10.5.3.2.2 */ GSM48_IE_GMM_MS_NET_CAPA = 0x31, /* 10.5.5.12 */ GSM48_IE_GMM_PDP_CTX_STATUS = 0x32, /* 10.5.7.1 */ GSM48_IE_GMM_PS_LCS_CAPA = 0x33, /* 10.5.5.22 */ GSM48_IE_GMM_GMM_MBMS_CTX_ST = 0x35, /* 10.5.7.6 */ + GSM48_IE_GMM_TIMER_T3346 = 0x3A, /* 10.5.7.4 */ GSM48_IE_GMM_NET_FEAT_SUPPORT = 0xB0, /* 10.5.5.23 */ }; -- To view, visit https://gerrit.osmocom.org/c/libosmocore/+/32288 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: libosmocore Gerrit-Branch: master Gerrit-Change-Id: I2d36d76ee6fe8ed1a36e37a7d74fbbdc9c27c2c7 Gerrit-Change-Number: 32288 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-MessageType: merged

1 0

[M] Change in osmocom-bb[master]: layer23: modem: Forward SNSM primitives SNDCP<->SM layers
by pespin 13 Apr '23

13 Apr '23

pespin has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmocom-bb/+/32296 ) Change subject: layer23: modem: Forward SNSM primitives SNDCP<->SM layers ...................................................................... layer23: modem: Forward SNSM primitives SNDCP<->SM layers Related: OS#5501 Depends: libosmo-gprs.git Change-Id Ic5525bfa92e2591757d999610c0d59849d022d8d Change-Id: Iad63f41557fbd078588dd890e2f18ed9606a25f7 --- M src/host/layer23/src/modem/sm.c M src/host/layer23/src/modem/sndcp.c 2 files changed, 50 insertions(+), 1 deletion(-) git pull ssh://gerrit.osmocom.org:29418/osmocom-bb refs/changes/96/32296/1 diff --git a/src/host/layer23/src/modem/sm.c b/src/host/layer23/src/modem/sm.c index 5dc80e0..376910a 100644 --- a/src/host/layer23/src/modem/sm.c +++ b/src/host/layer23/src/modem/sm.c @@ -34,6 +34,8 @@ #include <osmocom/gprs/gmm/gmm.h> #include <osmocom/gprs/gmm/gmm_prim.h> +#include <osmocom/gprs/sndcp/sndcp.h> +#include <osmocom/gprs/sndcp/sndcp_prim.h> #include <osmocom/gprs/sm/sm_prim.h> #include <osmocom/gprs/sm/sm.h> #include <osmocom/gprs/rlcmac/rlcmac_prim.h> @@ -78,6 +80,30 @@ return rc; } +int modem_sm_prim_sndcp_up_cb(struct osmo_gprs_sndcp_prim *sndcp_prim, void *user_data) +{ + const char *pdu_name = osmo_gprs_sndcp_prim_name(sndcp_prim); + int rc; + + switch (sndcp_prim->oph.sap) { + case OSMO_GPRS_SNDCP_SAP_SNSM: + switch (OSMO_PRIM_HDR(&sndcp_prim->oph)) { + case OSMO_PRIM(OSMO_GPRS_SM_SMREG_PDP_ACTIVATE, PRIM_OP_INDICATION): + LOGP(DSM, LOGL_INFO, "%s(): Rx %s\n", __func__, pdu_name); + rc = osmo_gprs_sndcp_prim_dispatch_snsm(sndcp_prim); + break; + default: + LOGP(DSM, LOGL_ERROR, "%s(): Unexpected Rx %s\n", __func__, pdu_name); + OSMO_ASSERT(0); + } + break; + default: + LOGP(DSM, LOGL_ERROR, "%s(): Unexpected Rx %s\n", __func__, pdu_name); + OSMO_ASSERT(0); + } + return rc; +} + static int modem_sm_prim_down_cb(struct osmo_gprs_sm_prim *sm_prim, void *user_data) { const char *pdu_name = osmo_gprs_sm_prim_name(sm_prim); @@ -113,6 +139,7 @@ osmo_gprs_sm_set_log_cat(OSMO_GPRS_SM_LOGC_SM, DSM); osmo_gprs_sm_prim_set_up_cb(modem_sm_prim_up_cb, ms); + osmo_gprs_sm_prim_set_sndcp_up_cb(modem_sm_prim_sndcp_up_cb, ms); osmo_gprs_sm_prim_set_down_cb(modem_sm_prim_down_cb, ms); osmo_gprs_sm_prim_set_gmm_down_cb(modem_sm_prim_gmm_down_cb, ms); diff --git a/src/host/layer23/src/modem/sndcp.c b/src/host/layer23/src/modem/sndcp.c index ee5380f..e99c096 100644 --- a/src/host/layer23/src/modem/sndcp.c +++ b/src/host/layer23/src/modem/sndcp.c @@ -33,6 +33,8 @@ #include <osmocom/gprs/llc/llc.h> #include <osmocom/gprs/llc/llc_prim.h> +#include <osmocom/gprs/sm/sm_prim.h> +#include <osmocom/gprs/sm/sm.h> #include <osmocom/gprs/sndcp/sndcp_prim.h> #include <osmocom/gprs/sndcp/sndcp.h> @@ -132,13 +134,22 @@ static int modem_sndcp_prim_snsm_cb(struct osmo_gprs_sndcp_prim *sndcp_prim, void *user_data) { const char *npdu_name = osmo_gprs_sndcp_prim_name(sndcp_prim); + int rc; if (sndcp_prim->oph.sap != OSMO_GPRS_SNDCP_SAP_SNSM) { LOGP(DSNDCP, LOGL_ERROR, "%s(): Unexpected Rx %s\n", __func__, npdu_name); OSMO_ASSERT(0); } - LOGP(DSNDCP, LOGL_ERROR, "%s(): Rx %s UNIMPLEMENTED\n", __func__, npdu_name); + switch (OSMO_PRIM_HDR(&sndcp_prim->oph)) { + case OSMO_PRIM(OSMO_GPRS_SM_SMREG_PDP_ACTIVATE, PRIM_OP_RESPONSE): + LOGP(DSNDCP, LOGL_INFO, "%s(): Rx %s\n", __func__, npdu_name); + rc = osmo_gprs_sm_prim_sndcp_upper_down(sndcp_prim); + break; + default: + LOGP(DSNDCP, LOGL_ERROR, "%s(): Unexpected Rx %s\n", __func__, npdu_name); + OSMO_ASSERT(0); + } return 0; } -- To view, visit https://gerrit.osmocom.org/c/osmocom-bb/+/32296 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmocom-bb Gerrit-Branch: master Gerrit-Change-Id: Iad63f41557fbd078588dd890e2f18ed9606a25f7 Gerrit-Change-Number: 32296 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-MessageType: newchange

1 0

[S] Change in libosmocore[master]: gsm_04_08_gprs.h: Add missing GMM IEs for T3302 and T3346
by fixeria 13 Apr '23

13 Apr '23

Attention is currently required from: osmith, pespin. fixeria has posted comments on this change. ( https://gerrit.osmocom.org/c/libosmocore/+/32288 ) Change subject: gsm_04_08_gprs.h: Add missing GMM IEs for T3302 and T3346 ...................................................................... Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/libosmocore/+/32288 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: libosmocore Gerrit-Branch: master Gerrit-Change-Id: I2d36d76ee6fe8ed1a36e37a7d74fbbdc9c27c2c7 Gerrit-Change-Number: 32288 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Attention: osmith <osmith(a)sysmocom.de> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Comment-Date: Thu, 13 Apr 2023 17:08:36 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

1 0

[M] Change in libosmo-gprs[master]: sm: Start using SNSM SAP
by pespin 13 Apr '23

13 Apr '23

Hello Jenkins Builder, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/c/libosmo-gprs/+/32295 to look at the new patch set (#2). Change subject: sm: Start using SNSM SAP ...................................................................... sm: Start using SNSM SAP Related: OS#5501 Change-Id: Ic5525bfa92e2591757d999610c0d59849d022d8d --- M include/osmocom/gprs/sm/sm_ms_fsm.h M include/osmocom/gprs/sm/sm_prim.h M include/osmocom/gprs/sm/sm_private.h M libosmo-gprs-sm.pc.in M src/sm/Makefile.am M src/sm/sm.c M src/sm/sm_ms_fsm.c M src/sm/sm_prim.c M tests/sm/Makefile.am M tests/sm/sm_prim_test.c M tests/sm/sm_prim_test.err M tests/sm/sm_prim_test.ok 12 files changed, 213 insertions(+), 4 deletions(-) git pull ssh://gerrit.osmocom.org:29418/libosmo-gprs refs/changes/95/32295/2 -- To view, visit https://gerrit.osmocom.org/c/libosmo-gprs/+/32295 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: libosmo-gprs Gerrit-Branch: master Gerrit-Change-Id: Ic5525bfa92e2591757d999610c0d59849d022d8d Gerrit-Change-Number: 32295 Gerrit-PatchSet: 2 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-MessageType: newpatchset

1 0

[M] Change in libosmo-gprs[master]: sm: Start using SNSM SAP
by pespin 13 Apr '23

13 Apr '23

pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/libosmo-gprs/+/32295 ) Change subject: sm: Start using SNSM SAP ...................................................................... Patch Set 1: (1 comment) Patchset: PS1: I now need to forward this new interface in the modem app too. -- To view, visit https://gerrit.osmocom.org/c/libosmo-gprs/+/32295 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: libosmo-gprs Gerrit-Branch: master Gerrit-Change-Id: Ic5525bfa92e2591757d999610c0d59849d022d8d Gerrit-Change-Number: 32295 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Comment-Date: Thu, 13 Apr 2023 16:56:20 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment

1 0

[S] Change in libosmo-gprs[master]: sm: Fix typo in LOGMS macro
by pespin 13 Apr '23

13 Apr '23

pespin has uploaded this change for review. ( https://gerrit.osmocom.org/c/libosmo-gprs/+/32294 ) Change subject: sm: Fix typo in LOGMS macro ...................................................................... sm: Fix typo in LOGMS macro Change-Id: I954fcd03ba2c24cebbd8e92c309e490afae14613 --- M include/osmocom/gprs/sm/sm_private.h 1 file changed, 10 insertions(+), 1 deletion(-) git pull ssh://gerrit.osmocom.org:29418/libosmo-gprs refs/changes/94/32294/1 diff --git a/include/osmocom/gprs/sm/sm_private.h b/include/osmocom/gprs/sm/sm_private.h index 58b4a61..e599759 100644 --- a/include/osmocom/gprs/sm/sm_private.h +++ b/include/osmocom/gprs/sm/sm_private.h @@ -168,7 +168,7 @@ int gprs_sm_rx(struct gprs_sm_entity *sme, struct gsm48_hdr *gh, unsigned int len); -#define LOGMS(snme, level, fmt, args...) \ +#define LOGMS(ms, level, fmt, args...) \ LOGSM(level, "MS(ID-%u) " fmt, \ ms->ms_id, \ ## args) -- To view, visit https://gerrit.osmocom.org/c/libosmo-gprs/+/32294 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: libosmo-gprs Gerrit-Branch: master Gerrit-Change-Id: I954fcd03ba2c24cebbd8e92c309e490afae14613 Gerrit-Change-Number: 32294 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-MessageType: newchange

1 0

[M] Change in libosmo-gprs[master]: sm: Start using SNSM SAP
by pespin 13 Apr '23

13 Apr '23

pespin has uploaded this change for review. ( https://gerrit.osmocom.org/c/libosmo-gprs/+/32295 ) Change subject: sm: Start using SNSM SAP ...................................................................... sm: Start using SNSM SAP Related: OS#5501 Change-Id: Ic5525bfa92e2591757d999610c0d59849d022d8d --- M include/osmocom/gprs/sm/sm_ms_fsm.h M include/osmocom/gprs/sm/sm_prim.h M include/osmocom/gprs/sm/sm_private.h M libosmo-gprs-sm.pc.in M src/sm/Makefile.am M src/sm/sm.c M src/sm/sm_ms_fsm.c M src/sm/sm_prim.c M tests/sm/Makefile.am M tests/sm/sm_prim_test.c M tests/sm/sm_prim_test.err M tests/sm/sm_prim_test.ok 12 files changed, 211 insertions(+), 2 deletions(-) git pull ssh://gerrit.osmocom.org:29418/libosmo-gprs refs/changes/95/32295/1 diff --git a/include/osmocom/gprs/sm/sm_ms_fsm.h b/include/osmocom/gprs/sm/sm_ms_fsm.h index 3970d73..d1d4689 100644 --- a/include/osmocom/gprs/sm/sm_ms_fsm.h +++ b/include/osmocom/gprs/sm/sm_ms_fsm.h @@ -22,6 +22,7 @@ GPRS_SM_MS_EV_TX_ACT_PDP_CTX_REQ, GPRS_SM_MS_EV_RX_ACT_PDP_CTX_REJ, /* data: enum gsm48_gsm_cause *cause */ GPRS_SM_MS_EV_RX_ACT_PDP_CTX_ACC, + GPRS_SM_MS_EV_NSAPI_ACTIVATED, GPRS_SM_MS_EV_TX_DEACT_PDP_CTX_REQ, GPRS_SM_MS_EV_RX_DEACT_PDP_CTX_REQ, GPRS_SM_MS_EV_RX_DEACT_PDP_CTX_ACC, diff --git a/include/osmocom/gprs/sm/sm_prim.h b/include/osmocom/gprs/sm/sm_prim.h index 57fc32e..575ea80 100644 --- a/include/osmocom/gprs/sm/sm_prim.h +++ b/include/osmocom/gprs/sm/sm_prim.h @@ -18,6 +18,7 @@ struct osmo_gprs_gmm_prim; +struct osmo_gprs_sndcp_prim; /* 3GPP TS 24.007 */ enum osmo_gprs_sm_prim_sap { @@ -191,6 +192,9 @@ typedef int (*osmo_gprs_sm_prim_up_cb)(struct osmo_gprs_sm_prim *sm_prim, void *up_user_data); void osmo_gprs_sm_prim_set_up_cb(osmo_gprs_sm_prim_up_cb up_cb, void *up_user_data); +typedef int (*osmo_gprs_sm_prim_sndcp_up_cb)(struct osmo_gprs_sndcp_prim *sndcp_prim, void *sndcp_up_user_data); +void osmo_gprs_sm_prim_set_sndcp_up_cb(osmo_gprs_sm_prim_sndcp_up_cb sndcp_up_cb, void *sndcp_up_user_data); + typedef int (*osmo_gprs_sm_prim_down_cb)(struct osmo_gprs_sm_prim *sm_prim, void *down_user_data); void osmo_gprs_sm_prim_set_down_cb(osmo_gprs_sm_prim_down_cb down_cb, void *down_user_data); @@ -198,6 +202,7 @@ void osmo_gprs_sm_prim_set_gmm_down_cb(osmo_gprs_sm_prim_gmm_down_cb gmm_down_cb, void *gmm_down_user_data); int osmo_gprs_sm_prim_upper_down(struct osmo_gprs_sm_prim *sm_prim); +int osmo_gprs_sm_prim_sndcp_upper_down(struct osmo_gprs_sndcp_prim *sndcp_prim); int osmo_gprs_sm_prim_lower_up(struct osmo_gprs_sm_prim *sm_prim); int osmo_gprs_sm_prim_gmm_lower_up(struct osmo_gprs_gmm_prim *gmm_prim); diff --git a/include/osmocom/gprs/sm/sm_private.h b/include/osmocom/gprs/sm/sm_private.h index e599759..357b334 100644 --- a/include/osmocom/gprs/sm/sm_private.h +++ b/include/osmocom/gprs/sm/sm_private.h @@ -76,6 +76,9 @@ osmo_gprs_sm_prim_up_cb sm_up_cb; void *sm_up_cb_user_data; + osmo_gprs_sm_prim_sndcp_up_cb sm_sndcp_up_cb; + void *sm_sndcp_up_cb_user_data; + osmo_gprs_sm_prim_down_cb sm_down_cb; void *sm_down_cb_user_data; @@ -149,6 +152,7 @@ int gprs_sm_prim_call_up_cb(struct osmo_gprs_sm_prim *sm_prim); int gprs_sm_prim_call_down_cb(struct osmo_gprs_sm_prim *sm_prim); int gprs_sm_prim_call_gmm_down_cb(struct osmo_gprs_gmm_prim *gmm_prim); +int gprs_sm_prim_call_sndcp_up_cb(struct osmo_gprs_sndcp_prim *sndcp_prim); struct osmo_gprs_sm_prim *gprs_sm_prim_alloc_smreg_pdp_act_cnf(void); struct osmo_gprs_sm_prim *gprs_sm_prim_alloc_smreg_pdp_act_ind(void); @@ -157,6 +161,7 @@ struct gprs_sm_ms *gprs_sm_ms_alloc(uint32_t ms_id); void gprs_sm_ms_free(struct gprs_sm_ms *ms); struct gprs_sm_ms *gprs_sm_find_ms_by_id(uint32_t ms_id); +struct gprs_sm_ms *gprs_sm_find_ms_by_tlli(uint32_t tlli); struct gprs_sm_entity *gprs_sm_entity_alloc(struct gprs_sm_ms *ms, uint32_t nsapi); void gprs_sm_entity_free(struct gprs_sm_entity *sme); @@ -164,6 +169,7 @@ int gprs_sm_submit_gmmsm_assign_req(const struct gprs_sm_entity *sme); int gprs_sm_submit_smreg_pdp_act_cnf(const struct gprs_sm_entity *sme, enum gsm48_gsm_cause cause); +int gprs_sm_submit_snsm_act_ind(const struct gprs_sm_entity *sme); int gprs_sm_tx_act_pdp_ctx_req(struct gprs_sm_entity *sme); int gprs_sm_rx(struct gprs_sm_entity *sme, struct gsm48_hdr *gh, unsigned int len); diff --git a/libosmo-gprs-sm.pc.in b/libosmo-gprs-sm.pc.in index 85158f1..95d5b2b 100644 --- a/libosmo-gprs-sm.pc.in +++ b/libosmo-gprs-sm.pc.in @@ -6,7 +6,7 @@ Name: Osmocom [E]GPRS SM (Session Management) Library Description: C Utility Library Version: @VERSION@ -Requires: libosmo-gprs-gmm libosmocore +Requires: libosmo-gprs-gmm libosmo-gprs-sndcp libosmocore Libs: -L${libdir} -losmo-gprs-sm Libs.private: -ltalloc Cflags: -I${includedir}/ diff --git a/src/sm/Makefile.am b/src/sm/Makefile.am index 8fb9090..ecd8ba3 100644 --- a/src/sm/Makefile.am +++ b/src/sm/Makefile.am @@ -34,6 +34,7 @@ libosmo_gprs_sm_la_LIBADD = \ $(top_builddir)/src/gmm/libosmo-gprs-gmm.la \ + $(top_builddir)/src/sndcp/libosmo-gprs-sndcp.la \ $(LIBOSMOCORE_LIBS) \ $(LIBOSMOGSM_LIBS) \ -lm \ diff --git a/src/sm/sm.c b/src/sm/sm.c index c83e739..b3d65da 100644 --- a/src/sm/sm.c +++ b/src/sm/sm.c @@ -27,6 +27,8 @@ #include <osmocom/core/tdef.h> #include <osmocom/gsm/protocol/gsm_04_08_gprs.h> +#include <osmocom/gprs/sndcp/sndcp_prim.h> + #include <osmocom/gprs/sm/sm.h> #include <osmocom/gprs/sm/sm_prim.h> #include <osmocom/gprs/sm/sm_private.h> @@ -120,6 +122,17 @@ return NULL; } +struct gprs_sm_ms *gprs_sm_find_ms_by_tlli(uint32_t tlli) +{ + struct gprs_sm_ms *ms; + + llist_for_each_entry(ms, &g_sm_ctx->ms_list, list) { + if (ms->gmm.ptmsi == tlli) + return ms; + } + return NULL; +} + struct gprs_sm_entity *gprs_sm_entity_alloc(struct gprs_sm_ms *ms, uint32_t nsapi) { struct gprs_sm_entity *sme; @@ -202,6 +215,23 @@ return rc; } + +int gprs_sm_submit_snsm_act_ind(const struct gprs_sm_entity *sme) +{ + struct osmo_gprs_sndcp_prim *sndcp_prim_tx; + int rc; + + sndcp_prim_tx = osmo_gprs_sndcp_prim_alloc_snsm_activate_ind( + sme->ms->gmm.ptmsi, + sme->nsapi, + sme->llc_sapi); + //sndcp_prim_tx->snsm.activat_ind.qos_params = ; /* TODO */ + //sndcp_prim_tx->snsm.activat_ind.radio_prio = 0; /* TODO */ + + rc = gprs_sm_prim_call_sndcp_up_cb(sndcp_prim_tx); + return rc; +} + /* Tx SM Activate PDP context request, 9.5.1 */ int gprs_sm_tx_act_pdp_ctx_req(struct gprs_sm_entity *sme) { diff --git a/src/sm/sm_ms_fsm.c b/src/sm/sm_ms_fsm.c index 92137c4..297e81c 100644 --- a/src/sm/sm_ms_fsm.c +++ b/src/sm/sm_ms_fsm.c @@ -75,6 +75,13 @@ gprs_sm_submit_smreg_pdp_act_cnf(ctx->sme, *((enum gsm48_gsm_cause *)data)); break; case GPRS_SM_MS_EV_RX_ACT_PDP_CTX_ACC: + gprs_sm_submit_snsm_act_ind(ctx->sme); + /* Submitting SMREG-PDP-ACT-CNF is delayed until , + * SNSM-ACTIVATE-RSP (GPRS_SM_MS_EV_NSAPI_ACTIVATED) is received + * from SNDCP, see TS 24.007 C.6 */ + break; + case GPRS_SM_MS_EV_NSAPI_ACTIVATED: + /* see TS 24.007 C.6 */ sm_ms_fsm_state_chg(fi, GPRS_SM_MS_ST_PDP_ACTIVE); gprs_sm_submit_smreg_pdp_act_cnf(ctx->sme, (enum gsm48_gsm_cause)0); break; @@ -176,7 +183,8 @@ X(GPRS_SM_MS_EV_RX_GMM_ESTABLISH_CNF) | X(GPRS_SM_MS_EV_RX_GMM_ESTABLISH_REJ) | X(GPRS_SM_MS_EV_RX_ACT_PDP_CTX_REJ) | - X(GPRS_SM_MS_EV_RX_ACT_PDP_CTX_ACC), + X(GPRS_SM_MS_EV_RX_ACT_PDP_CTX_ACC) | + X(GPRS_SM_MS_EV_NSAPI_ACTIVATED), .out_state_mask = X(GPRS_SM_MS_ST_PDP_INACTIVE) | X(GPRS_SM_MS_ST_PDP_ACTIVE_PENDING) | @@ -225,6 +233,7 @@ { GPRS_SM_MS_EV_TX_ACT_PDP_CTX_REQ, "TX_ACT_PDP_CTX_REQ" }, { GPRS_SM_MS_EV_RX_ACT_PDP_CTX_REJ, "RX_ACT_PDP_CTX_REJ" }, { GPRS_SM_MS_EV_RX_ACT_PDP_CTX_ACC, "RX_ACT_PDP_CTX_ACC" }, + { GPRS_SM_MS_EV_NSAPI_ACTIVATED, "NSAPI_ACTIVATED" }, { GPRS_SM_MS_EV_TX_DEACT_PDP_CTX_REQ, "TX_DEACT_PDP_CTX_REQ" }, { GPRS_SM_MS_EV_RX_DEACT_PDP_CTX_REQ, "RX_DEACT_PDP_CTX_REQ" }, { GPRS_SM_MS_EV_RX_DEACT_PDP_CTX_ACC, "RX_DEACT_PDP_CTX_ACC" }, diff --git a/src/sm/sm_prim.c b/src/sm/sm_prim.c index 33d15ba..6ec8af8 100644 --- a/src/sm/sm_prim.c +++ b/src/sm/sm_prim.c @@ -30,6 +30,7 @@ #include <osmocom/gprs/sm/sm_prim.h> #include <osmocom/gprs/sm/sm_private.h> #include <osmocom/gprs/gmm/gmm_prim.h> +#include <osmocom/gprs/sndcp/sndcp_prim.h> #define SM_MSGB_HEADROOM 0 @@ -72,6 +73,12 @@ return 0; } +static int sm_sndcp_up_cb_dummy(struct osmo_gprs_sndcp_prim *sndcp_prim, void *user_data) +{ + LOGSM(LOGL_INFO, "%s(%s)\n", __func__, osmo_gprs_sndcp_prim_name(sndcp_prim)); + return 0; +} + static int sm_down_cb_dummy(struct osmo_gprs_sm_prim *sm_prim, void *user_data) { LOGSM(LOGL_INFO, "%s(%s)\n", __func__, osmo_gprs_sm_prim_name(sm_prim)); @@ -91,6 +98,13 @@ g_sm_ctx->sm_up_cb_user_data = up_user_data; } +/* Set callback used by SM layer to push primitives to SNDCP higher layer in protocol stack */ +void osmo_gprs_sm_prim_set_sndcp_up_cb(osmo_gprs_sm_prim_sndcp_up_cb sndcp_up_cb, void *sndcp_up_user_data) +{ + g_sm_ctx->sm_sndcp_up_cb = sndcp_up_cb; + g_sm_ctx->sm_sndcp_up_cb_user_data = sndcp_up_user_data; +} + /* Set callback used by SM layer to push primitives to lower layers in protocol stack */ void osmo_gprs_sm_prim_set_down_cb(osmo_gprs_sm_prim_down_cb down_cb, void *down_user_data) { @@ -186,6 +200,13 @@ return -ENOTSUP; } +static int gprs_sm_prim_handle_sndcp_unsupported(struct osmo_gprs_sndcp_prim *sndcp_prim) +{ + LOGSM(LOGL_ERROR, "Unsupported sndcp_prim! %s\n", osmo_gprs_sndcp_prim_name(sndcp_prim)); + msgb_free(sndcp_prim->oph.msg); + return -ENOTSUP; +} + /******************************** * Handling from/to upper layers: ********************************/ @@ -297,6 +318,93 @@ return rc; } +/* SM layer pushes SNDCP primitive up to higher layers (SNSM): */ +int gprs_sm_prim_call_sndcp_up_cb(struct osmo_gprs_sndcp_prim *sndcp_prim) +{ + int rc; + if (g_sm_ctx->sm_sndcp_up_cb) + rc = g_sm_ctx->sm_sndcp_up_cb(sndcp_prim, g_sm_ctx->sm_sndcp_up_cb_user_data); + else + rc = sm_sndcp_up_cb_dummy(sndcp_prim, g_sm_ctx->sm_sndcp_up_cb_user_data); + /* Special return value '1' means: do not free */ + if (rc != 1) + msgb_free(sndcp_prim->oph.msg); + else + rc = 0; + return rc; +} + +/* TS 24.007 6.6.1.1 SMREG-Attach.request:*/ +static int gprs_sm_prim_handle_snsm_act_resp(struct osmo_gprs_sndcp_prim *sndcp_prim) +{ + int rc; + struct gprs_sm_ms *ms; + struct gprs_sm_entity *sme; + + ms = gprs_sm_find_ms_by_tlli(sndcp_prim->snsm.tlli); + if (!ms) { + LOGSM(LOGL_ERROR, "Rx %s: Unable to find MS with TLLI=0x%08x\n", + osmo_gprs_sndcp_prim_name(sndcp_prim), sndcp_prim->snsm.tlli); + return -ENOENT; + } + + sme = gprs_sm_ms_get_pdp_ctx(ms, sndcp_prim->snsm.activate_rsp.nsapi); + if (!sme) { + LOGMS(ms, LOGL_ERROR, "Rx %s: Unable to find NSAPI=%u\n", + osmo_gprs_sndcp_prim_name(sndcp_prim), + sndcp_prim->snsm.activate_rsp.nsapi); + return -ENOENT; + } + + rc = osmo_fsm_inst_dispatch(sme->ms_fsm.fi, GPRS_SM_MS_EV_NSAPI_ACTIVATED, NULL); + return rc; +} + +/* SNDCP higher layers push SNDCP primitive (SNSM) down to SM layer: */ +static int gprs_sm_prim_handle_sndcp_snsm(struct osmo_gprs_sndcp_prim *sndcp_prim) +{ + int rc; + + switch (OSMO_PRIM_HDR(&sndcp_prim->oph)) { + case OSMO_PRIM(OSMO_GPRS_SNDCP_SNSM_ACTIVATE, PRIM_OP_RESPONSE): + rc = gprs_sm_prim_handle_snsm_act_resp(sndcp_prim); + break; + case OSMO_PRIM(OSMO_GPRS_SNDCP_SNSM_DEACTIVATE, PRIM_OP_RESPONSE): + case OSMO_PRIM(OSMO_GPRS_SNDCP_SNSM_MODIFY, PRIM_OP_RESPONSE): + case OSMO_PRIM(OSMO_GPRS_SNDCP_SNSM_STATUS, PRIM_OP_REQUEST): + case OSMO_PRIM(OSMO_GPRS_SNDCP_SNSM_SEQUENCE, PRIM_OP_RESPONSE): + case OSMO_PRIM(OSMO_GPRS_SNDCP_SNSM_STOP_ASSIGN, PRIM_OP_RESPONSE): + default: + rc = gprs_sm_prim_handle_sndcp_unsupported(sndcp_prim); + } + return rc; +} + +/* SM higher layers push SM primitive down to SM layer: */ +int osmo_gprs_sm_prim_sndcp_upper_down(struct osmo_gprs_sndcp_prim *sndcp_prim) +{ + int rc; + + LOGSM(LOGL_INFO, "Rx from SNDCP layer: %s\n", osmo_gprs_sndcp_prim_name(sndcp_prim)); + + + switch (sndcp_prim->oph.sap) { + case OSMO_GPRS_SNDCP_SAP_SNSM: + rc = gprs_sm_prim_handle_sndcp_snsm(sndcp_prim); + break; + default: + rc = gprs_sm_prim_handle_sndcp_unsupported(sndcp_prim); + rc = 1; + } + + /* Special return value '1' means: do not free */ + if (rc != 1) + msgb_free(sndcp_prim->oph.msg); + else + rc = 0; + return rc; +} + /******************************** * Handling from/to lower layers: ********************************/ diff --git a/tests/sm/Makefile.am b/tests/sm/Makefile.am index f9df676..37d2497 100644 --- a/tests/sm/Makefile.am +++ b/tests/sm/Makefile.am @@ -26,6 +26,7 @@ sm_prim_test_SOURCES = sm_prim_test.c sm_prim_test_LDADD = \ $(top_builddir)/src/sm/libosmo-gprs-sm.la \ + $(top_builddir)/src/sndcp/.libs/libosmo-gprs-sndcp.a \ $(top_builddir)/src/gmm/.libs/libosmo-gprs-gmm.a \ $(top_builddir)/src/llc/.libs/libosmo-gprs-llc.la \ $(LIBOSMOGSM_LIBS) \ diff --git a/tests/sm/sm_prim_test.c b/tests/sm/sm_prim_test.c index 5703985..35613e3 100644 --- a/tests/sm/sm_prim_test.c +++ b/tests/sm/sm_prim_test.c @@ -31,6 +31,8 @@ #include <osmocom/gprs/gmm/gmm_prim.h> #include <osmocom/gprs/gmm/gmm_private.h> +#include <osmocom/gprs/sndcp/sndcp_prim.h> +#include <osmocom/gprs/sndcp/sndcp_private.h> #include <osmocom/gprs/sm/sm.h> #include <osmocom/gprs/sm/sm_prim.h> @@ -140,6 +142,28 @@ return 0; } +int test_sm_prim_sndcp_up_cb(struct osmo_gprs_sndcp_prim *sndcp_prim, void *user_data) +{ + const char *pdu_name = osmo_gprs_sndcp_prim_name(sndcp_prim); + + switch (sndcp_prim->oph.sap) { + case OSMO_GPRS_SNDCP_SAP_SNSM: + switch (OSMO_PRIM_HDR(&sndcp_prim->oph)) { + case OSMO_PRIM(OSMO_GPRS_SM_SMREG_PDP_ACTIVATE, PRIM_OP_INDICATION): + printf("%s(): Rx %s\n", __func__, pdu_name); + break; + default: + printf("%s(): Unexpected Rx %s\n", __func__, pdu_name); + OSMO_ASSERT(0) + } + break; + default: + printf("%s(): Unexpected Rx %s\n", __func__, pdu_name); + OSMO_ASSERT(0); + } + return 0; +} + int test_sm_prim_down_cb(struct osmo_gprs_sm_prim *sm_prim, void *user_data) { const char *pdu_name = osmo_gprs_sm_prim_name(sm_prim); @@ -199,6 +223,7 @@ { struct osmo_gprs_sm_prim *sm_prim; struct osmo_gprs_gmm_prim *gmm_prim; + struct osmo_gprs_sndcp_prim *sndcp_prim; int rc; uint8_t nsapi = 6; enum osmo_gprs_sm_llc_sapi llc_sapi = OSMO_GPRS_SM_LLC_SAPI_SAPI3; @@ -217,6 +242,7 @@ OSMO_ASSERT(rc == 0); osmo_gprs_sm_prim_set_up_cb(test_sm_prim_up_cb, NULL); + osmo_gprs_sm_prim_set_sndcp_up_cb(test_sm_prim_sndcp_up_cb, NULL); osmo_gprs_sm_prim_set_down_cb(test_sm_prim_down_cb, NULL); osmo_gprs_sm_prim_set_gmm_down_cb(test_sm_prim_gmm_down_cb, NULL); @@ -254,6 +280,15 @@ rc = osmo_gprs_sm_prim_gmm_lower_up(gmm_prim); OSMO_ASSERT(rc == 0); + /* SM layer will trigger SNSM-ACTIVATE.ind to SNDCP layer here. Now, + * SNDCP is expected to do XID config and once done, answer with + * SNSM-ACTIVATE.rsp: */ + + sndcp_prim = gprs_sndcp_prim_alloc_snsm_activate_rsp(ptmsi, nsapi); + OSMO_ASSERT(sndcp_prim); + rc = osmo_gprs_sm_prim_sndcp_upper_down(sndcp_prim); + OSMO_ASSERT(rc == 0); + printf("==== %s() [end] ====\n", __func__); } diff --git a/tests/sm/sm_prim_test.err b/tests/sm/sm_prim_test.err index d9dc296..0961fc1 100644 --- a/tests/sm/sm_prim_test.err +++ b/tests/sm/sm_prim_test.err @@ -8,4 +8,6 @@ DLGLOBAL INFO Rx from lower layers: GMMSM-UNITDATA.indication DLGLOBAL INFO PDP(ID-0:NSAPI-6) Rx SM Activate PDP Context Accept DLGLOBAL INFO SM_MS{PDP_ACTIVE_PENDING}: Received Event RX_ACT_PDP_CTX_ACC +DLGLOBAL INFO Rx from SNDCP layer: SNSM-ACTIVATE.response +DLGLOBAL INFO SM_MS{PDP_ACTIVE_PENDING}: Received Event NSAPI_ACTIVATED DLGLOBAL INFO SM_MS{PDP_ACTIVE_PENDING}: state_chg to PDP_ACTIVE diff --git a/tests/sm/sm_prim_test.ok b/tests/sm/sm_prim_test.ok index 82d4a97..6909031 100644 --- a/tests/sm/sm_prim_test.ok +++ b/tests/sm/sm_prim_test.ok @@ -1,5 +1,6 @@ ==== test_sm_prim_ms() [start] ==== test_sm_prim_gmm_down_cb(): Rx GMMSM-ESTABLISH.request sess_id=0 test_sm_prim_gmm_down_cb(): Rx GMMSM-UNITDATA.request sess_id=0 SMPDU=[8a 41 06 03 01 00 06 10 21 00 00 00 00 28 04 27 01 00 ] +test_sm_prim_sndcp_up_cb(): Rx SNSM-ACTIVATE.indication test_sm_prim_up_cb(): Rx SMREG-PDP_ACTIVATE.confirm ==== test_sm_prim_ms() [end] ==== -- To view, visit https://gerrit.osmocom.org/c/libosmo-gprs/+/32295 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: libosmo-gprs Gerrit-Branch: master Gerrit-Change-Id: Ic5525bfa92e2591757d999610c0d59849d022d8d Gerrit-Change-Number: 32295 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-MessageType: newchange

1 0

[L] Change in libosmo-gprs[master]: Rename internal g_ctx -> g_{layer_name}_ctx
by pespin 13 Apr '23

13 Apr '23

pespin has uploaded this change for review. ( https://gerrit.osmocom.org/c/libosmo-gprs/+/32293 ) Change subject: Rename internal g_ctx -> g_{layer_name}_ctx ...................................................................... Rename internal g_ctx -> g_{layer_name}_ctx Otherwise if a test includes {layer_name}_private.h from several libraries to use private APIs it will fail due to colliding g_ctx pointers of different types. Change-Id: Ib8a844551452b79bdb3a81968ffc1fd42ee1c5d9 --- M include/osmocom/gprs/gmm/gmm_private.h M include/osmocom/gprs/llc/llc_private.h M include/osmocom/gprs/rlcmac/rlcmac_private.h M include/osmocom/gprs/sm/sm_private.h M include/osmocom/gprs/sndcp/sndcp_private.h M src/gmm/gmm.c M src/gmm/gmm_ms_fsm.c M src/gmm/gmm_prim.c M src/llc/llc.c M src/llc/llc_ll.c M src/llc/llc_llgmm.c M src/llc/llc_prim.c M src/rlcmac/gre.c M src/rlcmac/rlcmac.c M src/rlcmac/rlcmac_prim.c M src/rlcmac/sched.c M src/rlcmac/tbf.c M src/rlcmac/tbf_dl.c M src/rlcmac/tbf_dl_ass_fsm.c M src/rlcmac/tbf_dl_fsm.c M src/rlcmac/tbf_ul.c M src/rlcmac/tbf_ul_ass_fsm.c M src/rlcmac/tbf_ul_fsm.c M src/sm/sm.c M src/sm/sm_ms_fsm.c M src/sm/sm_prim.c M src/sndcp/sndcp.c M src/sndcp/sndcp_prim.c 28 files changed, 250 insertions(+), 237 deletions(-) git pull ssh://gerrit.osmocom.org:29418/libosmo-gprs refs/changes/93/32293/1 diff --git a/include/osmocom/gprs/gmm/gmm_private.h b/include/osmocom/gprs/gmm/gmm_private.h index 473c670..21c7467 100644 --- a/include/osmocom/gprs/gmm/gmm_private.h +++ b/include/osmocom/gprs/gmm/gmm_private.h @@ -46,7 +46,7 @@ struct llist_head gmme_list; /* list of struct gprs_gmm_entity->list */ }; -extern struct gprs_gmm_ctx *g_ctx; +extern struct gprs_gmm_ctx *g_gmm_ctx; /* GMM Entity: */ struct gprs_gmm_entity { diff --git a/include/osmocom/gprs/llc/llc_private.h b/include/osmocom/gprs/llc/llc_private.h index 1db33b6..3f7ee15 100644 --- a/include/osmocom/gprs/llc/llc_private.h +++ b/include/osmocom/gprs/llc/llc_private.h @@ -273,7 +273,7 @@ struct llist_head llme_list; }; -extern struct gprs_llc_ctx *g_ctx; +extern struct gprs_llc_ctx *g_llc_ctx; /* llc_bssgp.c */ int gprs_llc_prim_lower_up_bssgp(struct osmo_gprs_llc_prim *llc_prim); @@ -367,7 +367,7 @@ /* 6.2.2 Command/Response bit (C/R) */ static inline bool gprs_llc_received_cr_is_cmd(uint8_t cr) { - if (g_ctx->location == OSMO_GPRS_LLC_LOCATION_SGSN) + if (g_llc_ctx->location == OSMO_GPRS_LLC_LOCATION_SGSN) return !cr; /*received from MS */ else return !!cr; /*received from SGSN */ @@ -376,7 +376,7 @@ static inline void gprs_llc_encode_is_cmd_as_cr(bool is_cmd, uint32_t *flags) { - if (g_ctx->location == OSMO_GPRS_LLC_LOCATION_SGSN) + if (g_llc_ctx->location == OSMO_GPRS_LLC_LOCATION_SGSN) *flags |= OSMO_GPRS_LLC_PDU_F_CMD_RSP; /*Transmit to MS */ else *flags &= ~OSMO_GPRS_LLC_PDU_F_CMD_RSP; /* Transmit to SGSN */ diff --git a/include/osmocom/gprs/rlcmac/rlcmac_private.h b/include/osmocom/gprs/rlcmac/rlcmac_private.h index cd03066..da7ba67 100644 --- a/include/osmocom/gprs/rlcmac/rlcmac_private.h +++ b/include/osmocom/gprs/rlcmac/rlcmac_private.h @@ -82,7 +82,7 @@ SI13_RestOctets_t si13ro; }; -extern struct gprs_rlcmac_ctx *g_ctx; +extern struct gprs_rlcmac_ctx *g_rlcmac_ctx; /* rlcmac.c */ struct gprs_rlcmac_entity *gprs_rlcmac_find_entity_by_tlli(uint32_t tlli); diff --git a/include/osmocom/gprs/sm/sm_private.h b/include/osmocom/gprs/sm/sm_private.h index b0043e6..58b4a61 100644 --- a/include/osmocom/gprs/sm/sm_private.h +++ b/include/osmocom/gprs/sm/sm_private.h @@ -89,7 +89,7 @@ uint32_t next_sess_id; }; -extern struct gprs_sm_ctx *g_ctx; +extern struct gprs_sm_ctx *g_sm_ctx; /* SM Entity, PDP CTX */ struct gprs_sm_entity { diff --git a/include/osmocom/gprs/sndcp/sndcp_private.h b/include/osmocom/gprs/sndcp/sndcp_private.h index 37fac23..6f4da3b 100644 --- a/include/osmocom/gprs/sndcp/sndcp_private.h +++ b/include/osmocom/gprs/sndcp/sndcp_private.h @@ -97,7 +97,7 @@ struct gprs_sndcp_ctx_cfg cfg; }; -extern struct gprs_sndcp_ctx *g_ctx; +extern struct gprs_sndcp_ctx *g_sndcp_ctx; /* A fragment queue header, maintaining list of fragments for one N-PDU */ diff --git a/src/gmm/gmm.c b/src/gmm/gmm.c index e97b398..fbe7d40 100644 --- a/src/gmm/gmm.c +++ b/src/gmm/gmm.c @@ -34,7 +34,7 @@ #include <osmocom/gprs/gmm/gmm_pdu.h> -struct gprs_gmm_ctx *g_ctx; +struct gprs_gmm_ctx *g_gmm_ctx; /* Section 11.2.2 / Table 11.3 GPRS Mobility management timers - MS side */ #define GSM0408_T3310_SECS 15 @@ -82,10 +82,10 @@ { struct gprs_gmm_entity *gmme; - while ((gmme = llist_first_entry_or_null(&g_ctx->gmme_list, struct gprs_gmm_entity, list))) + while ((gmme = llist_first_entry_or_null(&g_gmm_ctx->gmme_list, struct gprs_gmm_entity, list))) gprs_gmm_gmme_free(gmme); - talloc_free(g_ctx); + talloc_free(g_gmm_ctx); } int osmo_gprs_gmm_init(enum osmo_gprs_gmm_location location) @@ -94,22 +94,22 @@ int rc; OSMO_ASSERT(location == OSMO_GPRS_GMM_LOCATION_MS || location == OSMO_GPRS_GMM_LOCATION_NETWORK) - if (g_ctx) { + if (g_gmm_ctx) { first_init = false; gprs_gmm_ctx_free(); } - g_ctx = talloc_zero(NULL, struct gprs_gmm_ctx); - g_ctx->location = location; - g_ctx->T_defs = T_defs_gmm; - INIT_LLIST_HEAD(&g_ctx->gmme_list); + g_gmm_ctx = talloc_zero(NULL, struct gprs_gmm_ctx); + g_gmm_ctx->location = location; + g_gmm_ctx->T_defs = T_defs_gmm; + INIT_LLIST_HEAD(&g_gmm_ctx->gmme_list); - osmo_tdefs_reset(g_ctx->T_defs); + osmo_tdefs_reset(g_gmm_ctx->T_defs); if (first_init) { rc = gprs_gmm_ms_fsm_init(); if (rc != 0) { - TALLOC_FREE(g_ctx); + TALLOC_FREE(g_gmm_ctx); return rc; } } @@ -124,15 +124,15 @@ struct gprs_gmm_entity *gmme; int ev; - if (g_ctx->gprs_enabled == enable_gprs) + if (g_gmm_ctx->gprs_enabled == enable_gprs) return; - g_ctx->gprs_enabled = enable_gprs; + g_gmm_ctx->gprs_enabled = enable_gprs; /* Inform all existing MS: */ ev = enable_gprs ? GPRS_GMM_MS_EV_ENABLE_GPRS_MODE : GPRS_GMM_MS_EV_DISABLE_GPRS_MODE; - llist_for_each_entry(gmme, &g_ctx->gmme_list, list) + llist_for_each_entry(gmme, &g_gmm_ctx->gmme_list, list) osmo_fsm_inst_dispatch(gmme->ms_fsm.fi, ev, NULL); } @@ -140,7 +140,7 @@ { struct gprs_gmm_entity *gmme; - gmme = talloc_zero(g_ctx, struct gprs_gmm_entity); + gmme = talloc_zero(g_gmm_ctx, struct gprs_gmm_entity); if (!gmme) return NULL; @@ -151,10 +151,10 @@ /* Initialize timers to default values. They may be overwritten by the * network later on: */ - gmme->t3302 = osmo_tdef_get(g_ctx->T_defs, 3302, OSMO_TDEF_S, -1); - gmme->t3346 = osmo_tdef_get(g_ctx->T_defs, 3346, OSMO_TDEF_S, -1); + gmme->t3302 = osmo_tdef_get(g_gmm_ctx->T_defs, 3302, OSMO_TDEF_S, -1); + gmme->t3346 = osmo_tdef_get(g_gmm_ctx->T_defs, 3346, OSMO_TDEF_S, -1); - llist_add(&gmme->list, &g_ctx->gmme_list); + llist_add(&gmme->list, &g_gmm_ctx->gmme_list); return gmme; } @@ -174,7 +174,7 @@ { struct gprs_gmm_entity *gmme; - llist_for_each_entry(gmme, &g_ctx->gmme_list, list) { + llist_for_each_entry(gmme, &g_gmm_ctx->gmme_list, list) { if (gmme->ptmsi == tlli || gmme->old_ptmsi == tlli) return gmme; } diff --git a/src/gmm/gmm_ms_fsm.c b/src/gmm/gmm_ms_fsm.c index 92bf6f2..902a86f 100644 --- a/src/gmm/gmm_ms_fsm.c +++ b/src/gmm/gmm_ms_fsm.c @@ -41,7 +41,7 @@ }; #define gmm_ms_fsm_state_chg(fi, NEXT_STATE) \ - osmo_tdef_fsm_inst_state_chg(fi, NEXT_STATE, gmm_ms_fsm_timeouts, g_ctx->T_defs, -1) + osmo_tdef_fsm_inst_state_chg(fi, NEXT_STATE, gmm_ms_fsm_timeouts, g_gmm_ctx->T_defs, -1) static int reinit_attach_proc(struct gprs_gmm_ms_fsm_ctx *ctx) @@ -52,7 +52,7 @@ /* Rearm T3110 */ OSMO_ASSERT(ctx->fi->T == 3110); - val_sec = osmo_tdef_get(g_ctx->T_defs, ctx->fi->T, OSMO_TDEF_S, -1); + val_sec = osmo_tdef_get(g_gmm_ctx->T_defs, ctx->fi->T, OSMO_TDEF_S, -1); osmo_timer_schedule(&ctx->fi->timer, val_sec, 0); return gprs_gmm_tx_att_req(ctx->gmme, ctx->attach.type, @@ -397,7 +397,7 @@ return -ENODATA; /* Transition to state GMM-DEREGISTERED right away if GPRS is enabled: */ - if (g_ctx->gprs_enabled) + if (g_gmm_ctx->gprs_enabled) osmo_fsm_inst_dispatch(ctx->fi, GPRS_GMM_MS_EV_ENABLE_GPRS_MODE, NULL); return 0; diff --git a/src/gmm/gmm_prim.c b/src/gmm/gmm_prim.c index 9888e50..69e74a2 100644 --- a/src/gmm/gmm_prim.c +++ b/src/gmm/gmm_prim.c @@ -133,22 +133,22 @@ /* Set callback used by GMM layer to push primitives to higher layers in protocol stack */ void osmo_gprs_gmm_prim_set_up_cb(osmo_gprs_gmm_prim_up_cb up_cb, void *up_user_data) { - g_ctx->gmm_up_cb = up_cb; - g_ctx->gmm_up_cb_user_data = up_user_data; + g_gmm_ctx->gmm_up_cb = up_cb; + g_gmm_ctx->gmm_up_cb_user_data = up_user_data; } /* Set callback used by GMM layer to push primitives to lower layers in protocol stack */ void osmo_gprs_gmm_prim_set_down_cb(osmo_gprs_gmm_prim_down_cb down_cb, void *down_user_data) { - g_ctx->gmm_down_cb = down_cb; - g_ctx->gmm_down_cb_user_data = down_user_data; + g_gmm_ctx->gmm_down_cb = down_cb; + g_gmm_ctx->gmm_down_cb_user_data = down_user_data; } /* Set callback used by GMM layer to push primitives to LLC lower layer in protocol stack */ void osmo_gprs_gmm_prim_set_llc_down_cb(osmo_gprs_gmm_prim_llc_down_cb llc_down_cb, void *llc_down_user_data) { - g_ctx->gmm_llc_down_cb = llc_down_cb; - g_ctx->gmm_llc_down_cb_user_data = llc_down_user_data; + g_gmm_ctx->gmm_llc_down_cb = llc_down_cb; + g_gmm_ctx->gmm_llc_down_cb_user_data = llc_down_user_data; } /******************************** @@ -342,10 +342,10 @@ int gprs_gmm_prim_call_up_cb(struct osmo_gprs_gmm_prim *gmm_prim) { int rc; - if (g_ctx->gmm_up_cb) - rc = g_ctx->gmm_up_cb(gmm_prim, g_ctx->gmm_up_cb_user_data); + if (g_gmm_ctx->gmm_up_cb) + rc = g_gmm_ctx->gmm_up_cb(gmm_prim, g_gmm_ctx->gmm_up_cb_user_data); else - rc = gmm_up_cb_dummy(gmm_prim, g_ctx->gmm_up_cb_user_data); + rc = gmm_up_cb_dummy(gmm_prim, g_gmm_ctx->gmm_up_cb_user_data); msgb_free(gmm_prim->oph.msg); return rc; } @@ -356,7 +356,7 @@ int rc; struct gprs_gmm_entity *gmme; - gmme = llist_first_entry_or_null(&g_ctx->gmme_list, struct gprs_gmm_entity, list); + gmme = llist_first_entry_or_null(&g_gmm_ctx->gmme_list, struct gprs_gmm_entity, list); if (!gmme) { gmme = gprs_gmm_gmme_alloc(); OSMO_ASSERT(gmme); @@ -417,7 +417,7 @@ int rc; struct gprs_gmm_entity *gmme; - gmme = llist_first_entry_or_null(&g_ctx->gmme_list, struct gprs_gmm_entity, list); + gmme = llist_first_entry_or_null(&g_gmm_ctx->gmme_list, struct gprs_gmm_entity, list); if (!gmme) { gmme = gprs_gmm_gmme_alloc(); OSMO_ASSERT(gmme); @@ -501,10 +501,10 @@ int gprs_gmm_prim_call_down_cb(struct osmo_gprs_gmm_prim *gmm_prim) { int rc; - if (g_ctx->gmm_down_cb) - rc = g_ctx->gmm_down_cb(gmm_prim, g_ctx->gmm_down_cb_user_data); + if (g_gmm_ctx->gmm_down_cb) + rc = g_gmm_ctx->gmm_down_cb(gmm_prim, g_gmm_ctx->gmm_down_cb_user_data); else - rc = gmm_down_cb_dummy(gmm_prim, g_ctx->gmm_down_cb_user_data); + rc = gmm_down_cb_dummy(gmm_prim, g_gmm_ctx->gmm_down_cb_user_data); msgb_free(gmm_prim->oph.msg); return rc; } @@ -527,7 +527,7 @@ /* GMM lower layers (LLC) push GMM primitive up to GMM layer: */ int osmo_gprs_gmm_prim_lower_up(struct osmo_gprs_gmm_prim *gmm_prim) { - OSMO_ASSERT(g_ctx); + OSMO_ASSERT(g_gmm_ctx); OSMO_ASSERT(gmm_prim); struct msgb *msg = gmm_prim->oph.msg; int rc; @@ -554,10 +554,10 @@ int gprs_gmm_prim_call_llc_down_cb(struct osmo_gprs_llc_prim *llc_prim) { int rc; - if (g_ctx->gmm_llc_down_cb) - rc = g_ctx->gmm_llc_down_cb(llc_prim, g_ctx->gmm_llc_down_cb_user_data); + if (g_gmm_ctx->gmm_llc_down_cb) + rc = g_gmm_ctx->gmm_llc_down_cb(llc_prim, g_gmm_ctx->gmm_llc_down_cb_user_data); else - rc = gmm_llc_down_cb_dummy(llc_prim, g_ctx->gmm_llc_down_cb_user_data); + rc = gmm_llc_down_cb_dummy(llc_prim, g_gmm_ctx->gmm_llc_down_cb_user_data); /* Special return value '1' means: do not free */ if (rc != 1) msgb_free(llc_prim->oph.msg); @@ -635,7 +635,7 @@ /* GMM lower layers (LLC) push GMM primitive up to GMM layer: */ int osmo_gprs_gmm_prim_llc_lower_up(struct osmo_gprs_llc_prim *llc_prim) { - OSMO_ASSERT(g_ctx); + OSMO_ASSERT(g_gmm_ctx); OSMO_ASSERT(llc_prim); struct msgb *msg = llc_prim->oph.msg; int rc; diff --git a/src/llc/llc.c b/src/llc/llc.c index 2ad0228..e9e8e3b 100644 --- a/src/llc/llc.c +++ b/src/llc/llc.c @@ -34,7 +34,7 @@ #include <osmocom/gprs/llc/llc_private.h> -struct gprs_llc_ctx *g_ctx; +struct gprs_llc_ctx *g_llc_ctx; const struct value_string gprs_llc_llme_state_names[] = { { OSMO_GPRS_LLC_LLMS_UNASSIGNED, "UNASSIGNED" }, @@ -125,10 +125,10 @@ { struct gprs_llc_llme *llme; - while ((llme = llist_first_entry_or_null(&g_ctx->llme_list, struct gprs_llc_llme, list))) + while ((llme = llist_first_entry_or_null(&g_llc_ctx->llme_list, struct gprs_llc_llme, list))) gprs_llc_llme_free(llme); - talloc_free(g_ctx); + talloc_free(g_llc_ctx); } int osmo_gprs_llc_init(enum osmo_gprs_llc_location location, const char *cipher_plugin_path) @@ -141,12 +141,12 @@ return rc; } - if (g_ctx) + if (g_llc_ctx) gprs_llc_ctx_free(); - g_ctx = talloc_zero(NULL, struct gprs_llc_ctx); - g_ctx->location = location; - INIT_LLIST_HEAD(&g_ctx->llme_list); + g_llc_ctx = talloc_zero(NULL, struct gprs_llc_ctx); + g_llc_ctx->location = location; + INIT_LLIST_HEAD(&g_llc_ctx->llme_list); return 0; } @@ -167,7 +167,7 @@ struct gprs_llc_llme *llme; uint32_t i; - llme = talloc_zero(g_ctx, struct gprs_llc_llme); + llme = talloc_zero(g_llc_ctx, struct gprs_llc_llme); if (!llme) return NULL; @@ -180,7 +180,7 @@ for (i = 0; i < ARRAY_SIZE(llme->lle); i++) lle_init(llme, i); - llist_add(&llme->list, &g_ctx->llme_list); + llist_add(&llme->list, &g_llc_ctx->llme_list); //llme->comp.proto = gprs_sndcp_comp_alloc(llme); //llme->comp.data = gprs_sndcp_comp_alloc(llme); @@ -203,7 +203,7 @@ { struct gprs_llc_llme *llme; - llist_for_each_entry(llme, &g_ctx->llme_list, list) { + llist_for_each_entry(llme, &g_llc_ctx->llme_list, list) { if (llme->tlli == tlli || llme->old_tlli == tlli) return llme; } @@ -319,7 +319,7 @@ /* TODO: we are probably missing the ciphering enc part, see osmo-sgsn apply_gea() */ /* LLC payload is put directly below: */ - if (g_ctx->location == OSMO_GPRS_LLC_LOCATION_SGSN) + if (g_llc_ctx->location == OSMO_GPRS_LLC_LOCATION_SGSN) llc_prim = gprs_llc_prim_alloc_bssgp_dl_unitdata_req(lle->llme->tlli, NULL, 4096 - sizeof(llc_prim)); else llc_prim = gprs_llc_prim_alloc_grr_unitdata_req(lle->llme->tlli, NULL, 4096 - sizeof(llc_prim)); @@ -333,7 +333,7 @@ return rc; } - if (g_ctx->location == OSMO_GPRS_LLC_LOCATION_SGSN) { + if (g_llc_ctx->location == OSMO_GPRS_LLC_LOCATION_SGSN) { llc_prim->bssgp.ll_pdu = msgb_l3(msg); llc_prim->bssgp.ll_pdu_len = msgb_l3len(msg); } else { diff --git a/src/llc/llc_ll.c b/src/llc/llc_ll.c index 51038af..72ef889 100644 --- a/src/llc/llc_ll.c +++ b/src/llc/llc_ll.c @@ -244,18 +244,18 @@ int rc; switch (OSMO_PRIM_HDR(&llc_prim->oph)) { case OSMO_PRIM(OSMO_GPRS_LLC_LL_XID, PRIM_OP_REQUEST): - OSMO_ASSERT(g_ctx->location == OSMO_GPRS_LLC_LOCATION_MS || - g_ctx->location == OSMO_GPRS_LLC_LOCATION_SGSN); + OSMO_ASSERT(g_llc_ctx->location == OSMO_GPRS_LLC_LOCATION_MS || + g_llc_ctx->location == OSMO_GPRS_LLC_LOCATION_SGSN); rc = llc_prim_handle_ll_xid_req(llc_prim); break; case OSMO_PRIM(OSMO_GPRS_LLC_LL_XID, PRIM_OP_RESPONSE): - OSMO_ASSERT(g_ctx->location == OSMO_GPRS_LLC_LOCATION_MS || - g_ctx->location == OSMO_GPRS_LLC_LOCATION_SGSN); + OSMO_ASSERT(g_llc_ctx->location == OSMO_GPRS_LLC_LOCATION_MS || + g_llc_ctx->location == OSMO_GPRS_LLC_LOCATION_SGSN); rc = llc_prim_handle_ll_xid_resp(llc_prim); break; case OSMO_PRIM(OSMO_GPRS_LLC_LL_UNITDATA, PRIM_OP_REQUEST): - OSMO_ASSERT(g_ctx->location == OSMO_GPRS_LLC_LOCATION_MS || - g_ctx->location == OSMO_GPRS_LLC_LOCATION_SGSN); + OSMO_ASSERT(g_llc_ctx->location == OSMO_GPRS_LLC_LOCATION_MS || + g_llc_ctx->location == OSMO_GPRS_LLC_LOCATION_SGSN); rc = llc_prim_handle_ll_unitdata_req(llc_prim); break; /* TODO: others */ diff --git a/src/llc/llc_llgmm.c b/src/llc/llc_llgmm.c index 9797456..cd8c189 100644 --- a/src/llc/llc_llgmm.c +++ b/src/llc/llc_llgmm.c @@ -244,40 +244,40 @@ int rc; switch (OSMO_PRIM_HDR(&llc_prim->oph)) { case OSMO_PRIM(OSMO_GPRS_LLC_LLGMM_ASSIGN, PRIM_OP_REQUEST): - OSMO_ASSERT(g_ctx->location == OSMO_GPRS_LLC_LOCATION_MS || - g_ctx->location == OSMO_GPRS_LLC_LOCATION_SGSN); + OSMO_ASSERT(g_llc_ctx->location == OSMO_GPRS_LLC_LOCATION_MS || + g_llc_ctx->location == OSMO_GPRS_LLC_LOCATION_SGSN); rc = llc_prim_handle_llgm_assign_req(llc_prim); break; case OSMO_PRIM(OSMO_GPRS_LLC_LLGMM_RESET, PRIM_OP_REQUEST): - OSMO_ASSERT(g_ctx->location == OSMO_GPRS_LLC_LOCATION_SGSN); + OSMO_ASSERT(g_llc_ctx->location == OSMO_GPRS_LLC_LOCATION_SGSN); rc = llc_prim_handle_llgm_reset_req(llc_prim); break; case OSMO_PRIM(OSMO_GPRS_LLC_LLGMM_TRIGGER, PRIM_OP_REQUEST): - OSMO_ASSERT(g_ctx->location == OSMO_GPRS_LLC_LOCATION_MS); + OSMO_ASSERT(g_llc_ctx->location == OSMO_GPRS_LLC_LOCATION_MS); rc = gprs_llc_prim_handle_unsupported(llc_prim); break; case OSMO_PRIM(OSMO_GPRS_LLC_LLGMM_SUSPEND, PRIM_OP_REQUEST): - OSMO_ASSERT(g_ctx->location == OSMO_GPRS_LLC_LOCATION_MS || - g_ctx->location == OSMO_GPRS_LLC_LOCATION_SGSN); + OSMO_ASSERT(g_llc_ctx->location == OSMO_GPRS_LLC_LOCATION_MS || + g_llc_ctx->location == OSMO_GPRS_LLC_LOCATION_SGSN); rc = gprs_llc_prim_handle_unsupported(llc_prim); break; case OSMO_PRIM(OSMO_GPRS_LLC_LLGMM_RESUME, PRIM_OP_REQUEST): - OSMO_ASSERT(g_ctx->location == OSMO_GPRS_LLC_LOCATION_MS || - g_ctx->location == OSMO_GPRS_LLC_LOCATION_SGSN); + OSMO_ASSERT(g_llc_ctx->location == OSMO_GPRS_LLC_LOCATION_MS || + g_llc_ctx->location == OSMO_GPRS_LLC_LOCATION_SGSN); rc = gprs_llc_prim_handle_unsupported(llc_prim); break; case OSMO_PRIM(OSMO_GPRS_LLC_LLGMM_IOV, PRIM_OP_REQUEST): - OSMO_ASSERT(g_ctx->location == OSMO_GPRS_LLC_LOCATION_SGSN); + OSMO_ASSERT(g_llc_ctx->location == OSMO_GPRS_LLC_LOCATION_SGSN); rc = gprs_llc_prim_handle_unsupported(llc_prim); break; case OSMO_PRIM(OSMO_GPRS_LLC_LLGMM_PSHO, PRIM_OP_REQUEST): - OSMO_ASSERT(g_ctx->location == OSMO_GPRS_LLC_LOCATION_MS || - g_ctx->location == OSMO_GPRS_LLC_LOCATION_SGSN); + OSMO_ASSERT(g_llc_ctx->location == OSMO_GPRS_LLC_LOCATION_MS || + g_llc_ctx->location == OSMO_GPRS_LLC_LOCATION_SGSN); rc = gprs_llc_prim_handle_unsupported(llc_prim); break; case OSMO_PRIM(OSMO_GPRS_LLC_LLGMM_ASSIGN_UP, PRIM_OP_REQUEST): - OSMO_ASSERT(g_ctx->location == OSMO_GPRS_LLC_LOCATION_MS || - g_ctx->location == OSMO_GPRS_LLC_LOCATION_SGSN); + OSMO_ASSERT(g_llc_ctx->location == OSMO_GPRS_LLC_LOCATION_MS || + g_llc_ctx->location == OSMO_GPRS_LLC_LOCATION_SGSN); rc = gprs_llc_prim_handle_unsupported(llc_prim); break; default: diff --git a/src/llc/llc_prim.c b/src/llc/llc_prim.c index d653f03..4b520ad 100644 --- a/src/llc/llc_prim.c +++ b/src/llc/llc_prim.c @@ -83,15 +83,15 @@ /* Set callback used by LLC layer to push primitives to higher layers in protocol stack */ void osmo_gprs_llc_prim_set_up_cb(osmo_gprs_llc_prim_up_cb up_cb, void *up_user_data) { - g_ctx->llc_up_cb = up_cb; - g_ctx->llc_up_cb_user_data = up_user_data; + g_llc_ctx->llc_up_cb = up_cb; + g_llc_ctx->llc_up_cb_user_data = up_user_data; } /* Set callback used by LLC layer to push primitives to lower layers in protocol stack */ void osmo_gprs_llc_prim_set_down_cb(osmo_gprs_llc_prim_down_cb down_cb, void *down_user_data) { - g_ctx->llc_down_cb = down_cb; - g_ctx->llc_down_cb_user_data = down_user_data; + g_llc_ctx->llc_down_cb = down_cb; + g_llc_ctx->llc_down_cb_user_data = down_user_data; } /******************************** @@ -138,10 +138,10 @@ int gprs_llc_prim_call_up_cb(struct osmo_gprs_llc_prim *llc_prim) { int rc; - if (g_ctx->llc_up_cb) - rc = g_ctx->llc_up_cb(llc_prim, g_ctx->llc_up_cb_user_data); + if (g_llc_ctx->llc_up_cb) + rc = g_llc_ctx->llc_up_cb(llc_prim, g_llc_ctx->llc_up_cb_user_data); else - rc = llc_up_cb_dummy(llc_prim, g_ctx->llc_up_cb_user_data); + rc = llc_up_cb_dummy(llc_prim, g_llc_ctx->llc_up_cb_user_data); /* Special return value '1' means: do not free */ if (rc != 1) msgb_free(llc_prim->oph.msg); @@ -152,7 +152,7 @@ int osmo_gprs_llc_prim_upper_down(struct osmo_gprs_llc_prim *llc_prim) { int rc; - OSMO_ASSERT(g_ctx); + OSMO_ASSERT(g_llc_ctx); LOGLLC(LOGL_INFO, "Rx from upper layers: %s\n", osmo_gprs_llc_prim_name(llc_prim)); @@ -176,10 +176,10 @@ int gprs_llc_prim_call_down_cb(struct osmo_gprs_llc_prim *llc_prim) { int rc; - if (g_ctx->llc_down_cb) - rc = g_ctx->llc_down_cb(llc_prim, g_ctx->llc_down_cb_user_data); + if (g_llc_ctx->llc_down_cb) + rc = g_llc_ctx->llc_down_cb(llc_prim, g_llc_ctx->llc_down_cb_user_data); else - rc = llc_down_cb_dummy(llc_prim, g_ctx->llc_down_cb_user_data); + rc = llc_down_cb_dummy(llc_prim, g_llc_ctx->llc_down_cb_user_data); /* Special return value '1' means: do not free */ if (rc != 1) msgb_free(llc_prim->oph.msg); @@ -191,7 +191,7 @@ /* LLC lower layers push LLC primitive up to LLC layer: */ int osmo_gprs_llc_prim_lower_up(struct osmo_gprs_llc_prim *llc_prim) { - OSMO_ASSERT(g_ctx); + OSMO_ASSERT(g_llc_ctx); OSMO_ASSERT(llc_prim); struct msgb *msg = llc_prim->oph.msg; int rc; @@ -200,11 +200,11 @@ switch (llc_prim->oph.sap) { case OSMO_GPRS_LLC_SAP_GRR: - OSMO_ASSERT(g_ctx->location == OSMO_GPRS_LLC_LOCATION_MS); + OSMO_ASSERT(g_llc_ctx->location == OSMO_GPRS_LLC_LOCATION_MS); rc = gprs_llc_prim_lower_up_grr(llc_prim); break; case OSMO_GPRS_LLC_SAP_BSSGP: - OSMO_ASSERT(g_ctx->location == OSMO_GPRS_LLC_LOCATION_SGSN); + OSMO_ASSERT(g_llc_ctx->location == OSMO_GPRS_LLC_LOCATION_SGSN); rc = gprs_llc_prim_lower_up_bssgp(llc_prim); break; default: diff --git a/src/rlcmac/gre.c b/src/rlcmac/gre.c index 2bab3c1..fe18a43 100644 --- a/src/rlcmac/gre.c +++ b/src/rlcmac/gre.c @@ -40,7 +40,7 @@ struct gprs_rlcmac_entity *gre; int rc; - gre = talloc_zero(g_ctx, struct gprs_rlcmac_entity); + gre = talloc_zero(g_rlcmac_ctx, struct gprs_rlcmac_entity); if (!gre) return NULL; @@ -48,15 +48,15 @@ if (!gre->llc_queue) goto err_free_gre; gprs_rlcmac_llc_queue_set_codel_params(gre->llc_queue, - g_ctx->cfg.codel.use, - g_ctx->cfg.codel.interval_msec); + g_rlcmac_ctx->cfg.codel.use, + g_rlcmac_ctx->cfg.codel.interval_msec); rc = gprs_rlcmac_tbf_dl_ass_fsm_constructor(&gre->dl_tbf_dl_ass_fsm, gre); if (rc < 0) goto err_free_gre; gre->tlli = tlli; - llist_add_tail(&gre->entry, &g_ctx->gre_list); + llist_add_tail(&gre->entry, &g_rlcmac_ctx->gre_list); return gre; diff --git a/src/rlcmac/rlcmac.c b/src/rlcmac/rlcmac.c index 574ff5a..00b45ff 100644 --- a/src/rlcmac/rlcmac.c +++ b/src/rlcmac/rlcmac.c @@ -40,7 +40,7 @@ #define GPRS_CODEL_SLOW_INTERVAL_MS 4000 -struct gprs_rlcmac_ctx *g_ctx; +struct gprs_rlcmac_ctx *g_rlcmac_ctx; /* TS 44.060 Table 13.1.1 */ static struct osmo_tdef T_defs_rlcmac[] = { @@ -56,10 +56,10 @@ { struct gprs_rlcmac_entity *gre; - while ((gre = llist_first_entry_or_null(&g_ctx->gre_list, struct gprs_rlcmac_entity, entry))) + while ((gre = llist_first_entry_or_null(&g_rlcmac_ctx->gre_list, struct gprs_rlcmac_entity, entry))) gprs_rlcmac_entity_free(gre); - talloc_free(g_ctx); + talloc_free(g_rlcmac_ctx); } int osmo_gprs_rlcmac_init(enum osmo_gprs_rlcmac_location location) @@ -69,48 +69,48 @@ unsigned int i; OSMO_ASSERT(location == OSMO_GPRS_RLCMAC_LOCATION_MS || location == OSMO_GPRS_RLCMAC_LOCATION_PCU) - if (g_ctx) { + if (g_rlcmac_ctx) { gprs_rlcmac_ctx_free(); first_init = false; } - g_ctx = talloc_zero(NULL, struct gprs_rlcmac_ctx); - g_ctx->cfg.location = location; - g_ctx->cfg.codel.use = true; - g_ctx->cfg.codel.interval_msec = GPRS_CODEL_SLOW_INTERVAL_MS; - g_ctx->cfg.egprs_arq_type = GPRS_RLCMAC_EGPRS_ARQ1; - g_ctx->cfg.ul_tbf_preemptive_retransmission = true; - g_ctx->T_defs = T_defs_rlcmac; - INIT_LLIST_HEAD(&g_ctx->gre_list); + g_rlcmac_ctx = talloc_zero(NULL, struct gprs_rlcmac_ctx); + g_rlcmac_ctx->cfg.location = location; + g_rlcmac_ctx->cfg.codel.use = true; + g_rlcmac_ctx->cfg.codel.interval_msec = GPRS_CODEL_SLOW_INTERVAL_MS; + g_rlcmac_ctx->cfg.egprs_arq_type = GPRS_RLCMAC_EGPRS_ARQ1; + g_rlcmac_ctx->cfg.ul_tbf_preemptive_retransmission = true; + g_rlcmac_ctx->T_defs = T_defs_rlcmac; + INIT_LLIST_HEAD(&g_rlcmac_ctx->gre_list); - osmo_tdefs_reset(g_ctx->T_defs); + osmo_tdefs_reset(g_rlcmac_ctx->T_defs); if (first_init) { rc = gprs_rlcmac_tbf_dl_ass_fsm_init(); if (rc != 0) { - TALLOC_FREE(g_ctx); + TALLOC_FREE(g_rlcmac_ctx); return rc; } rc = gprs_rlcmac_tbf_dl_fsm_init(); if (rc != 0) { - TALLOC_FREE(g_ctx); + TALLOC_FREE(g_rlcmac_ctx); return rc; } rc = gprs_rlcmac_tbf_ul_fsm_init(); if (rc != 0) { - TALLOC_FREE(g_ctx); + TALLOC_FREE(g_rlcmac_ctx); return rc; } rc = gprs_rlcmac_tbf_ul_ass_fsm_init(); if (rc != 0) { - TALLOC_FREE(g_ctx); + TALLOC_FREE(g_rlcmac_ctx); return rc; } } - for (i = 0; i < ARRAY_SIZE(g_ctx->sched.ulc); i++) { - g_ctx->sched.ulc[i] = gprs_rlcmac_pdch_ulc_alloc(g_ctx, i); - OSMO_ASSERT(g_ctx->sched.ulc[i]); + for (i = 0; i < ARRAY_SIZE(g_rlcmac_ctx->sched.ulc); i++) { + g_rlcmac_ctx->sched.ulc[i] = gprs_rlcmac_pdch_ulc_alloc(g_rlcmac_ctx, i); + OSMO_ASSERT(g_rlcmac_ctx->sched.ulc[i]); } return 0; @@ -126,8 +126,8 @@ if (interval_msec == 0) interval_msec = GPRS_CODEL_SLOW_INTERVAL_MS; - g_ctx->cfg.codel.use = use; - g_ctx->cfg.codel.interval_msec = interval_msec; + g_rlcmac_ctx->cfg.codel.use = use; + g_rlcmac_ctx->cfg.codel.interval_msec = interval_msec; return 0; } @@ -135,7 +135,7 @@ { struct gprs_rlcmac_entity *gre; - llist_for_each_entry(gre, &g_ctx->gre_list, entry) { + llist_for_each_entry(gre, &g_rlcmac_ctx->gre_list, entry) { if (gre->tlli == tlli) return gre; } @@ -146,7 +146,7 @@ { struct gprs_rlcmac_entity *gre; - llist_for_each_entry(gre, &g_ctx->gre_list, entry) { + llist_for_each_entry(gre, &g_rlcmac_ctx->gre_list, entry) { if (!gre->dl_tbf) continue; if (gre->dl_tbf->cur_alloc.dl_tfi != dl_tfi) @@ -160,7 +160,7 @@ { struct gprs_rlcmac_entity *gre; - llist_for_each_entry(gre, &g_ctx->gre_list, entry) { + llist_for_each_entry(gre, &g_rlcmac_ctx->gre_list, entry) { if (!gre->ul_tbf) continue; if (gre->ul_tbf->cur_alloc.ul_tfi != ul_tfi) @@ -182,7 +182,7 @@ .iaro = iaro }; - llist_for_each_entry(gre, &g_ctx->gre_list, entry) { + llist_for_each_entry(gre, &g_rlcmac_ctx->gre_list, entry) { ul_tbf = gre->ul_tbf; if (!ul_tbf) continue; @@ -309,9 +309,9 @@ int rc; LOGRLCMAC(LOGL_DEBUG, "Rx SI13 from lower layers\n"); - memcpy(g_ctx->si13, si13, GSM_MACBLOCK_LEN); + memcpy(g_rlcmac_ctx->si13, si13, GSM_MACBLOCK_LEN); - rc = osmo_gprs_rlcmac_decode_si13ro(&g_ctx->si13ro, si13->rest_octets, + rc = osmo_gprs_rlcmac_decode_si13ro(&g_rlcmac_ctx->si13ro, si13->rest_octets, GSM_MACBLOCK_LEN - offsetof(struct gsm48_system_information_type_13, rest_octets)); if (rc < 0) { LOGRLCMAC(LOGL_ERROR, "Error decoding SI13: %s\n", @@ -319,16 +319,16 @@ return rc; } - g_ctx->si13_available = true; + g_rlcmac_ctx->si13_available = true; /* Update tdef for T3168: * TS 44.060 Table 12.24.2: Range: 0 to 7. The timeout value is given as the binary value plus one in units of 500 ms. */ - osmo_tdef_set(g_ctx->T_defs, 3168, - (g_ctx->si13ro.u.PBCCH_Not_present.GPRS_Cell_Options.T3168 + 1) * 500, + osmo_tdef_set(g_rlcmac_ctx->T_defs, 3168, + (g_rlcmac_ctx->si13ro.u.PBCCH_Not_present.GPRS_Cell_Options.T3168 + 1) * 500, OSMO_TDEF_MS); /* TODO: Update tdef for T3192 as per TS 44.060 Table 12.24.2 - * osmo_tdef_set(g_ctx->T_defs, 3192, si13ro.u.PBCCH_Not_present.GPRS_Cell_Options.T3192, enum osmo_tdef_unit val_unit); + * osmo_tdef_set(g_rlcmac_ctx->T_defs, 3192, si13ro.u.PBCCH_Not_present.GPRS_Cell_Options.T3192, enum osmo_tdef_unit val_unit); */ return rc; @@ -402,7 +402,7 @@ if (dl_block->SP) { uint32_t poll_fn = rrbp2fn(rlcmac_prim->l1ctl.pdch_data_ind.fn, dl_block->RRBP); - gprs_rlcmac_pdch_ulc_reserve(g_ctx->sched.ulc[rlcmac_prim->l1ctl.pdch_data_ind.ts_nr], + gprs_rlcmac_pdch_ulc_reserve(g_rlcmac_ctx->sched.ulc[rlcmac_prim->l1ctl.pdch_data_ind.ts_nr], poll_fn, GPRS_RLCMAC_PDCH_ULC_POLL_DL_ASS, gre); @@ -429,7 +429,7 @@ /* If RRBP contains valid data, schedule a response (PKT CONTROL ACK or PKT RESOURCE REQ). */ if (dl_block->SP) { uint32_t poll_fn = rrbp2fn(rlcmac_prim->l1ctl.pdch_data_ind.fn, dl_block->RRBP); - gprs_rlcmac_pdch_ulc_reserve(g_ctx->sched.ulc[rlcmac_prim->l1ctl.pdch_data_ind.ts_nr], poll_fn, + gprs_rlcmac_pdch_ulc_reserve(g_rlcmac_ctx->sched.ulc[rlcmac_prim->l1ctl.pdch_data_ind.ts_nr], poll_fn, GPRS_RLCMAC_PDCH_ULC_POLL_UL_ACK, ul_tbf_as_tbf(ul_tbf)); } @@ -485,11 +485,11 @@ size_t max_len = gprs_rlcmac_mcs_max_bytes_dl(GPRS_RLCMAC_CS_1); int rc; - bv = bitvec_alloc(max_len, g_ctx); + bv = bitvec_alloc(max_len, g_rlcmac_ctx); OSMO_ASSERT(bv); bitvec_unpack(bv, rlcmac_prim->l1ctl.pdch_data_ind.data); - dl_ctrl_block = (RlcMacDownlink_t *)talloc_zero(g_ctx, RlcMacDownlink_t); + dl_ctrl_block = (RlcMacDownlink_t *)talloc_zero(g_rlcmac_ctx, RlcMacDownlink_t); OSMO_ASSERT(dl_ctrl_block); rc = osmo_gprs_rlcmac_decode_downlink(bv, dl_ctrl_block); if (rc < 0) { diff --git a/src/rlcmac/rlcmac_prim.c b/src/rlcmac/rlcmac_prim.c index 8f27ff7..e6f151a 100644 --- a/src/rlcmac/rlcmac_prim.c +++ b/src/rlcmac/rlcmac_prim.c @@ -112,15 +112,15 @@ /* Set callback used by LLC layer to push primitives to higher layers in protocol stack */ void osmo_gprs_rlcmac_prim_set_up_cb(osmo_gprs_rlcmac_prim_up_cb up_cb, void *up_user_data) { - g_ctx->rlcmac_up_cb = up_cb; - g_ctx->rlcmac_up_cb_user_data = up_user_data; + g_rlcmac_ctx->rlcmac_up_cb = up_cb; + g_rlcmac_ctx->rlcmac_up_cb_user_data = up_user_data; } /* Set callback used by LLC layer to push primitives to lower layers in protocol stack */ void osmo_gprs_rlcmac_prim_set_down_cb(osmo_gprs_rlcmac_prim_down_cb down_cb, void *down_user_data) { - g_ctx->rlcmac_down_cb = down_cb; - g_ctx->rlcmac_down_cb_user_data = down_user_data; + g_rlcmac_ctx->rlcmac_down_cb = down_cb; + g_rlcmac_ctx->rlcmac_down_cb_user_data = down_user_data; } /******************************** @@ -355,10 +355,10 @@ int gprs_rlcmac_prim_call_up_cb(struct osmo_gprs_rlcmac_prim *rlcmac_prim) { int rc; - if (g_ctx->rlcmac_up_cb) - rc = g_ctx->rlcmac_up_cb(rlcmac_prim, g_ctx->rlcmac_up_cb_user_data); + if (g_rlcmac_ctx->rlcmac_up_cb) + rc = g_rlcmac_ctx->rlcmac_up_cb(rlcmac_prim, g_rlcmac_ctx->rlcmac_up_cb_user_data); else - rc = rlcmac_up_cb_dummy(rlcmac_prim, g_ctx->rlcmac_up_cb_user_data); + rc = rlcmac_up_cb_dummy(rlcmac_prim, g_rlcmac_ctx->rlcmac_up_cb_user_data); /* Special return value '1' means: do not free */ if (rc != 1) msgb_free(rlcmac_prim->oph.msg); @@ -447,10 +447,10 @@ LOGRLCMAC(LOGL_DEBUG, "Tx to lower layers: %s\n", osmo_gprs_rlcmac_prim_name(rlcmac_prim)); - if (g_ctx->rlcmac_down_cb) - rc = g_ctx->rlcmac_down_cb(rlcmac_prim, g_ctx->rlcmac_down_cb_user_data); + if (g_rlcmac_ctx->rlcmac_down_cb) + rc = g_rlcmac_ctx->rlcmac_down_cb(rlcmac_prim, g_rlcmac_ctx->rlcmac_down_cb_user_data); else - rc = rlcmac_down_cb_dummy(rlcmac_prim, g_ctx->rlcmac_down_cb_user_data); + rc = rlcmac_down_cb_dummy(rlcmac_prim, g_rlcmac_ctx->rlcmac_down_cb_user_data); /* Special return value '1' means: do not free */ if (rc != 1) msgb_free(rlcmac_prim->oph.msg); @@ -551,7 +551,7 @@ int osmo_gprs_rlcmac_prim_lower_up(struct osmo_gprs_rlcmac_prim *rlcmac_prim) { - OSMO_ASSERT(g_ctx); + OSMO_ASSERT(g_rlcmac_ctx); OSMO_ASSERT(rlcmac_prim); struct msgb *msg = rlcmac_prim->oph.msg; int rc; diff --git a/src/rlcmac/sched.c b/src/rlcmac/sched.c index 50a3910..25ee6da 100644 --- a/src/rlcmac/sched.c +++ b/src/rlcmac/sched.c @@ -72,7 +72,7 @@ struct gprs_rlcmac_ul_tbf *ul_tbf; struct gprs_rlcmac_pdch_ulc_node *node; - node = gprs_rlcmac_pdch_ulc_get_node(g_ctx->sched.ulc[bi->ts], bi->fn); + node = gprs_rlcmac_pdch_ulc_get_node(g_rlcmac_ctx->sched.ulc[bi->ts], bi->fn); if (node) { switch (node->reason) { case GPRS_RLCMAC_PDCH_ULC_POLL_UL_ASS: @@ -116,11 +116,11 @@ /* TODO */ break; } - gprs_rlcmac_pdch_ulc_release_node(g_ctx->sched.ulc[bi->ts], node); + gprs_rlcmac_pdch_ulc_release_node(g_rlcmac_ctx->sched.ulc[bi->ts], node); } /* Iterate over UL TBFs: */ - llist_for_each_entry(gre, &g_ctx->gre_list, entry) { + llist_for_each_entry(gre, &g_rlcmac_ctx->gre_list, entry) { if (!gre->ul_tbf) continue; ul_tbf = gre->ul_tbf; @@ -134,7 +134,7 @@ static struct gprs_rlcmac_ul_tbf *find_requested_ul_tbf_for_data(const struct gprs_rlcmac_rts_block_ind *bi) { struct gprs_rlcmac_entity *gre; - llist_for_each_entry(gre, &g_ctx->gre_list, entry) { + llist_for_each_entry(gre, &g_rlcmac_ctx->gre_list, entry) { if (!gre->ul_tbf) continue; if (gprs_rlcmac_ul_tbf_data_rts(gre->ul_tbf, bi)) @@ -146,7 +146,7 @@ static struct gprs_rlcmac_ul_tbf *find_requested_ul_tbf_for_dummy(const struct gprs_rlcmac_rts_block_ind *bi) { struct gprs_rlcmac_entity *gre; - llist_for_each_entry(gre, &g_ctx->gre_list, entry) { + llist_for_each_entry(gre, &g_rlcmac_ctx->gre_list, entry) { if (!gre->ul_tbf) continue; if (gprs_rlcmac_ul_tbf_dummy_rts(gre->ul_tbf, bi)) @@ -268,7 +268,7 @@ { struct gprs_rlcmac_entity *gre; - llist_for_each_entry(gre, &g_ctx->gre_list, entry) { + llist_for_each_entry(gre, &g_rlcmac_ctx->gre_list, entry) { if (gre->ul_tbf && gprs_rlcmac_tbf_ul_ass_waiting_tbf_starting_time(gre->ul_tbf)) gprs_rlcmac_tbf_ul_ass_fn_tick(gre->ul_tbf, bi->fn, bi->ts); if (gprs_rlcmac_tbf_start_pending(&gre->dl_tbf_dl_ass_fsm)) @@ -307,6 +307,6 @@ msgb_free(msg); ret_rc: - gprs_rlcmac_pdch_ulc_expire_fn(g_ctx->sched.ulc[bi->ts], bi->fn); + gprs_rlcmac_pdch_ulc_expire_fn(g_rlcmac_ctx->sched.ulc[bi->ts], bi->fn); return rc; } diff --git a/src/rlcmac/tbf.c b/src/rlcmac/tbf.c index a88e924..2168801 100644 --- a/src/rlcmac/tbf.c +++ b/src/rlcmac/tbf.c @@ -36,8 +36,8 @@ void gprs_rlcmac_tbf_destructor(struct gprs_rlcmac_tbf *tbf) { unsigned int i; - for (i = 0; i < ARRAY_SIZE(g_ctx->sched.ulc); i++) - gprs_rlcmac_pdch_ulc_release_tbf(g_ctx->sched.ulc[i], tbf); + for (i = 0; i < ARRAY_SIZE(g_rlcmac_ctx->sched.ulc); i++) + gprs_rlcmac_pdch_ulc_release_tbf(g_rlcmac_ctx->sched.ulc[i], tbf); } void gprs_rlcmac_tbf_free(struct gprs_rlcmac_tbf *tbf) diff --git a/src/rlcmac/tbf_dl.c b/src/rlcmac/tbf_dl.c index 4ed0bfd..741f2a0 100644 --- a/src/rlcmac/tbf_dl.c +++ b/src/rlcmac/tbf_dl.c @@ -42,7 +42,7 @@ if (rc < 0) goto err_tbf_destruct; - dl_tbf->tbf.nr = g_ctx->next_dl_tbf_nr++; + dl_tbf->tbf.nr = g_rlcmac_ctx->next_dl_tbf_nr++; dl_tbf->dlw = gprs_rlcmac_rlc_dl_window_alloc(dl_tbf); OSMO_ASSERT(dl_tbf->dlw); @@ -298,7 +298,7 @@ /* If RRBP contains valid data, schedule a DL ACK/NACK. */ if (rlc->es_p) { uint32_t poll_fn = rrbp2fn(fn, rlc->rrbp); - gprs_rlcmac_pdch_ulc_reserve(g_ctx->sched.ulc[ts_nr], poll_fn, + gprs_rlcmac_pdch_ulc_reserve(g_rlcmac_ctx->sched.ulc[ts_nr], poll_fn, GPRS_RLCMAC_PDCH_ULC_POLL_DL_ACK, dl_tbf_as_tbf(dl_tbf)); } diff --git a/src/rlcmac/tbf_dl_ass_fsm.c b/src/rlcmac/tbf_dl_ass_fsm.c index 365772a..4bb6ae9 100644 --- a/src/rlcmac/tbf_dl_ass_fsm.c +++ b/src/rlcmac/tbf_dl_ass_fsm.c @@ -55,7 +55,7 @@ #define tbf_dl_ass_fsm_state_chg(fi, NEXT_STATE) \ osmo_tdef_fsm_inst_state_chg(fi, NEXT_STATE, \ tbf_dl_ass_fsm_timeouts, \ - g_ctx->T_defs, \ + g_rlcmac_ctx->T_defs, \ -1) static int handle_imm_ass(struct gprs_rlcmac_tbf_dl_ass_fsm_ctx *ctx, const struct tbf_start_ev_rx_ccch_imm_ass_ctx *d) diff --git a/src/rlcmac/tbf_dl_fsm.c b/src/rlcmac/tbf_dl_fsm.c index e4a4cf2..250b66c 100644 --- a/src/rlcmac/tbf_dl_fsm.c +++ b/src/rlcmac/tbf_dl_fsm.c @@ -47,7 +47,7 @@ #define tbf_dl_fsm_state_chg(fi, NEXT_STATE) \ osmo_tdef_fsm_inst_state_chg(fi, NEXT_STATE, \ tbf_dl_fsm_timeouts, \ - g_ctx->T_defs, \ + g_rlcmac_ctx->T_defs, \ -1) static void st_new(struct osmo_fsm_inst *fi, uint32_t event, void *data) diff --git a/src/rlcmac/tbf_ul.c b/src/rlcmac/tbf_ul.c index 143c880..8e03207 100644 --- a/src/rlcmac/tbf_ul.c +++ b/src/rlcmac/tbf_ul.c @@ -48,7 +48,7 @@ if (rc < 0) goto err_state_fsm_destruct; - ul_tbf->tbf.nr = g_ctx->next_ul_tbf_nr++; + ul_tbf->tbf.nr = g_rlcmac_ctx->next_ul_tbf_nr++; ul_tbf->tx_cs = GPRS_RLCMAC_CS_1; ul_tbf->ulw = gprs_rlcmac_rlc_ul_window_alloc(ul_tbf); @@ -121,8 +121,8 @@ /* 3GPP TS 44.060 13.3: * N3104_MAX = 3 * (BS_CV_MAX + 3) * number of uplink timeslots assigned */ /* If we didn't receive SI13 yet, use maximum value bs_cv_max in range 0..15 */ - uint8_t bs_cv_max = g_ctx->si13_available ? - g_ctx->si13ro.u.PBCCH_Not_present.GPRS_Cell_Options.BS_CV_MAX : + uint8_t bs_cv_max = g_rlcmac_ctx->si13_available ? + g_rlcmac_ctx->si13ro.u.PBCCH_Not_present.GPRS_Cell_Options.BS_CV_MAX : 15; return 3 * (bs_cv_max + 3) * ul_tbf->cur_alloc.num_ts; } @@ -522,7 +522,7 @@ if (ul_tbf->tbf.is_egprs) { /* Table 8.1.1.2 and Table 8.1.1.1 of 44.060 */ blk->cs_current_trans = gprs_rlcmac_get_retx_mcs(blk->cs_init, tx_cs, - g_ctx->cfg.egprs_arq_type == GPRS_RLCMAC_EGPRS_ARQ1); + g_rlcmac_ctx->cfg.egprs_arq_type == GPRS_RLCMAC_EGPRS_ARQ1); LOGPTBFUL(ul_tbf, LOGL_DEBUG, "initial_cs_dl(%s) last_mcs(%s) demanded_mcs(%s) cs_trans(%s) arq_type(%d) bsn(%d)\n", @@ -530,7 +530,7 @@ gprs_rlcmac_mcs_name(blk->cs_last), gprs_rlcmac_mcs_name(tx_cs), gprs_rlcmac_mcs_name(blk->cs_current_trans), - g_ctx->cfg.egprs_arq_type, bsn); + g_rlcmac_ctx->cfg.egprs_arq_type, bsn); /* TODO: Need to remove this check when MCS-8 -> MCS-6 * transistion is handled. @@ -571,7 +571,7 @@ force_data_len != -1 ? " (forced)" : ""); bsn = create_new_bsn(ul_tbf, bi, tx_cs); - } else if (g_ctx->cfg.ul_tbf_preemptive_retransmission && + } else if (g_rlcmac_ctx->cfg.ul_tbf_preemptive_retransmission && !gprs_rlcmac_rlc_ul_window_window_empty(ul_tbf->ulw)) { /* The window contains unacked packages, but not acked. * Mark unacked bsns as RESEND */ diff --git a/src/rlcmac/tbf_ul_ass_fsm.c b/src/rlcmac/tbf_ul_ass_fsm.c index 8b61cbe..f9ccfcb 100644 --- a/src/rlcmac/tbf_ul_ass_fsm.c +++ b/src/rlcmac/tbf_ul_ass_fsm.c @@ -66,7 +66,7 @@ #define tbf_ul_ass_fsm_state_chg(fi, NEXT_STATE) \ osmo_tdef_fsm_inst_state_chg(fi, NEXT_STATE, \ tbf_ul_ass_fsm_timeouts, \ - g_ctx->T_defs, \ + g_rlcmac_ctx->T_defs, \ -1) static struct msgb *create_pkt_resource_req(const struct gprs_rlcmac_tbf_ul_ass_fsm_ctx *ctx, @@ -363,7 +363,7 @@ /* If RRBP contains valid data, schedule a response (PKT CONTROL ACK or PKT RESOURCE REQ). */ if (d->dl_block->SP) { uint32_t poll_fn = rrbp2fn(d->fn, d->dl_block->RRBP); - gprs_rlcmac_pdch_ulc_reserve(g_ctx->sched.ulc[d->ts_nr], poll_fn, + gprs_rlcmac_pdch_ulc_reserve(g_rlcmac_ctx->sched.ulc[d->ts_nr], poll_fn, GPRS_RLCMAC_PDCH_ULC_POLL_UL_ASS, ul_tbf_as_tbf(ctx->ul_tbf)); } @@ -396,7 +396,7 @@ /* If RRBP contains valid data, schedule a response (PKT CONTROL ACK or PKT RESOURCE REQ). */ if (d->dl_block->SP) { uint32_t poll_fn = rrbp2fn(d->fn, d->dl_block->RRBP); - gprs_rlcmac_pdch_ulc_reserve(g_ctx->sched.ulc[d->ts_nr], poll_fn, + gprs_rlcmac_pdch_ulc_reserve(g_rlcmac_ctx->sched.ulc[d->ts_nr], poll_fn, GPRS_RLCMAC_PDCH_ULC_POLL_UL_ASS, ul_tbf_as_tbf(ctx->ul_tbf)); } diff --git a/src/rlcmac/tbf_ul_fsm.c b/src/rlcmac/tbf_ul_fsm.c index 9bf708c..96adf8b 100644 --- a/src/rlcmac/tbf_ul_fsm.c +++ b/src/rlcmac/tbf_ul_fsm.c @@ -53,7 +53,7 @@ #define tbf_ul_fsm_state_chg(fi, NEXT_STATE) \ osmo_tdef_fsm_inst_state_chg(fi, NEXT_STATE, \ tbf_ul_fsm_timeouts, \ - g_ctx->T_defs, \ + g_rlcmac_ctx->T_defs, \ -1) static uint8_t ul_tbf_ul_slotmask(struct gprs_rlcmac_ul_tbf *ul_tbf) @@ -110,7 +110,7 @@ (osmo_timer_pending(&fi->timer) && fi->T == 3182)); LOGPFSML(ctx->fi, LOGL_INFO, "Last UL block sent (CV=0), start T3182\n"); fi->T = 3182; - val_sec = osmo_tdef_get(g_ctx->T_defs, fi->T, OSMO_TDEF_S, -1); + val_sec = osmo_tdef_get(g_rlcmac_ctx->T_defs, fi->T, OSMO_TDEF_S, -1); osmo_timer_schedule(&fi->timer, val_sec, 0); } } @@ -169,7 +169,7 @@ if (ctx->ul_tbf->ul_ass_fsm.ass_type == GPRS_RLCMAC_TBF_UL_ASS_TYPE_1PHASE) { LOGPFSML(ctx->fi, LOGL_INFO, "First UL block sent (1 phase access), start T3166\n"); fi->T = 3166; - val_sec = osmo_tdef_get(g_ctx->T_defs, fi->T, OSMO_TDEF_S, -1); + val_sec = osmo_tdef_get(g_rlcmac_ctx->T_defs, fi->T, OSMO_TDEF_S, -1); osmo_timer_schedule(&fi->timer, val_sec, 0); } break; diff --git a/src/sm/sm.c b/src/sm/sm.c index 59a5e7b..c83e739 100644 --- a/src/sm/sm.c +++ b/src/sm/sm.c @@ -33,7 +33,7 @@ #include <osmocom/gprs/sm/sm_ms_fsm.h> #include <osmocom/gprs/sm/sm_pdu.h> -struct gprs_sm_ctx *g_ctx; +struct gprs_sm_ctx *g_sm_ctx; /* TS 24.008 */ static struct osmo_tdef T_defs_sm[] = { @@ -45,10 +45,10 @@ { struct gprs_sm_ms *ms; - while ((ms = llist_first_entry_or_null(&g_ctx->ms_list, struct gprs_sm_ms, list))) + while ((ms = llist_first_entry_or_null(&g_sm_ctx->ms_list, struct gprs_sm_ms, list))) gprs_sm_ms_free(ms); - talloc_free(g_ctx); + talloc_free(g_sm_ctx); } int osmo_gprs_sm_init(enum osmo_gprs_sm_location location) @@ -57,22 +57,22 @@ int rc; OSMO_ASSERT(location == OSMO_GPRS_SM_LOCATION_MS || location == OSMO_GPRS_SM_LOCATION_NETWORK) - if (g_ctx) { + if (g_sm_ctx) { first_init = false; gprs_sm_ctx_free(); } - g_ctx = talloc_zero(NULL, struct gprs_sm_ctx); - g_ctx->location = location; - g_ctx->T_defs = T_defs_sm; - INIT_LLIST_HEAD(&g_ctx->ms_list); + g_sm_ctx = talloc_zero(NULL, struct gprs_sm_ctx); + g_sm_ctx->location = location; + g_sm_ctx->T_defs = T_defs_sm; + INIT_LLIST_HEAD(&g_sm_ctx->ms_list); - osmo_tdefs_reset(g_ctx->T_defs); + osmo_tdefs_reset(g_sm_ctx->T_defs); if (first_init) { rc = gprs_sm_ms_fsm_init(); if (rc != 0) { - TALLOC_FREE(g_ctx); + TALLOC_FREE(g_sm_ctx); return rc; } } @@ -83,13 +83,13 @@ { struct gprs_sm_ms *ms; - ms = talloc_zero(g_ctx, struct gprs_sm_ms); + ms = talloc_zero(g_sm_ctx, struct gprs_sm_ms); if (!ms) return NULL; ms->ms_id = ms_id; - llist_add(&ms->list, &g_ctx->ms_list); + llist_add(&ms->list, &g_sm_ctx->ms_list); return ms; } @@ -113,7 +113,7 @@ { struct gprs_sm_ms *ms; - llist_for_each_entry(ms, &g_ctx->ms_list, list) { + llist_for_each_entry(ms, &g_sm_ctx->ms_list, list) { if (ms->ms_id == ms_id) return ms; } @@ -123,12 +123,12 @@ struct gprs_sm_entity *gprs_sm_entity_alloc(struct gprs_sm_ms *ms, uint32_t nsapi) { struct gprs_sm_entity *sme; - sme = talloc_zero(g_ctx, struct gprs_sm_entity); + sme = talloc_zero(g_sm_ctx, struct gprs_sm_entity); if (!sme) return NULL; sme->ms = ms; - sme->sess_id = g_ctx->next_sess_id++; + sme->sess_id = g_sm_ctx->next_sess_id++; sme->nsapi = nsapi; if (gprs_sm_ms_fsm_ctx_init(&sme->ms_fsm, sme) < 0) { @@ -157,7 +157,7 @@ struct gprs_sm_ms *ms; unsigned int i; - llist_for_each_entry(ms, &g_ctx->ms_list, list) { + llist_for_each_entry(ms, &g_sm_ctx->ms_list, list) { for (i = 0; i < ARRAY_SIZE(ms->pdp); i++) { if (!ms->pdp[i]) continue; diff --git a/src/sm/sm_ms_fsm.c b/src/sm/sm_ms_fsm.c index b6fb79f..92137c4 100644 --- a/src/sm/sm_ms_fsm.c +++ b/src/sm/sm_ms_fsm.c @@ -39,7 +39,7 @@ }; #define sm_ms_fsm_state_chg(fi, NEXT_STATE) \ - osmo_tdef_fsm_inst_state_chg(fi, NEXT_STATE, sm_ms_fsm_timeouts, g_ctx->T_defs, -1) + osmo_tdef_fsm_inst_state_chg(fi, NEXT_STATE, sm_ms_fsm_timeouts, g_sm_ctx->T_defs, -1) static void st_sm_ms_pdp_inactive(struct osmo_fsm_inst *fi, uint32_t event, void *data) { diff --git a/src/sm/sm_prim.c b/src/sm/sm_prim.c index 197c8a4..33d15ba 100644 --- a/src/sm/sm_prim.c +++ b/src/sm/sm_prim.c @@ -87,22 +87,22 @@ /* Set callback used by SM layer to push primitives to higher layers in protocol stack */ void osmo_gprs_sm_prim_set_up_cb(osmo_gprs_sm_prim_up_cb up_cb, void *up_user_data) { - g_ctx->sm_up_cb = up_cb; - g_ctx->sm_up_cb_user_data = up_user_data; + g_sm_ctx->sm_up_cb = up_cb; + g_sm_ctx->sm_up_cb_user_data = up_user_data; } /* Set callback used by SM layer to push primitives to lower layers in protocol stack */ void osmo_gprs_sm_prim_set_down_cb(osmo_gprs_sm_prim_down_cb down_cb, void *down_user_data) { - g_ctx->sm_down_cb = down_cb; - g_ctx->sm_down_cb_user_data = down_user_data; + g_sm_ctx->sm_down_cb = down_cb; + g_sm_ctx->sm_down_cb_user_data = down_user_data; } /* Set callback used by SM layer to push primitives to GMM lower layer in protocol stack */ void osmo_gprs_sm_prim_set_gmm_down_cb(osmo_gprs_sm_prim_gmm_down_cb gmm_down_cb, void *gmm_down_user_data) { - g_ctx->sm_gmm_down_cb = gmm_down_cb; - g_ctx->sm_gmm_down_cb_user_data = gmm_down_user_data; + g_sm_ctx->sm_gmm_down_cb = gmm_down_cb; + g_sm_ctx->sm_gmm_down_cb_user_data = gmm_down_user_data; } /******************************** @@ -193,10 +193,10 @@ int gprs_sm_prim_call_up_cb(struct osmo_gprs_sm_prim *sm_prim) { int rc; - if (g_ctx->sm_up_cb) - rc = g_ctx->sm_up_cb(sm_prim, g_ctx->sm_up_cb_user_data); + if (g_sm_ctx->sm_up_cb) + rc = g_sm_ctx->sm_up_cb(sm_prim, g_sm_ctx->sm_up_cb_user_data); else - rc = sm_up_cb_dummy(sm_prim, g_ctx->sm_up_cb_user_data); + rc = sm_up_cb_dummy(sm_prim, g_sm_ctx->sm_up_cb_user_data); msgb_free(sm_prim->oph.msg); return rc; } @@ -304,10 +304,10 @@ int gprs_sm_prim_call_down_cb(struct osmo_gprs_sm_prim *sm_prim) { int rc; - if (g_ctx->sm_down_cb) - rc = g_ctx->sm_down_cb(sm_prim, g_ctx->sm_down_cb_user_data); + if (g_sm_ctx->sm_down_cb) + rc = g_sm_ctx->sm_down_cb(sm_prim, g_sm_ctx->sm_down_cb_user_data); else - rc = sm_down_cb_dummy(sm_prim, g_ctx->sm_down_cb_user_data); + rc = sm_down_cb_dummy(sm_prim, g_sm_ctx->sm_down_cb_user_data); msgb_free(sm_prim->oph.msg); return rc; } @@ -315,7 +315,7 @@ /* SM lower layers (GMM) push SM primitive up to SM layer: */ int osmo_gprs_sm_prim_lower_up(struct osmo_gprs_sm_prim *sm_prim) { - OSMO_ASSERT(g_ctx); + OSMO_ASSERT(g_sm_ctx); OSMO_ASSERT(sm_prim); struct msgb *msg = sm_prim->oph.msg; int rc; @@ -339,10 +339,10 @@ int gprs_sm_prim_call_gmm_down_cb(struct osmo_gprs_gmm_prim *gmm_prim) { int rc; - if (g_ctx->sm_gmm_down_cb) - rc = g_ctx->sm_gmm_down_cb(gmm_prim, g_ctx->sm_gmm_down_cb_user_data); + if (g_sm_ctx->sm_gmm_down_cb) + rc = g_sm_ctx->sm_gmm_down_cb(gmm_prim, g_sm_ctx->sm_gmm_down_cb_user_data); else - rc = sm_gmm_down_cb_dummy(gmm_prim, g_ctx->sm_gmm_down_cb_user_data); + rc = sm_gmm_down_cb_dummy(gmm_prim, g_sm_ctx->sm_gmm_down_cb_user_data); /* Special return value '1' means: do not free */ if (rc != 1) msgb_free(gmm_prim->oph.msg); @@ -412,7 +412,7 @@ /* SM lower layers (GMM) push SM primitive up to SM layer: */ int osmo_gprs_sm_prim_gmm_lower_up(struct osmo_gprs_gmm_prim *gmm_prim) { - OSMO_ASSERT(g_ctx); + OSMO_ASSERT(g_sm_ctx); OSMO_ASSERT(gmm_prim); struct msgb *msg = gmm_prim->oph.msg; int rc; diff --git a/src/sndcp/sndcp.c b/src/sndcp/sndcp.c index 252ba06..1ff5893 100644 --- a/src/sndcp/sndcp.c +++ b/src/sndcp/sndcp.c @@ -32,15 +32,15 @@ #include <osmocom/gprs/sndcp/sndcp_private.h> #include <osmocom/gprs/llc/llc_prim.h> -struct gprs_sndcp_ctx *g_ctx; +struct gprs_sndcp_ctx *g_sndcp_ctx; int osmo_gprs_sndcp_init(void) { - if (g_ctx) - talloc_free(g_ctx); + if (g_sndcp_ctx) + talloc_free(g_sndcp_ctx); - g_ctx = talloc_zero(NULL, struct gprs_sndcp_ctx); - INIT_LLIST_HEAD(&g_ctx->snme_list); + g_sndcp_ctx = talloc_zero(NULL, struct gprs_sndcp_ctx); + INIT_LLIST_HEAD(&g_sndcp_ctx->snme_list); return 0; } @@ -48,12 +48,12 @@ { struct gprs_sndcp_mgmt_entity *snme; - snme = talloc_zero(g_ctx, struct gprs_sndcp_mgmt_entity); + snme = talloc_zero(g_sndcp_ctx, struct gprs_sndcp_mgmt_entity); if (!snme) return NULL; snme->tlli = tlli; - llist_add(&snme->list, &g_ctx->snme_list); + llist_add(&snme->list, &g_sndcp_ctx->snme_list); return snme; } @@ -73,7 +73,7 @@ { struct gprs_sndcp_mgmt_entity *snme; - llist_for_each_entry(snme, &g_ctx->snme_list, list) { + llist_for_each_entry(snme, &g_sndcp_ctx->snme_list, list) { if (snme->tlli == tlli) return snme; } @@ -111,7 +111,7 @@ { struct gprs_sndcp_entity *sne; - sne = talloc_zero(g_ctx, struct gprs_sndcp_entity); + sne = talloc_zero(g_sndcp_ctx, struct gprs_sndcp_entity); if (!sne) return NULL; @@ -172,7 +172,7 @@ { struct defrag_queue_entry *dqe; - dqe = talloc_zero(g_ctx, struct defrag_queue_entry); + dqe = talloc_zero(g_sndcp_ctx, struct defrag_queue_entry); if (!dqe) return -ENOMEM; dqe->data = talloc_zero_size(dqe, data_len); @@ -323,7 +323,7 @@ * hands it off to the correct GTP tunnel + GGSN via gtp_data_req() */ /* Decompress packet */ - if (any_pcomp_or_dcomp_active(g_ctx)) { + if (any_pcomp_or_dcomp_active(g_sndcp_ctx)) { expnd = decompress_segment(sne, msg, npdu, npdu_len, &npdu_len); if (!expnd) { rc = -EIO; @@ -542,7 +542,7 @@ LOGSNE(sne, "===================================================\n"); debug_ip_packet(npdu, npdu_len, 0, __func__ "()"); #endif - if (any_pcomp_or_dcomp_active(g_ctx)) { + if (any_pcomp_or_dcomp_active(g_sndcp_ctx)) { /* Apply header compression */ rc = gprs_sndcp_pcomp_compress(msg->data, msg->len, &pcomp, @@ -810,8 +810,8 @@ * hands it off to the correct GTP tunnel + GGSN via gtp_data_req() */ /* Decompress packet */ - if (any_pcomp_or_dcomp_active(g_ctx)) { - expnd = decompress_segment(sne, g_ctx, npdu, npdu_len, (unsigned int *)&npdu_len); + if (any_pcomp_or_dcomp_active(g_sndcp_ctx)) { + expnd = decompress_segment(sne, g_sndcp_ctx, npdu, npdu_len, (unsigned int *)&npdu_len); if (!expnd) { rc = -EIO; goto ret_free; @@ -825,7 +825,7 @@ rc = gprs_sndcp_prim_call_up_cb(sndcp_prim_tx); ret_free: - if (any_pcomp_or_dcomp_active(g_ctx)) + if (any_pcomp_or_dcomp_active(g_sndcp_ctx)) talloc_free(expnd); return rc; } @@ -846,7 +846,7 @@ /* Process proposed parameters */ switch (comp_field->algo.pcomp) { case RFC_1144: - if (g_ctx->cfg.pcomp_rfc1144_passive_accept + if (g_sndcp_ctx->cfg.pcomp_rfc1144_passive_accept && comp_field->rfc1144_params->nsapi_len > 0) { LOGSNME(snme, LOGL_DEBUG, "Accepting RFC1144 header compression...\n"); gprs_sndcp_comp_add(snme, snme->comp.proto, comp_field); @@ -888,7 +888,7 @@ /* Process proposed parameters */ switch (comp_field->algo.dcomp) { case V42BIS: - if (g_ctx->cfg.dcomp_v42bis_passive_accept && + if (g_sndcp_ctx->cfg.dcomp_v42bis_passive_accept && comp_field->v42bis_params->nsapi_len > 0) { LOGSNME(snme, LOGL_DEBUG, "Accepting V.42bis data compression...\n"); gprs_sndcp_comp_add(snme, snme->comp.data, comp_field); diff --git a/src/sndcp/sndcp_prim.c b/src/sndcp/sndcp_prim.c index dee7dbe..bc22c9c 100644 --- a/src/sndcp/sndcp_prim.c +++ b/src/sndcp/sndcp_prim.c @@ -99,22 +99,22 @@ /* Set callback used by SNDCP layer to push primitives to higher layers in protocol stack */ void osmo_gprs_sndcp_prim_set_up_cb(osmo_gprs_sndcp_prim_up_cb up_cb, void *up_user_data) { - g_ctx->sndcp_up_cb = up_cb; - g_ctx->sndcp_up_cb_user_data = up_user_data; + g_sndcp_ctx->sndcp_up_cb = up_cb; + g_sndcp_ctx->sndcp_up_cb_user_data = up_user_data; } /* Set callback used by SNDCP layer to push primitives to lower layers in protocol stack */ void osmo_gprs_sndcp_prim_set_down_cb(osmo_gprs_sndcp_prim_down_cb down_cb, void *down_user_data) { - g_ctx->sndcp_down_cb = down_cb; - g_ctx->sndcp_down_cb_user_data = down_user_data; + g_sndcp_ctx->sndcp_down_cb = down_cb; + g_sndcp_ctx->sndcp_down_cb_user_data = down_user_data; } /* Set callback used by SNDCP layer to push primitives to SM sublayer */ void osmo_gprs_sndcp_prim_set_snsm_cb(osmo_gprs_sndcp_prim_snsm_cb snsm_cb, void *snsm_user_data) { - g_ctx->sndcp_snsm_cb = snsm_cb; - g_ctx->sndcp_snsm_cb_user_data = snsm_user_data; + g_sndcp_ctx->sndcp_snsm_cb = snsm_cb; + g_sndcp_ctx->sndcp_snsm_cb_user_data = snsm_user_data; } /******************************** @@ -289,10 +289,10 @@ int gprs_sndcp_prim_call_up_cb(struct osmo_gprs_sndcp_prim *sndcp_prim) { int rc; - if (g_ctx->sndcp_up_cb) - rc = g_ctx->sndcp_up_cb(sndcp_prim, g_ctx->sndcp_up_cb_user_data); + if (g_sndcp_ctx->sndcp_up_cb) + rc = g_sndcp_ctx->sndcp_up_cb(sndcp_prim, g_sndcp_ctx->sndcp_up_cb_user_data); else - rc = sndcp_up_cb_dummy(sndcp_prim, g_ctx->sndcp_up_cb_user_data); + rc = sndcp_up_cb_dummy(sndcp_prim, g_sndcp_ctx->sndcp_up_cb_user_data); msgb_free(sndcp_prim->oph.msg); return rc; } @@ -395,7 +395,7 @@ int osmo_gprs_sndcp_prim_upper_down(struct osmo_gprs_sndcp_prim *sndcp_prim) { int rc; - OSMO_ASSERT(g_ctx); + OSMO_ASSERT(g_sndcp_ctx); LOGSNDCP(LOGL_INFO, "Rx from upper layers: %s\n", osmo_gprs_sndcp_prim_name(sndcp_prim)); @@ -428,10 +428,10 @@ int gprs_sndcp_prim_call_down_cb(struct osmo_gprs_llc_prim *llc_prim) { int rc; - if (g_ctx->sndcp_down_cb) - rc = g_ctx->sndcp_down_cb(llc_prim, g_ctx->sndcp_down_cb_user_data); + if (g_sndcp_ctx->sndcp_down_cb) + rc = g_sndcp_ctx->sndcp_down_cb(llc_prim, g_sndcp_ctx->sndcp_down_cb_user_data); else - rc = sndcp_down_cb_dummy(llc_prim, g_ctx->sndcp_down_cb_user_data); + rc = sndcp_down_cb_dummy(llc_prim, g_sndcp_ctx->sndcp_down_cb_user_data); msgb_free(llc_prim->oph.msg); return rc; } @@ -515,7 +515,7 @@ /* SNDCP lower layers (LLC) push SNDCP primitive up to SNDCP layer: */ int osmo_gprs_sndcp_prim_lower_up(struct osmo_gprs_llc_prim *llc_prim) { - OSMO_ASSERT(g_ctx); + OSMO_ASSERT(g_sndcp_ctx); OSMO_ASSERT(llc_prim); struct msgb *msg = llc_prim->oph.msg; int rc; @@ -545,10 +545,10 @@ int gprs_sndcp_prim_call_snsm_cb(struct osmo_gprs_sndcp_prim *sndcp_prim) { int rc; - if (g_ctx->sndcp_snsm_cb) - rc = g_ctx->sndcp_snsm_cb(sndcp_prim, g_ctx->sndcp_snsm_cb_user_data); + if (g_sndcp_ctx->sndcp_snsm_cb) + rc = g_sndcp_ctx->sndcp_snsm_cb(sndcp_prim, g_sndcp_ctx->sndcp_snsm_cb_user_data); else - rc = sndcp_snsm_cb_dummy(sndcp_prim, g_ctx->sndcp_snsm_cb_user_data); + rc = sndcp_snsm_cb_dummy(sndcp_prim, g_sndcp_ctx->sndcp_snsm_cb_user_data); msgb_free(sndcp_prim->oph.msg); return rc; } @@ -646,7 +646,7 @@ int osmo_gprs_sndcp_prim_dispatch_snsm(struct osmo_gprs_sndcp_prim *sndcp_prim) { int rc; - OSMO_ASSERT(g_ctx); + OSMO_ASSERT(g_sndcp_ctx); OSMO_ASSERT(sndcp_prim); struct msgb *msg = sndcp_prim->oph.msg; -- To view, visit https://gerrit.osmocom.org/c/libosmo-gprs/+/32293 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: libosmo-gprs Gerrit-Branch: master Gerrit-Change-Id: Ib8a844551452b79bdb3a81968ffc1fd42ee1c5d9 Gerrit-Change-Number: 32293 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-MessageType: newchange

1 0

[S] Change in osmo-ttcn3-hacks[master]: MGCP_Test: fix typo
by pespin 13 Apr '23

13 Apr '23

Attention is currently required from: dexter. pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/32292 ) Change subject: MGCP_Test: fix typo ...................................................................... Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/32292 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-Change-Id: I1c860acb60267a9b4d8049bf231c49f9c915923b Gerrit-Change-Number: 32292 Gerrit-PatchSet: 1 Gerrit-Owner: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: dexter <pmaier(a)sysmocom.de> Gerrit-Comment-Date: Thu, 13 Apr 2023 14:25:05 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

1 0

[M] Change in osmo-ttcn3-hacks[master]: MGCP_Test: add new testcases for oa/bwe conversion
by pespin 13 Apr '23

13 Apr '23

Attention is currently required from: dexter. pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/32220 ) Change subject: MGCP_Test: add new testcases for oa/bwe conversion ...................................................................... Patch Set 2: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/32220 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-Change-Id: I2b2d7ef7fb4fe31111aa8665c4d4295425502451 Gerrit-Change-Number: 32220 Gerrit-PatchSet: 2 Gerrit-Owner: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: dexter <pmaier(a)sysmocom.de> Gerrit-Comment-Date: Thu, 13 Apr 2023 14:24:52 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

1 0