svn commit: r332318 - in stable/11: . sys/net

[Steven Hartland]

Author: smh
Date: Mon Apr  9 08:25:29 2018
New Revision: 332318
URL: https://svnweb.freebsd.org/changeset/base/332318

Log:
  MFC r327559:
  
  Disabled the use of flowid for lagg by default
  
  Sponsored by:	Multiplay

Modified:
  stable/11/UPDATING
  stable/11/sys/net/if_lagg.c
Directory Properties:
  stable/11/   (props changed)

Modified: stable/11/UPDATING
==============================================================================
--- stable/11/UPDATING	Mon Apr  9 05:48:12 2018	(r332317)
+++ stable/11/UPDATING	Mon Apr  9 08:25:29 2018	(r332318)
@@ -16,6 +16,14 @@ from older versions of FreeBSD, try WITHOUT_CLANG and 
 the tip of head, and then rebuild without this option. The bootstrap process
 from older version of current across the gcc/clang cutover is a bit fragile.
 
+20180409:
+	The use of RSS hash from the network card aka flowid has been
+	disabled by default for lagg(4) as it's currently incompatible with
+	the lacp and loadbalance protocols.
+
+	This can be re-enabled by setting the following in loader.conf:
+	net.link.lagg.default_use_flowid="1"
+
 20180331:
 	Clang, llvm, lld, lldb, compiler-rt and libc++ have been upgraded to
 	6.0.0.  Please see the 20141231 entry below for information about

Modified: stable/11/sys/net/if_lagg.c
==============================================================================
--- stable/11/sys/net/if_lagg.c	Mon Apr  9 05:48:12 2018	(r332317)
+++ stable/11/sys/net/if_lagg.c	Mon Apr  9 08:25:29 2018	(r332318)
@@ -238,7 +238,7 @@ SYSCTL_INT(_net_link_lagg, OID_AUTO, failover_rx_all, 
     "Accept input from any interface in a failover lagg");
 
 /* Default value for using flowid */
-static VNET_DEFINE(int, def_use_flowid) = 1;
+static VNET_DEFINE(int, def_use_flowid) = 0;
 #define	V_def_use_flowid	VNET(def_use_flowid)
 SYSCTL_INT(_net_link_lagg, OID_AUTO, default_use_flowid, CTLFLAG_RWTUN,
     &VNET_NAME(def_use_flowid), 0,