diff options
| author | Dan Winship <danw@redhat.com> | 2015-01-13 15:08:21 -0500 |
|---|---|---|
| committer | Dan Winship <danw@redhat.com> | 2015-01-15 10:22:13 -0500 |
| commit | 53f2642c736ebb8466cbfd29c2ede2c3828a4728 (patch) | |
| tree | f191c2943f234cb59f60ff07019a45bd72183bf1 | |
| parent | 233498e0a45173efbb25086167444f9f4dc6432d (diff) | |
| download | NetworkManager-53f2642c736ebb8466cbfd29c2ede2c3828a4728.tar.gz | |
connectivity: improve debug logging
nm-connectivity was logging both "started" and "finished" for periodic
connectivity checks, but was only logging "finished" for manual ones,
which made the logs look weird. Fix it to log both periodic and manual
starts, and differentiate them.
Also add some additional logging to indicate when set_online() is
called, and when :state changes.
| -rw-r--r-- | src/nm-connectivity.c | 31 |
1 files changed, 30 insertions, 1 deletions
diff --git a/src/nm-connectivity.c b/src/nm-connectivity.c index 8a324a5c16..5913fd96c0 100644 --- a/src/nm-connectivity.c +++ b/src/nm-connectivity.c @@ -69,12 +69,33 @@ nm_connectivity_get_state (NMConnectivity *connectivity) return NM_CONNECTIVITY_GET_PRIVATE (connectivity)->state; } +static const char * +state_name (NMConnectivityState state) +{ + switch (state) { + case NM_CONNECTIVITY_UNKNOWN: + return "UNKNOWN"; + case NM_CONNECTIVITY_NONE: + return "NONE"; + case NM_CONNECTIVITY_LIMITED: + return "LIMITED"; + case NM_CONNECTIVITY_PORTAL: + return "PORTAL"; + case NM_CONNECTIVITY_FULL: + return "FULL"; + default: + return "???"; + } +} + static void update_state (NMConnectivity *self, NMConnectivityState state) { NMConnectivityPrivate *priv = NM_CONNECTIVITY_GET_PRIVATE (self); if (priv->state != state) { + nm_log_dbg (LOGD_CONCHECK, "Connectivity state changed from %s to %s", + state_name (priv->state), state_name (state)); priv->state = state; g_object_notify (G_OBJECT (self), NM_CONNECTIVITY_STATE); } @@ -155,7 +176,6 @@ run_check (gpointer user_data) nm_connectivity_check_async (self, run_check_complete, NULL); priv->running = TRUE; - nm_log_dbg (LOGD_CONCHECK, "Connectivity check with uri '%s' started.", priv->uri); return TRUE; } @@ -167,7 +187,11 @@ nm_connectivity_set_online (NMConnectivity *self, { #if WITH_CONCHECK NMConnectivityPrivate *priv = NM_CONNECTIVITY_GET_PRIVATE (self); +#endif + + nm_log_dbg (LOGD_CONCHECK, "nm_connectivity_set_online(%s)", online ? "TRUE" : "FALSE"); +#if WITH_CONCHECK if (online && priv->uri && priv->interval) { if (!priv->check_id) priv->check_id = g_timeout_add_seconds (priv->interval, run_check, self); @@ -201,6 +225,11 @@ nm_connectivity_check_async (NMConnectivity *self, g_return_if_fail (NM_IS_CONNECTIVITY (self)); priv = NM_CONNECTIVITY_GET_PRIVATE (self); + if (callback == run_check_complete) + nm_log_dbg (LOGD_CONCHECK, "Periodic connectivity check started with uri '%s'.", priv->uri); + else + nm_log_dbg (LOGD_CONCHECK, "Connectivity check started with uri '%s'.", priv->uri); + simple = g_simple_async_result_new (G_OBJECT (self), callback, user_data, nm_connectivity_check_async); |