From: Matthew Smith <mgsmith@netgate.com>
Date: Tue, 7 Jan 2020 17:34:43 +0000 (-0600)
Subject: bonding: process lacp when bond is admin down
X-Git-Tag: v20.05-rc0~23
X-Git-Url: https://gerrit.fd.io/r/gitweb?a=commitdiff_plain;h=refs%2Fchanges%2F30%2F24230%2F2;p=vpp.git

bonding: process lacp when bond is admin down

Type: fix
Fixes: 6dfd3785e4

When a bond interface is administratively down but the slaves are
up, process inbound LACP packets received over the slaves. This
was the old behavior with bond interfaces in LACP mode and was
altered unintentionally by another change. Restore the old behavior.

Change-Id: I61b0b700211dea4859b6ee447ab83b33197d9d11
Signed-off-by: Matthew Smith <mgsmith@netgate.com>
---

diff --git a/src/vnet/bonding/node.c b/src/vnet/bonding/node.c
index de720706822..49c1f28ed06 100644
--- a/src/vnet/bonding/node.c
+++ b/src/vnet/bonding/node.c
@@ -163,7 +163,6 @@ bond_update_next (vlib_main_t * vm, vlib_node_runtime_t * node,
     {
       *bond_sw_if_index = slave_sw_if_index;
       *error = node->errors[BOND_INPUT_ERROR_IF_DOWN];
-      return;
     }
 
   if (PREDICT_FALSE ((bif->mode == BOND_MODE_ACTIVE_BACKUP) &&