> We wouldn't need hwaddress anymore with the new patch, right?

Nope, I have found another problem with the kind of setup that you have, on
this setups brctl addif fails because the interface needs to be configured
by hostapd, and as the code for setting the mac was conditionally tight to
the brctl it would not be executed. I have moved it before the brctl addif
so that it will be executed no matter what. This is on a patch attached to
this bug. That should hopefully solve the final problem with mac setting.

> iface br0 inet dhcp
>     bridge_hw enp4s0MAC
>     bridge_ports regex (en|wl).*
>     post-up ifup /wl*
> iface br0 inet6 auto
>     privext 2
> 
> iface enp3s0 inet manual
> iface enp4s0 inet manual

these two iface enp* lines shouldn't be needed at all, the regex on bridge
ports should catch them and bridge-utils should configure the ports.
Like I said before I'd add the bridge_ports to the inet6 definition as well,
the full patch to fix this two definitions will need that info for
everything to run ok.

If you can add the attached patch and test everything it would be great.

Regards...
-- 
Manty/BestiaTester -> http://manty.net
--- /lib/udev/bridge-network-interface~	2021-02-16 13:01:37.000000000 +0100
+++ /lib/udev/bridge-network-interface	2021-02-21 00:24:11.936523677 +0100
@@ -31,10 +31,10 @@
 				if [ -d /sys/class/net/$port ]; then
 					ifup --allow auto $i
 					if [ -f /proc/sys/net/ipv6/conf/$port/disable_ipv6 ]; then echo 1 > /proc/sys/net/ipv6/conf/$port/disable_ipv6;fi
-					brctl addif $i $port && ip link set dev $port up &&
 					if [ "$(ifquery "$i"|sed -n -e's/^bridge[_-]hw: //p')" = "$port" ]; then
 						ip link set dev "$i" address "$(ip link show dev "$port" 2>/dev/null|sed -n "s|.*link/ether \([^ ]*\) brd.*|\1|p")"
 					fi
+					brctl addif $i $port && ip link set dev $port up
 				fi
 				break
 				;;

Reply via email to