diff options
author | Thomas Haller <thaller@redhat.com> | 2020-03-18 09:38:27 +0100 |
---|---|---|
committer | Thomas Haller <thaller@redhat.com> | 2020-03-18 09:52:16 +0100 |
commit | 6c837a366b3a81e104ebbd6b58d5a4b0e70ae838 (patch) | |
tree | b2ee6aed566531a65d228a885579159db117d3a7 | |
parent | 8fe2046f0858970531a599894683710e66fd22a7 (diff) | |
download | NetworkManager-6c837a366b3a81e104ebbd6b58d5a4b0e70ae838.tar.gz |
supplicant: combine multiple log lines about supported supplicant features
For many purposes, the supplicant features are not very interesting (as
they are also mostly static for a certain release/distribution). Combine
the multiple logging lines into one.
Also, sort the NMSupplCapType enum values consistently with the order
in which we log them.
Also, rename the logging output for features to match the enum name.
E.g. "FAST" instead of "EAP-FAST".
Now:
> supplicant: supported features: AP+ PMF+ FILS- P2P+ FT+ SHA384+ MESH+ FAST+ WFD+
-rw-r--r-- | src/supplicant/nm-supplicant-manager.c | 43 | ||||
-rw-r--r-- | src/supplicant/nm-supplicant-types.h | 8 |
2 files changed, 31 insertions, 20 deletions
diff --git a/src/supplicant/nm-supplicant-manager.c b/src/supplicant/nm-supplicant-manager.c index 2f089fb36f..e66a778934 100644 --- a/src/supplicant/nm-supplicant-manager.c +++ b/src/supplicant/nm-supplicant-manager.c @@ -166,18 +166,18 @@ _caps_set (NMSupplicantManagerPrivate *priv, priv->capabilities = NM_SUPPL_CAP_MASK_SET (priv->capabilities, type, value); } -static const char * -_caps_to_str (NMSupplicantManagerPrivate *priv, - NMSupplCapType type) +static char +_caps_to_char (NMSupplicantManagerPrivate *priv, + NMSupplCapType type) { NMTernary val; - val = NM_SUPPL_CAP_MASK_GET (priv->capabilities, type);; + val = NM_SUPPL_CAP_MASK_GET (priv->capabilities, type); if (val == NM_TERNARY_TRUE) - return "supported"; + return '+'; if (val == NM_TERNARY_FALSE) - return "not supported"; - return "possibly supported"; + return '-'; + return '?'; } /*****************************************************************************/ @@ -992,15 +992,26 @@ _dbus_get_capabilities_cb (GVariant *res, } } - _LOGD ("AP mode is %s", _caps_to_str (priv, NM_SUPPL_CAP_TYPE_AP)); - _LOGD ("PMF is %s", _caps_to_str (priv, NM_SUPPL_CAP_TYPE_PMF)); - _LOGD ("FILS is %s", _caps_to_str (priv, NM_SUPPL_CAP_TYPE_FILS)); - _LOGD ("P2P is %s", _caps_to_str (priv, NM_SUPPL_CAP_TYPE_P2P)); - _LOGD ("FT is %s", _caps_to_str (priv, NM_SUPPL_CAP_TYPE_FT)); - _LOGD ("SHA384 is %s", _caps_to_str (priv, NM_SUPPL_CAP_TYPE_SHA384)); - _LOGD ("Mesh is %s", _caps_to_str (priv, NM_SUPPL_CAP_TYPE_MESH)); - _LOGD ("EAP-FAST is %s", _caps_to_str (priv, NM_SUPPL_CAP_TYPE_FAST)); - _LOGD ("WFD is %s", _caps_to_str (priv, NM_SUPPL_CAP_TYPE_WFD)); + _LOGD ("supported features:" + " AP%c" + " PMF%c" + " FILS%c" + " P2P%c" + " FT%c" + " SHA384%c" + " MESH%c" + " FAST%c" + " WFD%c" + "", + _caps_to_char (priv, NM_SUPPL_CAP_TYPE_AP), + _caps_to_char (priv, NM_SUPPL_CAP_TYPE_PMF), + _caps_to_char (priv, NM_SUPPL_CAP_TYPE_FILS), + _caps_to_char (priv, NM_SUPPL_CAP_TYPE_P2P), + _caps_to_char (priv, NM_SUPPL_CAP_TYPE_FT), + _caps_to_char (priv, NM_SUPPL_CAP_TYPE_SHA384), + _caps_to_char (priv, NM_SUPPL_CAP_TYPE_MESH), + _caps_to_char (priv, NM_SUPPL_CAP_TYPE_FAST), + _caps_to_char (priv, NM_SUPPL_CAP_TYPE_WFD)); nm_assert (g_hash_table_size (priv->supp_ifaces) == 0); nm_assert (c_list_is_empty (&priv->supp_lst_head)); diff --git a/src/supplicant/nm-supplicant-types.h b/src/supplicant/nm-supplicant-types.h index bad52c345a..734ff1562b 100644 --- a/src/supplicant/nm-supplicant-types.h +++ b/src/supplicant/nm-supplicant-types.h @@ -36,15 +36,15 @@ typedef struct _NMSupplicantConfig NMSupplicantConfig; /*****************************************************************************/ typedef enum { - NM_SUPPL_CAP_TYPE_AP = 0, - NM_SUPPL_CAP_TYPE_FAST, + NM_SUPPL_CAP_TYPE_AP, NM_SUPPL_CAP_TYPE_PMF, NM_SUPPL_CAP_TYPE_FILS, NM_SUPPL_CAP_TYPE_P2P, - NM_SUPPL_CAP_TYPE_MESH, - NM_SUPPL_CAP_TYPE_WFD, NM_SUPPL_CAP_TYPE_FT, NM_SUPPL_CAP_TYPE_SHA384, + NM_SUPPL_CAP_TYPE_MESH, + NM_SUPPL_CAP_TYPE_FAST, + NM_SUPPL_CAP_TYPE_WFD, _NM_SUPPL_CAP_TYPE_NUM, } NMSupplCapType; |