aboutsummaryrefslogtreecommitdiffstats
path: root/src/vlib/log.c
diff options
context:
space:
mode:
authorDamjan Marion <damarion@cisco.com>2020-10-23 21:52:50 +0200
committerOle Tr�an <otroan@employees.org>2020-10-26 10:45:49 +0000
commit4d7ad4a6cc08e316d78de5ac7ce3ead4e9bfcb66 (patch)
treeef4158573a6aea024be7260ef7ebe4f12dfc64a8 /src/vlib/log.c
parent340b10a38ed84600ed3336657ef3a104d9ebe877 (diff)
vlib: add VLIB_REGISTER_LOG_CLASS macro
Type: improvement Change-Id: Iafcaeb88fa3569e042fb9cb270cf062b145416e2 Signed-off-by: Damjan Marion <damarion@cisco.com>
Diffstat (limited to 'src/vlib/log.c')
-rw-r--r--src/vlib/log.c67
1 files changed, 43 insertions, 24 deletions
diff --git a/src/vlib/log.c b/src/vlib/log.c
index 1bf19b0c2ea..e1ab036b25c 100644
--- a/src/vlib/log.c
+++ b/src/vlib/log.c
@@ -27,6 +27,12 @@ vlib_log_main_t log_main = {
.default_rate_limit = 50,
};
+/* *INDENT-OFF* */
+VLIB_REGISTER_LOG_CLASS (log_log, static) = {
+ .class_name = "log",
+};
+/* *INDENT-ON* */
+
static const int colors[] = {
[VLIB_LOG_LEVEL_EMERG] = 1, /* red */
[VLIB_LOG_LEVEL_ALERT] = 1, /* red */
@@ -38,6 +44,18 @@ static const int colors[] = {
[VLIB_LOG_LEVEL_DEBUG] = 6, /* cyan */
};
+static const int log_level_to_syslog_priority[] = {
+ [VLIB_LOG_LEVEL_EMERG] = LOG_EMERG,
+ [VLIB_LOG_LEVEL_ALERT] = LOG_ALERT,
+ [VLIB_LOG_LEVEL_CRIT] = LOG_CRIT,
+ [VLIB_LOG_LEVEL_ERR] = LOG_ERR,
+ [VLIB_LOG_LEVEL_WARNING] = LOG_WARNING,
+ [VLIB_LOG_LEVEL_NOTICE] = LOG_NOTICE,
+ [VLIB_LOG_LEVEL_INFO] = LOG_INFO,
+ [VLIB_LOG_LEVEL_DEBUG] = LOG_DEBUG,
+ [VLIB_LOG_LEVEL_DISABLED] = LOG_DEBUG,
+};
+
int
last_log_entry ()
{
@@ -65,22 +83,6 @@ get_subclass_data (vlib_log_class_t ci)
return vec_elt_at_index (c->subclasses, (ci & 0xffff));
}
-static int
-vlib_log_level_to_syslog_priority (vlib_log_level_t level)
-{
- switch (level)
- {
-#define LOG_DISABLED LOG_DEBUG
-#define _(n,uc,lc) \
- case VLIB_LOG_LEVEL_##uc:\
- return LOG_##uc;
- foreach_vlib_log_level
-#undef _
-#undef LOG_DISABLED
- }
- return LOG_DEBUG;
-}
-
u8 *
format_vlib_log_class (u8 * s, va_list * args)
{
@@ -201,7 +203,7 @@ vlib_log (vlib_log_level_t level, vlib_log_class_t class, char *fmt, ...)
else
{
l = format (l, "%U", format_vlib_log_class, class);
- int prio = vlib_log_level_to_syslog_priority (level);
+ int prio = log_level_to_syslog_priority[level];
int is_term = vec_c_string_is_terminated (l) ? 1 : 0;
syslog (prio, "%.*s: %.*s", (int) vec_len (l), l,
@@ -263,8 +265,8 @@ vlib_log_register_class_internal (char *class, char *subclass, u32 limit)
vec_add2 (lm->classes, c, 1);
c->index = c - lm->classes;
c->name = format (0, "%s", class);
- length = vec_len (c->name);
}
+ length = vec_len (c->name);
vec_add2 (c->subclasses, s, 1);
s->index = s - c->subclasses;
@@ -322,7 +324,7 @@ format_vlib_log_level (u8 * s, va_list * args)
switch (i)
{
-#define _(v,uc,lc) case VLIB_LOG_LEVEL_##uc: t = #lc; break;
+#define _(uc,lc) case VLIB_LOG_LEVEL_##uc: t = #lc; break;
foreach_vlib_log_level
#undef _
default:
@@ -335,12 +337,29 @@ static clib_error_t *
vlib_log_init (vlib_main_t * vm)
{
vlib_log_main_t *lm = &log_main;
+ vlib_log_class_registration_t *r = lm->registrations;
gettimeofday (&lm->time_zero_timeval, 0);
lm->time_zero = vlib_time_now (vm);
vec_validate (lm->entries, lm->size);
- lm->log_class = vlib_log_register_class ("log", 0);
+
+ while (r)
+ {
+ r->class = vlib_log_register_class (r->class_name, r->subclass_name);
+ if (r->default_level)
+ get_subclass_data (r->class)->level = r->default_level;
+ if (r->default_syslog_level)
+ get_subclass_data (r->class)->syslog_level = r->default_syslog_level;
+ r = r->next;
+ }
+
+ r = lm->registrations;
+ while (r)
+ {
+ vlib_log_debug (r->class, "initialized");
+ r = r->next;
+ }
return 0;
}
@@ -449,7 +468,7 @@ clear_log (vlib_main_t * vm,
lm->count = 0;
lm->next = 0;
- vlib_log_info (lm->log_class, "log cleared");
+ vlib_log_info (log_log.class, "log cleared");
return error;
}
@@ -469,11 +488,11 @@ unformat_vlib_log_level (unformat_input_t * input, va_list * args)
uword rv = 1;
if (unformat (input, "%s", &level_str))
{
-#define _(v, uc, lc) \
+#define _(uc, lc) \
const char __##uc[] = #lc; \
- if (!strcmp ((const char *) level_str, __##uc)) \
+ if (!strcmp ((const char *) level_str, __##uc)) \
{ \
- *level = VLIB_LOG_LEVEL_##uc; \
+ *level = VLIB_LOG_LEVEL_##uc; \
rv = 1; \
goto done; \
}