daniel submitted this change.

View Change

Approvals: pespin: Looks good to me, approved fixeria: Looks good to me, but someone else must approve Jenkins Builder: Verified
Add DLTCAP log category

Change-Id: I004dddeeb7db3b47d373bb6281708f33d6e8148f
Related: SYS#5423
---
M include/osmocom/core/logging.h
M src/core/logging.c
M tests/logging/logging_vty_test.vty
3 files changed, 11 insertions(+), 3 deletions(-)

diff --git a/include/osmocom/core/logging.h b/include/osmocom/core/logging.h
index 82e686f..9d4f1fb 100644
--- a/include/osmocom/core/logging.h
+++ b/include/osmocom/core/logging.h
@@ -155,7 +155,8 @@
#define DLM2PA -27 /*!< Osmocom M2PA (libosmo-sigtran) */
#define DLM2UA -28 /*!< Reserved for future Osmocom M2UA (libosmo-sigtran) */
#define DLIO -29 /*!< Osmocom IO sub-system */
-#define OSMO_NUM_DLIB 29 /*!< Number of logging sub-systems in libraries */
+#define DLTCAP -30 /*!< Osmocom TCAP */
+#define OSMO_NUM_DLIB 30 /*!< Number of logging sub-systems in libraries */

/* Colors that can be used in log_info_cat.color */
#define OSMO_LOGCOLOR_NORMAL NULL
diff --git a/src/core/logging.c b/src/core/logging.c
index e172124..cccfd93 100644
--- a/src/core/logging.c
+++ b/src/core/logging.c
@@ -393,6 +393,12 @@
.enabled = 1, .loglevel = LOGL_NOTICE,
.color = "\033[38;5;67m",
},
+ [INT2IDX(DLTCAP)] = {
+ .name = "DLTCAP",
+ .description = "TCAP",
+ .enabled = 1, .loglevel = LOGL_NOTICE,
+ .color = "\033[38;5;69m",
+ },
};

void assert_loginfo(const char *src)
diff --git a/tests/logging/logging_vty_test.vty b/tests/logging/logging_vty_test.vty
index da09be7..edf0512 100644
--- a/tests/logging/logging_vty_test.vty
+++ b/tests/logging/logging_vty_test.vty
@@ -54,7 +54,7 @@
logging print level (0|1)
logging print file (0|1|basename) [last]
logging set-log-mask MASK
- logging level (aa|bb|ccc|dddd|eee|lglobal|llapd|linp|lmux|lmi|lmib|lsms|lctrl|lgtp|lstats|lgsup|loap|lss7|lsccp|lsua|lm3ua|lmgcp|ljibuf|lrspro|lns|lbssgp|lnsdata|lnssignal|liuup|lpfcp|lcsn1|lio) (debug|info|notice|error|fatal)
+ logging level (aa|bb|ccc|dddd|eee|lglobal|llapd|linp|lmux|lmi|lmib|lsms|lctrl|lgtp|lstats|lgsup|loap|lss7|lsccp|lsua|lm3ua|lmgcp|ljibuf|lrspro|lns|lbssgp|lnsdata|lnssignal|liuup|lpfcp|lcsn1|lio|ltcap) (debug|info|notice|error|fatal)
logging level set-all (debug|info|notice|error|fatal)
logging level force-all (debug|info|notice|error|fatal)
no logging level force-all
@@ -568,7 +568,7 @@

logging_vty_test# list
...
- logp (aa|bb|ccc|dddd|eee|lglobal|llapd|linp|lmux|lmi|lmib|lsms|lctrl|lgtp|lstats|lgsup|loap|lss7|lsccp|lsua|lm3ua|lmgcp|ljibuf|lrspro|lns|lbssgp|lnsdata|lnssignal|liuup|lpfcp|lcsn1|lio) (debug|info|notice|error|fatal) .LOGMESSAGE
+ logp (aa|bb|ccc|dddd|eee|lglobal|llapd|linp|lmux|lmi|lmib|lsms|lctrl|lgtp|lstats|lgsup|loap|lss7|lsccp|lsua|lm3ua|lmgcp|ljibuf|lrspro|lns|lbssgp|lnsdata|lnssignal|liuup|lpfcp|lcsn1|lio|ltcap) (debug|info|notice|error|fatal) .LOGMESSAGE
...

logging_vty_test# logp?
@@ -607,6 +607,7 @@
lpfcp libosmo-pfcp Packet Forwarding Control Protocol
lcsn1 libosmo-csn1 Concrete Syntax Notation 1 codec
lio libosmocore IO Subsystem
+ ltcap TCAP

logging_vty_test# logp lglobal ?
debug Log debug messages and higher levels

To view, visit change 41520. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-MessageType: merged
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I004dddeeb7db3b47d373bb6281708f33d6e8148f
Gerrit-Change-Number: 41520
Gerrit-PatchSet: 4
Gerrit-Owner: daniel <dwillmann@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: daniel <dwillmann@sysmocom.de>
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>