lynxis lazus has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-pcu/+/21728 )


Change subject: ns: follow ns2 dialect changes
......................................................................

ns: follow ns2 dialect changes

NS2 introduce a ns dialect to differentiate
between the 4 possible dialects.

Change-Id: I16dc82c38eb75c2b9d1197640a955fec7df84efc
---
M src/bts.h
M src/gprs_bssgp_pcu.cpp
M src/pcu_vty.c
3 files changed, 8 insertions(+), 9 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-pcu refs/changes/28/21728/1

diff --git a/src/bts.h b/src/bts.h
index fd52067..c3880f4 100644
--- a/src/bts.h
+++ b/src/bts.h
@@ -27,6 +27,7 @@
 #include <osmocom/core/rate_ctr.h>
 #include <osmocom/core/stat_item.h>
 #include <osmocom/core/tdef.h>
+#include <osmocom/gprs/gprs_ns2.h>
 #include <osmocom/gsm/l1sap.h>
 #include <osmocom/gsm/protocol/gsm_04_08.h>
 #include <osmocom/gsm/gsm48.h>
@@ -170,7 +171,7 @@
        char *pcu_sock_path;

        /* Are we talking Gb with IP-SNS (true) or classic Gb? */
-       bool gb_dialect_sns;
+       enum gprs_ns2_dialect ns_dialect;

        /* Packet Application Information (3GPP TS 44.060 11.2.47, usually ETWS 
primary message). We don't need to store
         * more than one message, because they get sent so rarely. */
diff --git a/src/gprs_bssgp_pcu.cpp b/src/gprs_bssgp_pcu.cpp
index 934d8fc..4d24d2e 100644
--- a/src/gprs_bssgp_pcu.cpp
+++ b/src/gprs_bssgp_pcu.cpp
@@ -984,7 +984,7 @@
        bts->nse = gprs_ns2_nse_by_nsei(bts->nsi, nsei);
        if (!bts->nse)
                bts->nse = gprs_ns2_create_nse(bts->nsi, nsei,
-                                              GPRS_NS2_LL_UDP);
+                                              GPRS_NS2_LL_UDP, 
bts->ns_dialect);

        if (!bts->nse) {
                LOGP(DBSSGP, LOGL_ERROR, "Failed to create NSE\n");
@@ -997,7 +997,7 @@
                        continue;

                /* FIXME: for SNS we just use the first successful NS-VC 
instead of all for the initial connect */
-               if (bts->gb_dialect_sns) {
+               if (bts->ns_dialect == NS2_DIALECT_SNS) {
                        rc = gprs_ns2_ip_connect_sns(bind[i], &remote[i], nsei);
                        if (!rc)
                                return rc;
@@ -1066,7 +1066,7 @@
                gprs_ns2_free_nses(bts->nsi);
                gprs_ns2_free_binds(bts->nsi);
                rc = ns_create_nsvc(bts, nsei, local, remote, nsvci, valid);
-       } else if (bts->gb_dialect_sns) {
+       } else if (bts->ns_dialect == NS2_DIALECT_SNS) {
                /* SNS: check if the initial nsvc is the same, if not recreate 
it */
                const struct osmo_sockaddr *initial = 
gprs_ns2_nse_sns_remote(bts->nse);
                for (unsigned int i = 0; i < PCU_IF_NUM_NSVC; i++) {
diff --git a/src/pcu_vty.c b/src/pcu_vty.c
index bf45686..800079c 100644
--- a/src/pcu_vty.c
+++ b/src/pcu_vty.c
@@ -246,7 +246,7 @@
                }
        }

-       if (bts->gb_dialect_sns)
+       if (bts->ns_dialect == NS2_DIALECT_SNS)
                vty_out(vty, " gb-dialect ip-sns%s", VTY_NEWLINE);
        else
                vty_out(vty, " gb-dialect classic%s", VTY_NEWLINE);
@@ -1131,11 +1131,9 @@
        struct gprs_rlcmac_bts *bts = bts_main_data();

        if (!strcmp(argv[0], "ip-sns")) {
-               bts->gb_dialect_sns = true;
-               gprs_ns2_vty_force_vc_mode(true, NS2_VC_MODE_ALIVE, "gb-dialect 
is ip-sns");
+               bts->ns_dialect = NS2_DIALECT_SNS;
        } else {
-               bts->gb_dialect_sns = false;
-               gprs_ns2_vty_force_vc_mode(false, 0, NULL);
+               bts->ns_dialect = NS2_DIALECT_IPACCESS;
        }

        return CMD_SUCCESS;

--
To view, visit https://gerrit.osmocom.org/c/osmo-pcu/+/21728
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit-Change-Id: I16dc82c38eb75c2b9d1197640a955fec7df84efc
Gerrit-Change-Number: 21728
Gerrit-PatchSet: 1
Gerrit-Owner: lynxis lazus <lyn...@fe80.eu>
Gerrit-MessageType: newchange

Reply via email to