[patch] Force interface down and also set bond mode *before* trying to set other configuration

bonding2.patch (text/plain), 850 bytes, created by Ed Wildgoose on 2010-10-18 17:42:11 UTC

--- /lib64/rc/net/bonding.sh.orig	2010-10-18 17:10:43.805511273 +0100
+++ /lib64/rc/net/bonding.sh	2010-10-18 18:26:46.195657969 +0100
@@ -46,11 +46,26 @@
 		return 1
 	fi
 
+	# Must ensure bond is down before configuring
+	_down
+
 	# Configure the bond.
-	# Nice and dynamic :)
+	# Must configure bonding mode first
+	for x in /sys/class/net/"${IFACE}"/bonding/mode; do
+		[ -f "${x}" ] || continue
+		n=${x##*/}
+		eval s=\$${n}_${IFVAR}
+		if [ -n "${s}" ]; then
+			einfo "Setting ${n}: ${s}"
+			echo "${s}" >"${x}" || \
+			eerror "Failed to configure $n (${n}_${IFVAR})"
+		fi
+	done
+	# Nice and dynamic for remaining options:)
 	for x in /sys/class/net/"${IFACE}"/bonding/*; do
 		[ -f "${x}" ] || continue
 		n=${x##*/}
+		[ "${n}" != "mode" ] || continue
 		eval s=\$${n}_${IFVAR}
 		if [ -n "${s}" ]; then
 			einfo "Setting ${n}: ${s}"