Adding shared ingress block with ingress qdisc already exists results in a failure. So remove the ingress qdisc first. Also while at it log the slave name.
Signed-off-by: Roi Dayan <[email protected]> --- lib/netdev-linux.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/lib/netdev-linux.c b/lib/netdev-linux.c index 92cfb229d445..0d0d045f7c70 100644 --- a/lib/netdev-linux.c +++ b/lib/netdev-linux.c @@ -742,11 +742,14 @@ netdev_linux_update_lag(struct rtnetlink_change *change) lag->block_id = block_id; lag->node = shash_add(&lag_shash, change->ifname, lag); + /* delete ingress block in case it exists */ + tc_add_del_ingress_qdisc(change->if_index, false, 0); /* LAG master is linux netdev so add slave to same block. */ error = tc_add_del_ingress_qdisc(change->if_index, true, block_id); if (error) { - VLOG_WARN("failed to bind LAG slave to master's block"); + VLOG_WARN("failed to bind LAG slave %s to master's block", + change->ifname); shash_delete(&lag_shash, lag->node); free(lag); } -- 2.7.0 _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
