Skip to content

Commit

Permalink
lacp: warn transmit failure of lacp pdu
Browse files Browse the repository at this point in the history
It might be difficult to trace whether LACP PDU tx (as in
response) was successful when the pdu was not transmitted by
egress slave for various reasons (including resource contention
within NIC) and only way to trace its fate is by looking at
ofproto->stats.tx_[packets/bytes] and slave port stats.

Adding a warning when there is tx failure could help user
debug at the root of this problem.

Signed-off-by: Gowrishankar Muthukrishnan <gmuthukr@redhat.com>
Signed-off-by: Ben Pfaff <blp@ovn.org>
  • Loading branch information
Gowrishankar Muthukrishnan authored and blp committed Oct 30, 2019
1 parent 7139ca5 commit a773a5c
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion ofproto/ofproto-dpif.c
Expand Up @@ -3405,7 +3405,11 @@ send_pdu_cb(void *port_, const void *pdu, size_t pdu_size)
pdu_size);
memcpy(packet_pdu, pdu, pdu_size);

ofproto_dpif_send_packet(port, false, &packet);
error = ofproto_dpif_send_packet(port, false, &packet);
if (error) {
VLOG_WARN_RL(&rl, "port %s: cannot transmit LACP PDU (%s).",
port->bundle->name, ovs_strerror(error));
}
dp_packet_uninit(&packet);
} else {
static struct vlog_rate_limit rll = VLOG_RATE_LIMIT_INIT(1, 10);
Expand Down

0 comments on commit a773a5c

Please sign in to comment.