X-Git-Url: https://pintos-os.org/cgi-bin/gitweb.cgi?a=blobdiff_plain;f=lib%2Fvlog.c;h=d26e61391e58f7fc3b52171f87faf1e0abfc9794;hb=626186c3bea6b07442632684eacf4d02545fa05f;hp=377aaa65ab8b938726d6d01ca1559934529e383f;hpb=d98e60075528c3065ad453f7add4b30f22edcde3;p=openvswitch diff --git a/lib/vlog.c b/lib/vlog.c index 377aaa65..d26e6139 100644 --- a/lib/vlog.c +++ b/lib/vlog.c @@ -1,5 +1,5 @@ /* - * Copyright (c) 2008, 2009, 2010 Nicira Networks. + * Copyright (c) 2008, 2009, 2010, 2011, 2012 Nicira, Inc. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -22,6 +22,7 @@ #include #include #include +#include #include #include #include @@ -29,6 +30,7 @@ #include "dirs.h" #include "dynamic-string.h" #include "sat-math.h" +#include "svec.h" #include "timeval.h" #include "unixctl.h" #include "util.h" @@ -80,9 +82,6 @@ static struct facility facilities[VLF_N_FACILITIES] = { #undef VLOG_FACILITY }; -/* Time at which vlog was initialized, in milliseconds. */ -static long long int boot_time; - /* VLF_FILE configuration. */ static char *log_file_name; static FILE *log_file; @@ -185,7 +184,7 @@ update_min_level(struct vlog_module *module) { enum vlog_facility facility; - module->min_level = VLL_EMER; + module->min_level = VLL_OFF; for (facility = 0; facility < VLF_N_FACILITIES; facility++) { if (log_file || facility != VLF_FILE) { enum vlog_level level = module->levels[facility]; @@ -291,7 +290,7 @@ vlog_set_log_file(const char *file_name) old_log_file_name = log_file_name; log_file_name = (file_name ? xstrdup(file_name) - : xasprintf("%s/%s.log", ovs_logdir, program_name)); + : xasprintf("%s/%s.log", ovs_logdir(), program_name)); free(old_log_file_name); file_name = NULL; /* Might have been freed. */ @@ -321,72 +320,99 @@ vlog_set_log_file(const char *file_name) int vlog_reopen_log_file(void) { - return log_file_name ? vlog_set_log_file(log_file_name) : 0; + struct stat old, new; + + /* Skip re-opening if there's nothing to reopen. */ + if (!log_file_name) { + return 0; + } + + /* Skip re-opening if it would be a no-op because the old and new files are + * the same. (This avoids writing "closing log file" followed immediately + * by "opened log file".) */ + if (log_file + && !fstat(fileno(log_file), &old) + && !stat(log_file_name, &new) + && old.st_dev == new.st_dev + && old.st_ino == new.st_ino) { + return 0; + } + + return vlog_set_log_file(log_file_name); } -/* Set debugging levels: - * - * mod[:facility[:level]] mod2[:facility[:level]] ... - * - * Return null if successful, otherwise an error message that the caller must - * free(). - */ +/* Set debugging levels. Returns null if successful, otherwise an error + * message that the caller must free(). */ char * vlog_set_levels_from_string(const char *s_) { - char *save_ptr = NULL; char *s = xstrdup(s_); - char *module, *facility; - - for (module = strtok_r(s, ": \t", &save_ptr); module != NULL; - module = strtok_r(NULL, ": \t", &save_ptr)) { - struct vlog_module *e_module; - enum vlog_facility e_facility; + char *save_ptr = NULL; + char *msg = NULL; + char *word; - facility = strtok_r(NULL, ":", &save_ptr); + word = strtok_r(s, " ,:\t", &save_ptr); + if (word && !strcasecmp(word, "PATTERN")) { + enum vlog_facility facility; - if (!facility || !strcmp(facility, "ANY")) { - e_facility = VLF_ANY_FACILITY; - } else { - e_facility = vlog_get_facility_val(facility); - if (e_facility >= VLF_N_FACILITIES) { - char *msg = xasprintf("unknown facility \"%s\"", facility); - free(s); - return msg; - } + word = strtok_r(NULL, " ,:\t", &save_ptr); + if (!word) { + msg = xstrdup("missing facility"); + goto exit; } - if (!strcmp(module, "PATTERN")) { - vlog_set_pattern(e_facility, save_ptr); - break; - } else { - char *level; - enum vlog_level e_level; - - if (!strcmp(module, "ANY")) { - e_module = NULL; - } else { - e_module = vlog_module_from_name(module); - if (!e_module) { - char *msg = xasprintf("unknown module \"%s\"", module); - free(s); - return msg; + facility = (!strcasecmp(word, "ANY") + ? VLF_ANY_FACILITY + : vlog_get_facility_val(word)); + if (facility == VLF_N_FACILITIES) { + msg = xasprintf("unknown facility \"%s\"", word); + goto exit; + } + vlog_set_pattern(facility, save_ptr); + } else { + struct vlog_module *module = NULL; + enum vlog_level level = VLL_N_LEVELS; + enum vlog_facility facility = VLF_N_FACILITIES; + + for (; word != NULL; word = strtok_r(NULL, " ,:\t", &save_ptr)) { + if (!strcasecmp(word, "ANY")) { + continue; + } else if (vlog_get_facility_val(word) != VLF_N_FACILITIES) { + if (facility != VLF_N_FACILITIES) { + msg = xstrdup("cannot specify multiple facilities"); + goto exit; } + facility = vlog_get_facility_val(word); + } else if (vlog_get_level_val(word) != VLL_N_LEVELS) { + if (level != VLL_N_LEVELS) { + msg = xstrdup("cannot specify multiple levels"); + goto exit; + } + level = vlog_get_level_val(word); + } else if (vlog_module_from_name(word)) { + if (module) { + msg = xstrdup("cannot specify multiple modules"); + goto exit; + } + module = vlog_module_from_name(word); + } else { + msg = xasprintf("no facility, level, or module \"%s\"", word); + goto exit; } + } - level = strtok_r(NULL, ":", &save_ptr); - e_level = level ? vlog_get_level_val(level) : VLL_DBG; - if (e_level >= VLL_N_LEVELS) { - char *msg = xasprintf("unknown level \"%s\"", level); - free(s); - return msg; - } - - vlog_set_levels(e_module, e_facility, e_level); + if (facility == VLF_N_FACILITIES) { + facility = VLF_ANY_FACILITY; + } + if (level == VLL_N_LEVELS) { + level = VLL_DBG; } + vlog_set_levels(module, facility, level); } + +exit: free(s); - return NULL; + return msg; } /* If 'arg' is null, configure maximum verbosity. Otherwise, sets @@ -405,36 +431,44 @@ vlog_set_verbosity(const char *arg) } static void -vlog_unixctl_set(struct unixctl_conn *conn, - const char *args, void *aux OVS_UNUSED) +vlog_unixctl_set(struct unixctl_conn *conn, int argc, const char *argv[], + void *aux OVS_UNUSED) { - char *msg = vlog_set_levels_from_string(args); - unixctl_command_reply(conn, msg ? 501 : 202, msg); - free(msg); + int i; + + for (i = 1; i < argc; i++) { + char *msg = vlog_set_levels_from_string(argv[i]); + if (msg) { + unixctl_command_reply_error(conn, msg); + free(msg); + return; + } + } + unixctl_command_reply(conn, NULL); } static void -vlog_unixctl_list(struct unixctl_conn *conn, - const char *args OVS_UNUSED, void *aux OVS_UNUSED) +vlog_unixctl_list(struct unixctl_conn *conn, int argc OVS_UNUSED, + const char *argv[] OVS_UNUSED, void *aux OVS_UNUSED) { char *msg = vlog_get_levels(); - unixctl_command_reply(conn, 200, msg); + unixctl_command_reply(conn, msg); free(msg); } static void -vlog_unixctl_reopen(struct unixctl_conn *conn, - const char *args OVS_UNUSED, void *aux OVS_UNUSED) +vlog_unixctl_reopen(struct unixctl_conn *conn, int argc OVS_UNUSED, + const char *argv[] OVS_UNUSED, void *aux OVS_UNUSED) { if (log_file_name) { int error = vlog_reopen_log_file(); if (error) { - unixctl_command_reply(conn, 503, strerror(errno)); + unixctl_command_reply_error(conn, strerror(errno)); } else { - unixctl_command_reply(conn, 202, NULL); + unixctl_command_reply(conn, NULL); } } else { - unixctl_command_reply(conn, 403, "Logging to file not configured"); + unixctl_command_reply_error(conn, "Logging to file not configured"); } } @@ -443,6 +477,7 @@ vlog_unixctl_reopen(struct unixctl_conn *conn, void vlog_init(void) { + static char *program_name_copy; time_t now; if (vlog_inited) { @@ -450,22 +485,30 @@ vlog_init(void) } vlog_inited = true; - openlog(program_name, LOG_NDELAY, LOG_DAEMON); + /* openlog() is allowed to keep the pointer passed in, without making a + * copy. The daemonize code sometimes frees and replaces 'program_name', + * so make a private copy just for openlog(). (We keep a pointer to the + * private copy to suppress memory leak warnings in case openlog() does + * make its own copy.) */ + program_name_copy = program_name ? xstrdup(program_name) : NULL; + openlog(program_name_copy, LOG_NDELAY, LOG_DAEMON); - boot_time = time_msec(); now = time_wall(); if (now < 0) { struct tm tm; char s[128]; - localtime_r(&now, &tm); - strftime(s, sizeof s, "%a, %d %b %Y %H:%M:%S %z", &tm); + gmtime_r(&now, &tm); + strftime(s, sizeof s, "%a, %d %b %Y %H:%M:%S", &tm); VLOG_ERR("current time is negative: %s (%ld)", s, (long int) now); } - unixctl_command_register("vlog/set", vlog_unixctl_set, NULL); - unixctl_command_register("vlog/list", vlog_unixctl_list, NULL); - unixctl_command_register("vlog/reopen", vlog_unixctl_reopen, NULL); + unixctl_command_register( + "vlog/set", "{spec | PATTERN:facility:pattern}", + 1, INT_MAX, vlog_unixctl_set, NULL); + unixctl_command_register("vlog/list", "", 0, 0, vlog_unixctl_list, NULL); + unixctl_command_register("vlog/reopen", "", 0, 0, + vlog_unixctl_reopen, NULL); } /* Closes the logging subsystem. */ @@ -484,17 +527,27 @@ vlog_get_levels(void) { struct ds s = DS_EMPTY_INITIALIZER; struct vlog_module **mp; + struct svec lines = SVEC_EMPTY_INITIALIZER; + char *line; + size_t i; ds_put_format(&s, " console syslog file\n"); ds_put_format(&s, " ------- ------ ------\n"); for (mp = vlog_modules; mp < &vlog_modules[n_vlog_modules]; mp++) { - ds_put_format(&s, "%-16s %4s %4s %4s\n", + line = xasprintf("%-16s %4s %4s %4s\n", vlog_get_module_name(*mp), vlog_get_level_name(vlog_get_level(*mp, VLF_CONSOLE)), vlog_get_level_name(vlog_get_level(*mp, VLF_SYSLOG)), vlog_get_level_name(vlog_get_level(*mp, VLF_FILE))); + svec_add_nocopy(&lines, line); + } + + svec_sort(&lines); + SVEC_FOR_EACH (i, line, &lines) { + ds_put_cstr(&s, line); } + svec_destroy(&lines); return ds_cstr(&s); } @@ -569,7 +622,11 @@ format_log_message(const struct vlog_module *module, enum vlog_level level, break; case 'd': p = fetch_braces(p, "%Y-%m-%d %H:%M:%S", tmp, sizeof tmp); - ds_put_strftime(s, tmp, NULL); + ds_put_strftime(s, tmp, false); + break; + case 'D': + p = fetch_braces(p, "%Y-%m-%d %H:%M:%S", tmp, sizeof tmp); + ds_put_strftime(s, tmp, true); break; case 'm': /* Format user-supplied log message and trim trailing new-lines. */ @@ -594,7 +651,7 @@ format_log_message(const struct vlog_module *module, enum vlog_level level, ds_put_format(s, "%ld", (long int) getpid()); break; case 'r': - ds_put_format(s, "%lld", time_msec() - boot_time); + ds_put_format(s, "%lld", time_msec() - time_boot_msec()); break; default: ds_put_char(s, p[-1]); @@ -680,6 +737,30 @@ vlog(const struct vlog_module *module, enum vlog_level level, va_end(args); } +void +vlog_fatal_valist(const struct vlog_module *module_, + const char *message, va_list args) +{ + struct vlog_module *module = (struct vlog_module *) module_; + + /* Don't log this message to the console to avoid redundancy with the + * message written by the later ovs_fatal_valist(). */ + module->levels[VLF_CONSOLE] = VLL_OFF; + + vlog_valist(module, VLL_EMER, message, args); + ovs_fatal_valist(0, message, args); +} + +void +vlog_fatal(const struct vlog_module *module, const char *message, ...) +{ + va_list args; + + va_start(args, message); + vlog_fatal_valist(module, message, args); + va_end(args); +} + bool vlog_should_drop(const struct vlog_module *module, enum vlog_level level, struct vlog_rate_limit *rl) @@ -688,33 +769,26 @@ vlog_should_drop(const struct vlog_module *module, enum vlog_level level, return true; } - if (rl->tokens < VLOG_MSG_TOKENS) { + if (!token_bucket_withdraw(&rl->token_bucket, VLOG_MSG_TOKENS)) { time_t now = time_now(); - if (rl->last_fill > now) { - /* Last filled in the future? Time must have gone backward, or - * 'rl' has not been used before. */ - rl->tokens = rl->burst; - } else if (rl->last_fill < now) { - unsigned int add = sat_mul(rl->rate, now - rl->last_fill); - unsigned int tokens = sat_add(rl->tokens, add); - rl->tokens = MIN(tokens, rl->burst); - rl->last_fill = now; - } - if (rl->tokens < VLOG_MSG_TOKENS) { - if (!rl->n_dropped) { - rl->first_dropped = now; - } - rl->n_dropped++; - return true; + if (!rl->n_dropped) { + rl->first_dropped = now; } + rl->last_dropped = now; + rl->n_dropped++; + return true; } - rl->tokens -= VLOG_MSG_TOKENS; if (rl->n_dropped) { + time_t now = time_now(); + unsigned int first_dropped_elapsed = now - rl->first_dropped; + unsigned int last_dropped_elapsed = now - rl->last_dropped; + vlog(module, level, - "Dropped %u log messages in last %u seconds " - "due to excessive rate", - rl->n_dropped, (unsigned int) (time_now() - rl->first_dropped)); + "Dropped %u log messages in last %u seconds (most recently, " + "%u seconds ago) due to excessive rate", + rl->n_dropped, first_dropped_elapsed, last_dropped_elapsed); + rl->n_dropped = 0; } return false; @@ -737,9 +811,9 @@ void vlog_usage(void) { printf("\nLogging options:\n" - " -v, --verbose=MODULE[:FACILITY[:LEVEL]] set logging levels\n" + " -v, --verbose=[SPEC] set logging levels\n" " -v, --verbose set maximum verbosity level\n" " --log-file[=FILE] enable logging to specified FILE\n" " (default: %s/%s.log)\n", - ovs_logdir, program_name); + ovs_logdir(), program_name); }