git: 1315446ba0cd - stable/13 - LACP: Do not wait response for marker messages not sent

From: Wojciech Macek <wma_at_FreeBSD.org>
Date: Tue, 07 Jun 2022 03:58:37 UTC
The branch stable/13 has been updated by wma:

URL: https://cgit.FreeBSD.org/src/commit/?id=1315446ba0cdf7ba66b5bd274c53960bd8b4bc8f

commit 1315446ba0cdf7ba66b5bd274c53960bd8b4bc8f
Author:     Arnaud Ysmal <arnaud.ysmal@stormshield.eu>
AuthorDate: 2021-09-23 08:57:11 +0000
Commit:     Wojciech Macek <wma@FreeBSD.org>
CommitDate: 2022-06-07 03:57:29 +0000

    LACP: Do not wait response for marker messages not sent
    
    The error returned when a marker message can not be emitted on a port is not handled.
    
    This cause the lacp to block all emissions until the timeout of 3 seconds is reached.
    
    To fix this issue, I just clear the LACP_PORT_MARK flag when the packet could not be emitted.
    
    Differential revision:  https://reviews.freebsd.org/D30467
    Obtained from:          Stormshield
    
    (cherry picked from commit 0b92a7fe47a5f7148a494514bfa530ddc6b496bf)
---
 sys/net/ieee8023ad_lacp.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/sys/net/ieee8023ad_lacp.c b/sys/net/ieee8023ad_lacp.c
index d1ee6eab4b35..6656ebb2b400 100644
--- a/sys/net/ieee8023ad_lacp.c
+++ b/sys/net/ieee8023ad_lacp.c
@@ -920,7 +920,8 @@ lacp_suppress_distributing(struct lacp_softc *lsc, struct lacp_aggregator *la)
 	/* send a marker frame down each port to verify the queues are empty */
 	LIST_FOREACH(lp, &lsc->lsc_ports, lp_next) {
 		lp->lp_flags |= LACP_PORT_MARK;
-		lacp_xmit_marker(lp);
+		if (lacp_xmit_marker(lp) != 0)
+			lp->lp_flags &= ~LACP_PORT_MARK;
 	}
 
 	/* set a timeout for the marker frames */