aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorHarald Welte <laforge@osmocom.org>2020-12-10 13:39:21 +0100
committerHarald Welte <laforge@osmocom.org>2020-12-12 20:05:51 +0100
commit398f60e11cce6c71d7f6dc9269943ba93bec1861 (patch)
tree741ccddeaabcbc154c97ea96653f0e742d77d3fb
parentb645365546f437c76269763f2f5cee295083f0e1 (diff)
migrate to DLBSSGP as log sub-system for BSSGP
-rw-r--r--TODO-RELEASE2
-rw-r--r--src/gprs_debug.cpp7
-rw-r--r--src/gprs_debug.h5
-rw-r--r--src/pcu_main.cpp3
-rw-r--r--tests/tbf/TbfTest.cpp3
5 files changed, 8 insertions, 12 deletions
diff --git a/TODO-RELEASE b/TODO-RELEASE
index 40932ee8..eee6aa24 100644
--- a/TODO-RELEASE
+++ b/TODO-RELEASE
@@ -8,4 +8,4 @@
# If any interfaces have been removed or changed since the last public release: c:r:0.
#library what description / commit summary line
osmo-pcu update osmo-gsm-manuals dependency to > 0.3.0 for vty_cpu_sched.adoc include
-osmo-pcu update libosmocore dependency > 1.4.x for osmo_fd_{read,write}_{enable,disable}
+osmo-pcu update libosmocore dependency > 1.4.x for osmo_fd_{read,write}_{enable,disable}+DLBSSGP
diff --git a/src/gprs_debug.cpp b/src/gprs_debug.cpp
index da5974b1..a790e3f6 100644
--- a/src/gprs_debug.cpp
+++ b/src/gprs_debug.cpp
@@ -112,13 +112,6 @@ static const struct log_info_cat default_categories[] = {
.loglevel = LOGL_NOTICE,
.enabled = 1,
},
- [DBSSGP] = {
- .name = "DBSSGP",
- .color = "\033[1;34m",
- .description = "GPRS BSS Gateway Protocol (BSSGP)",
- .loglevel = LOGL_NOTICE,
- .enabled = 1,
- },
[DPCU] = {
.name = "DPCU",
.color = "\033[1;35m",
diff --git a/src/gprs_debug.h b/src/gprs_debug.h
index 4c57633f..84a0a073 100644
--- a/src/gprs_debug.h
+++ b/src/gprs_debug.h
@@ -27,6 +27,10 @@ extern "C" {
};
#endif
+/* we used to have DBSSGP definded in each application, and applications telling
+ * libosmogb which sub-system to use. That creates problems and has been deprecated */
+#define DBSSGP DLBSSGP
+
/* Debug Areas of the code */
enum {
DCSN1,
@@ -41,7 +45,6 @@ enum {
DTBFDL,
DTBFUL,
DNS,
- DBSSGP,
DPCU,
aDebug_LastEntry
};
diff --git a/src/pcu_main.cpp b/src/pcu_main.cpp
index 2138fd99..16d73de2 100644
--- a/src/pcu_main.cpp
+++ b/src/pcu_main.cpp
@@ -43,6 +43,7 @@ extern "C" {
#include <osmocom/vty/command.h>
#include <osmocom/vty/vty.h>
#include <osmocom/vty/ports.h>
+#include <osmocom/vty/logging.h>
#include <osmocom/vty/cpu_sched_vty.h>
#include <osmocom/core/rate_ctr.h>
#include <osmocom/core/select.h>
@@ -242,12 +243,12 @@ int main(int argc, char *argv[])
osmo_stats_init(tall_pcu_ctx);
rate_ctr_init(tall_pcu_ctx);
- bssgp_set_log_ss(DBSSGP);
pcu_vty_info.tall_ctx = tall_pcu_ctx;
vty_init(&pcu_vty_info);
pcu_vty_init();
osmo_cpu_sched_vty_init(tall_pcu_ctx);
+ logging_vty_add_deprecated_subsys(tall_pcu_ctx, "bssgp");
handle_options(argc, argv);
if ((!!spoof_mcc) + (!!spoof_mnc) == 1) {
diff --git a/tests/tbf/TbfTest.cpp b/tests/tbf/TbfTest.cpp
index 505f510f..1a1dc6f3 100644
--- a/tests/tbf/TbfTest.cpp
+++ b/tests/tbf/TbfTest.cpp
@@ -3256,9 +3256,8 @@ int main(int argc, char **argv)
osmo_init_logging2(tall_pcu_ctx, &gprs_log_info);
log_set_use_color(osmo_stderr_target, 0);
log_set_print_filename(osmo_stderr_target, 0);
- bssgp_set_log_ss(DBSSGP);
log_parse_category_mask(osmo_stderr_target, "DRLCMAC,1:DRLCMACDATA,3:DRLCMACDL,3:DRLCMACUL,3:"
- "DRLCMACSCHED,1:DRLCMACMEAS,3:DNS,3:DBSSGP,3:DPCU,5:"
+ "DRLCMACSCHED,1:DRLCMACMEAS,3:DNS,3:DLBSSGP,3:DPCU,5:"
"DL1IF,6:DTBF,1:DTBFUL,1:DTBFDL,1:DLGLOBAL,2:");
vty_init(&pcu_vty_info);