diff options
author | Thomas Haller <thaller@redhat.com> | 2023-02-22 11:27:48 +0100 |
---|---|---|
committer | Thomas Haller <thaller@redhat.com> | 2023-02-28 12:08:07 +0100 |
commit | 61388fd9c799ef92363ea73bde886a766157d1f3 (patch) | |
tree | b0649e5e09c24dadaa395482925aaedda320e7e5 | |
parent | bb9894abecb7f52f904d98fe6ed42fb8292a9eb3 (diff) | |
download | NetworkManager-61388fd9c799ef92363ea73bde886a766157d1f3.tar.gz |
platform: drop logging for unexpected sequence number
It is not clear how that information is relevant. Since it is also
only logged when building with a non-default configure option, this
doesn't seem useful. Drop it.
-rw-r--r-- | src/libnm-platform/nm-linux-platform.c | 15 |
1 files changed, 1 insertions, 14 deletions
diff --git a/src/libnm-platform/nm-linux-platform.c b/src/libnm-platform/nm-linux-platform.c index 4423b75ddb..9a671d0856 100644 --- a/src/libnm-platform/nm-linux-platform.c +++ b/src/libnm-platform/nm-linux-platform.c @@ -482,9 +482,6 @@ typedef struct { typedef struct { guint32 nlh_seq_next; guint32 nlh_seq_last_seen; -#if NM_MORE_LOGGING - guint32 nlh_seq_last_handled; -#endif } NetlinkProtocolPrivData; typedef struct { @@ -7742,7 +7739,7 @@ event_seq_check(NMPlatform *platform, if (!NM_FLAGS_ANY( priv->delayed_action.flags, nmp_netlink_protocol_info(netlink_protocol)->delayed_action_type_wait_for_response)) - goto out; + return; nm_assert(priv->delayed_action.list_wait_for_response_x[netlink_protocol]->len > 0); @@ -7767,16 +7764,6 @@ event_seq_check(NMPlatform *platform, return; } - -out: - -#if NM_MORE_LOGGING - if (seq_number != priv->proto_data_x[netlink_protocol].nlh_seq_last_handled) - _LOGt("netlink: recvmsg: unwaited sequence number %u", seq_number); - priv->proto_data_x[netlink_protocol].nlh_seq_last_handled = seq_number; -#else - (void) 0; -#endif } static void |