Author: rpokala
Date: Fri Jun 19 05:54:15 2020
New Revision: 362364
URL: https://svnweb.freebsd.org/changeset/base/362364

Log:
  MFC r362078:
  
  Decode the "LACP Fast Timeout" LAGG option flag
  
  r286700 added the "lacp_fast_timeout" option to `ifconfig', but we forgot to
  include the new option in the string used to decode the option bits. Add
  "LACP_FAST_TIMO" to LAGG_OPT_BITS.
  
  Also, s/LAGG_OPT_LACP_TIMEOUT/LAGG_OPT_LACP_FAST_TIMO/g , to be clearer that
  the flag indicates "Fast Timeout" mode.

Modified:
  stable/12/sbin/ifconfig/iflagg.c
  stable/12/sys/net/if_lagg.c
  stable/12/sys/net/if_lagg.h
Directory Properties:
  stable/12/   (props changed)

Modified: stable/12/sbin/ifconfig/iflagg.c
==============================================================================
--- stable/12/sbin/ifconfig/iflagg.c    Fri Jun 19 04:18:20 2020        
(r362363)
+++ stable/12/sbin/ifconfig/iflagg.c    Fri Jun 19 05:54:15 2020        
(r362364)
@@ -139,8 +139,8 @@ setlaggsetopt(const char *val, int d, int s, const str
        case -LAGG_OPT_LACP_TXTEST:
        case LAGG_OPT_LACP_RXTEST:
        case -LAGG_OPT_LACP_RXTEST:
-       case LAGG_OPT_LACP_TIMEOUT:
-       case -LAGG_OPT_LACP_TIMEOUT:
+       case LAGG_OPT_LACP_FAST_TIMO:
+       case -LAGG_OPT_LACP_FAST_TIMO:
                break;
        default:
                err(1, "Invalid lagg option");
@@ -312,8 +312,8 @@ static struct cmd lagg_cmds[] = {
        DEF_CMD("-lacp_txtest", -LAGG_OPT_LACP_TXTEST,  setlaggsetopt),
        DEF_CMD("lacp_rxtest",  LAGG_OPT_LACP_RXTEST,   setlaggsetopt),
        DEF_CMD("-lacp_rxtest", -LAGG_OPT_LACP_RXTEST,  setlaggsetopt),
-       DEF_CMD("lacp_fast_timeout",    LAGG_OPT_LACP_TIMEOUT,  setlaggsetopt),
-       DEF_CMD("-lacp_fast_timeout",   -LAGG_OPT_LACP_TIMEOUT, setlaggsetopt),
+       DEF_CMD("lacp_fast_timeout",    LAGG_OPT_LACP_FAST_TIMO,        
setlaggsetopt),
+       DEF_CMD("-lacp_fast_timeout",   -LAGG_OPT_LACP_FAST_TIMO,       
setlaggsetopt),
        DEF_CMD_ARG("flowid_shift",     setlaggflowidshift),
        DEF_CMD_ARG("rr_limit",         setlaggrr_limit),
 };

Modified: stable/12/sys/net/if_lagg.c
==============================================================================
--- stable/12/sys/net/if_lagg.c Fri Jun 19 04:18:20 2020        (r362363)
+++ stable/12/sys/net/if_lagg.c Fri Jun 19 05:54:15 2020        (r362364)
@@ -1220,7 +1220,7 @@ lagg_ioctl(struct ifnet *ifp, u_long cmd, caddr_t data
                        if (lsc->lsc_strict_mode != 0)
                                ro->ro_opts |= LAGG_OPT_LACP_STRICT;
                        if (lsc->lsc_fast_timeout != 0)
-                               ro->ro_opts |= LAGG_OPT_LACP_TIMEOUT;
+                               ro->ro_opts |= LAGG_OPT_LACP_FAST_TIMO;
 
                        ro->ro_active = sc->sc_active;
                } else {
@@ -1277,8 +1277,8 @@ lagg_ioctl(struct ifnet *ifp, u_long cmd, caddr_t data
                case -LAGG_OPT_LACP_RXTEST:
                case LAGG_OPT_LACP_STRICT:
                case -LAGG_OPT_LACP_STRICT:
-               case LAGG_OPT_LACP_TIMEOUT:
-               case -LAGG_OPT_LACP_TIMEOUT:
+               case LAGG_OPT_LACP_FAST_TIMO:
+               case -LAGG_OPT_LACP_FAST_TIMO:
                        valid = lacp = 1;
                        break;
                default:
@@ -1338,14 +1338,14 @@ lagg_ioctl(struct ifnet *ifp, u_long cmd, caddr_t data
                        case -LAGG_OPT_LACP_STRICT:
                                lsc->lsc_strict_mode = 0;
                                break;
-                       case LAGG_OPT_LACP_TIMEOUT:
+                       case LAGG_OPT_LACP_FAST_TIMO:
                                LACP_LOCK(lsc);
                                LIST_FOREACH(lp, &lsc->lsc_ports, lp_next)
                                        lp->lp_state |= LACP_STATE_TIMEOUT;
                                LACP_UNLOCK(lsc);
                                lsc->lsc_fast_timeout = 1;
                                break;
-                       case -LAGG_OPT_LACP_TIMEOUT:
+                       case -LAGG_OPT_LACP_FAST_TIMO:
                                LACP_LOCK(lsc);
                                LIST_FOREACH(lp, &lsc->lsc_ports, lp_next)
                                        lp->lp_state &= ~LACP_STATE_TIMEOUT;

Modified: stable/12/sys/net/if_lagg.h
==============================================================================
--- stable/12/sys/net/if_lagg.h Fri Jun 19 04:18:20 2020        (r362363)
+++ stable/12/sys/net/if_lagg.h Fri Jun 19 05:54:15 2020        (r362364)
@@ -147,7 +147,7 @@ struct lagg_reqopts {
 #define        LAGG_OPT_LACP_STRICT            0x10            /* LACP strict 
mode */
 #define        LAGG_OPT_LACP_TXTEST            0x20            /* LACP debug: 
txtest */
 #define        LAGG_OPT_LACP_RXTEST            0x40            /* LACP debug: 
rxtest */
-#define        LAGG_OPT_LACP_TIMEOUT           0x80            /* LACP timeout 
*/
+#define        LAGG_OPT_LACP_FAST_TIMO         0x80            /* LACP fast 
timeout */
 #define        LAGG_OPT_RR_LIMIT               0x100           /* RR stride */
        u_int                   ro_count;               /* number of ports */
        u_int                   ro_active;              /* active port count */
@@ -160,7 +160,8 @@ struct lagg_reqopts {
 #define        SIOCSLAGGOPTS            _IOW('i', 153, struct lagg_reqopts)
 
 #define        LAGG_OPT_BITS           "\020\001USE_FLOWID\005LACP_STRICT" \
-                               "\006LACP_TXTEST\007LACP_RXTEST"
+                               "\006LACP_TXTEST\007LACP_RXTEST" \
+                               "\010LACP_FAST_TIMO"
 
 #ifdef _KERNEL
 
_______________________________________________
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"

Reply via email to