X-Git-Url: https://pintos-os.org/cgi-bin/gitweb.cgi?a=blobdiff_plain;f=lib%2Freconnect.c;h=0333d9622c3e5b940a7e72f2226db972035de904;hb=5cc2c261d33f091ea45ff3de22737cf4deb724a2;hp=96917beca5935c619f948d17f4be8f3d9f894bd7;hpb=19df7f512c50e453c8eaa97675e213b4d6e9735d;p=openvswitch diff --git a/lib/reconnect.c b/lib/reconnect.c index 96917bec..0333d962 100644 --- a/lib/reconnect.c +++ b/lib/reconnect.c @@ -1,5 +1,5 @@ /* - * Copyright (c) 2008, 2009, 2010 Nicira Networks. + * Copyright (c) 2008, 2009, 2010 Nicira, Inc. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -21,14 +21,14 @@ #include #include "poll-loop.h" - -#define THIS_MODULE VLM_reconnect #include "vlog.h" +VLOG_DEFINE_THIS_MODULE(reconnect); + #define STATES \ STATE(VOID, 1 << 0) \ STATE(BACKOFF, 1 << 1) \ - STATE(CONNECT_IN_PROGRESS, 1 << 3) \ + STATE(CONNECTING, 1 << 3) \ STATE(ACTIVE, 1 << 4) \ STATE(IDLE, 1 << 5) \ STATE(RECONNECT, 1 << 6) \ @@ -52,6 +52,7 @@ struct reconnect { int max_backoff; int probe_interval; bool passive; + enum vlog_level info; /* Used for informational messages. */ /* State. */ enum state state; @@ -59,6 +60,7 @@ struct reconnect { int backoff; long long int last_received; long long int last_connected; + long long int last_disconnected; unsigned int max_tries; /* These values are simply for statistics reporting, not otherwise used @@ -94,16 +96,18 @@ reconnect_create(long long int now) struct reconnect *fsm = xzalloc(sizeof *fsm); fsm->name = xstrdup("void"); - fsm->min_backoff = 1000; - fsm->max_backoff = 8000; - fsm->probe_interval = 5000; + fsm->min_backoff = RECONNECT_DEFAULT_MIN_BACKOFF; + fsm->max_backoff = RECONNECT_DEFAULT_MAX_BACKOFF; + fsm->probe_interval = RECONNECT_DEFAULT_PROBE_INTERVAL; fsm->passive = false; + fsm->info = VLL_INFO; fsm->state = S_VOID; fsm->state_entered = now; fsm->backoff = 0; fsm->last_received = now; - fsm->last_connected = now; + fsm->last_connected = LLONG_MAX; + fsm->last_disconnected = LLONG_MAX; fsm->max_tries = UINT_MAX; fsm->creation_time = now; @@ -120,6 +124,22 @@ reconnect_destroy(struct reconnect *fsm) } } +/* If 'quiet' is true, 'fsm' will log informational messages at level VLL_DBG, + * by default keeping them out of log files. This is appropriate if the + * connection is one that is expected to be short-lived, so that the log + * messages are merely distracting. + * + * If 'quiet' is false, 'fsm' logs informational messages at level VLL_INFO. + * This is the default. + * + * This setting has no effect on the log level of debugging, warning, or error + * messages. */ +void +reconnect_set_quiet(struct reconnect *fsm, bool quiet) +{ + fsm->info = quiet ? VLL_DBG : VLL_INFO; +} + /* Returns 'fsm''s name. */ const char * reconnect_get_name(const struct reconnect *fsm) @@ -139,7 +159,7 @@ reconnect_set_name(struct reconnect *fsm, const char *name) } /* Return the minimum number of milliseconds to back off between consecutive - * connection attempts. The default is 1000 ms. */ + * connection attempts. The default is RECONNECT_DEFAULT_MIN_BACKOFF. */ int reconnect_get_min_backoff(const struct reconnect *fsm) { @@ -147,7 +167,7 @@ reconnect_get_min_backoff(const struct reconnect *fsm) } /* Return the maximum number of milliseconds to back off between consecutive - * connection attempts. The default is 8000 ms. */ + * connection attempts. The default is RECONNECT_DEFAULT_MAX_BACKOFF. */ int reconnect_get_max_backoff(const struct reconnect *fsm) { @@ -191,12 +211,16 @@ reconnect_get_max_tries(struct reconnect *fsm) * attempts. * * 'min_backoff' must be at least 1000, and 'max_backoff' must be greater than - * or equal to 'min_backoff'. */ + * or equal to 'min_backoff'. + * + * Pass 0 for 'min_backoff' or 'max_backoff' or both to use the defaults. */ void reconnect_set_backoff(struct reconnect *fsm, int min_backoff, int max_backoff) { fsm->min_backoff = MAX(min_backoff, 1000); - fsm->max_backoff = max_backoff ? MAX(max_backoff, 1000) : 8000; + fsm->max_backoff = (max_backoff + ? MAX(max_backoff, 1000) + : RECONNECT_DEFAULT_MAX_BACKOFF); if (fsm->min_backoff > fsm->max_backoff) { fsm->max_backoff = fsm->min_backoff; } @@ -239,7 +263,7 @@ reconnect_set_passive(struct reconnect *fsm, bool passive, long long int now) fsm->passive = passive; if (passive - ? fsm->state & (S_CONNECT_IN_PROGRESS | S_RECONNECT) + ? fsm->state & (S_CONNECTING | S_RECONNECT) : fsm->state == S_LISTENING && reconnect_may_retry(fsm)) { reconnect_transition__(fsm, now, S_BACKOFF); fsm->backoff = 0; @@ -288,7 +312,7 @@ reconnect_disable(struct reconnect *fsm, long long int now) void reconnect_force_reconnect(struct reconnect *fsm, long long int now) { - if (fsm->state & (S_CONNECT_IN_PROGRESS | S_ACTIVE | S_IDLE)) { + if (fsm->state & (S_CONNECTING | S_ACTIVE | S_IDLE)) { reconnect_transition__(fsm, now, S_RECONNECT); } } @@ -309,16 +333,17 @@ reconnect_disconnected(struct reconnect *fsm, long long int now, int error) VLOG_WARN("%s: connection dropped (%s)", fsm->name, strerror(error)); } else if (error == EOF) { - VLOG_INFO("%s: connection closed by peer", fsm->name); + VLOG(fsm->info, "%s: connection closed by peer", fsm->name); } else { - VLOG_INFO("%s: connection dropped", fsm->name); + VLOG(fsm->info, "%s: connection dropped", fsm->name); } } else if (fsm->state == S_LISTENING) { if (error > 0) { VLOG_WARN("%s: error listening for connections (%s)", fsm->name, strerror(error)); } else { - VLOG_INFO("%s: error listening for connections", fsm->name); + VLOG(fsm->info, "%s: error listening for connections", + fsm->name); } } else { const char *type = fsm->passive ? "listen" : "connection"; @@ -326,10 +351,13 @@ reconnect_disconnected(struct reconnect *fsm, long long int now, int error) VLOG_WARN("%s: %s attempt failed (%s)", fsm->name, type, strerror(error)); } else { - VLOG_INFO("%s: %s attempt timed out", fsm->name, type); + VLOG(fsm->info, "%s: %s attempt timed out", fsm->name, type); } } + if (fsm->state & (S_ACTIVE | S_IDLE)) { + fsm->last_disconnected = now; + } /* Back off. */ if (fsm->state & (S_ACTIVE | S_IDLE) && (fsm->last_received - fsm->last_connected >= fsm->backoff @@ -344,10 +372,10 @@ reconnect_disconnected(struct reconnect *fsm, long long int now, int error) fsm->backoff *= 2; } if (fsm->passive) { - VLOG_INFO("%s: waiting %.3g seconds before trying to " + VLOG(fsm->info, "%s: waiting %.3g seconds before trying to " "listen again", fsm->name, fsm->backoff / 1000.0); } else { - VLOG_INFO("%s: waiting %.3g seconds before reconnect", + VLOG(fsm->info, "%s: waiting %.3g seconds before reconnect", fsm->name, fsm->backoff / 1000.0); } } @@ -360,17 +388,18 @@ reconnect_disconnected(struct reconnect *fsm, long long int now, int error) /* Tell 'fsm' that a connection or listening attempt is in progress. * * The FSM will start a timer, after which the connection or listening attempt - * will be aborted (by returning RECONNECT_DISCONNECT from reconect_run()). */ + * will be aborted (by returning RECONNECT_DISCONNECT from + * reconnect_run()). */ void reconnect_connecting(struct reconnect *fsm, long long int now) { - if (fsm->state != S_CONNECT_IN_PROGRESS) { + if (fsm->state != S_CONNECTING) { if (fsm->passive) { - VLOG_INFO("%s: listening...", fsm->name); + VLOG(fsm->info, "%s: listening...", fsm->name); } else { - VLOG_INFO("%s: connecting...", fsm->name); + VLOG(fsm->info, "%s: connecting...", fsm->name); } - reconnect_transition__(fsm, now, S_CONNECT_IN_PROGRESS); + reconnect_transition__(fsm, now, S_CONNECTING); } } @@ -389,7 +418,7 @@ void reconnect_listening(struct reconnect *fsm, long long int now) { if (fsm->state != S_LISTENING) { - VLOG_INFO("%s: listening...", fsm->name); + VLOG(fsm->info, "%s: listening...", fsm->name); reconnect_transition__(fsm, now, S_LISTENING); } } @@ -422,7 +451,7 @@ reconnect_connected(struct reconnect *fsm, long long int now) if (!is_connected_state(fsm->state)) { reconnect_connecting(fsm, now); - VLOG_INFO("%s: connected", fsm->name); + VLOG(fsm->info, "%s: connected", fsm->name); reconnect_transition__(fsm, now, S_ACTIVE); fsm->last_connected = now; } @@ -453,7 +482,7 @@ static void reconnect_transition__(struct reconnect *fsm, long long int now, enum state state) { - if (fsm->state == S_CONNECT_IN_PROGRESS) { + if (fsm->state == S_CONNECTING) { fsm->n_attempted_connections++; if (state == S_ACTIVE) { fsm->n_successful_connections++; @@ -483,7 +512,7 @@ reconnect_deadline__(const struct reconnect *fsm) case S_BACKOFF: return fsm->state_entered + fsm->backoff; - case S_CONNECT_IN_PROGRESS: + case S_CONNECTING: return fsm->state_entered + MAX(1000, fsm->backoff); case S_ACTIVE: @@ -494,7 +523,10 @@ reconnect_deadline__(const struct reconnect *fsm) return LLONG_MAX; case S_IDLE: - return fsm->state_entered + fsm->probe_interval; + if (fsm->probe_interval) { + return fsm->state_entered + fsm->probe_interval; + } + return LLONG_MAX; case S_RECONNECT: return fsm->state_entered; @@ -550,7 +582,7 @@ reconnect_run(struct reconnect *fsm, long long int now) case S_BACKOFF: return RECONNECT_CONNECT; - case S_CONNECT_IN_PROGRESS: + case S_CONNECTING: return RECONNECT_DISCONNECT; case S_ACTIVE: @@ -613,13 +645,26 @@ reconnect_is_connected(const struct reconnect *fsm) return is_connected_state(fsm->state); } -/* Returns the number of milliseconds for which 'fsm' has been continuously - * connected to its peer. (If 'fsm' is not currently connected, this is 0.) */ +/* Returns the number of milliseconds since 'fsm' last successfully connected + * to its peer (even if it has since disconnected). Returns UINT_MAX if never + * connected. */ +unsigned int +reconnect_get_last_connect_elapsed(const struct reconnect *fsm, + long long int now) +{ + return fsm->last_connected == LLONG_MAX ? UINT_MAX + : now - fsm->last_connected; +} + +/* Returns the number of milliseconds since 'fsm' last disconnected + * from its peer (even if it has since reconnected). Returns UINT_MAX if never + * disconnected. */ unsigned int -reconnect_get_connection_duration(const struct reconnect *fsm, - long long int now) +reconnect_get_last_disconnect_elapsed(const struct reconnect *fsm, + long long int now) { - return reconnect_is_connected(fsm) ? now - fsm->last_connected : 0; + return fsm->last_disconnected == LLONG_MAX ? UINT_MAX + : now - fsm->last_disconnected; } /* Copies various statistics for 'fsm' into '*stats'. */ @@ -630,13 +675,17 @@ reconnect_get_stats(const struct reconnect *fsm, long long int now, stats->creation_time = fsm->creation_time; stats->last_received = fsm->last_received; stats->last_connected = fsm->last_connected; + stats->last_disconnected = fsm->last_disconnected; stats->backoff = fsm->backoff; stats->seqno = fsm->seqno; stats->is_connected = reconnect_is_connected(fsm); - stats->current_connection_duration - = reconnect_get_connection_duration(fsm, now); - stats->total_connected_duration = (stats->current_connection_duration - + fsm->total_connected_duration); + stats->msec_since_connect + = reconnect_get_last_connect_elapsed(fsm, now); + stats->msec_since_disconnect + = reconnect_get_last_disconnect_elapsed(fsm, now); + stats->total_connected_duration = fsm->total_connected_duration + + (is_connected_state(fsm->state) + ? reconnect_get_last_connect_elapsed(fsm, now) : 0); stats->n_attempted_connections = fsm->n_attempted_connections; stats->n_successful_connections = fsm->n_successful_connections; stats->state = reconnect_state_name__(fsm->state);