summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorThomas Haller <thaller@redhat.com>2023-02-22 11:27:48 +0100
committerThomas Haller <thaller@redhat.com>2023-02-28 12:08:07 +0100
commit61388fd9c799ef92363ea73bde886a766157d1f3 (patch)
treeb0649e5e09c24dadaa395482925aaedda320e7e5
parentbb9894abecb7f52f904d98fe6ed42fb8292a9eb3 (diff)
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.c15
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