lime-proto-batadv change MAC also of wlan interfaces#820
lime-proto-batadv change MAC also of wlan interfaces#820spiccinini merged 1 commit intolibremesh:masterfrom
Conversation
Codecov Report
@@ Coverage Diff @@
## master #820 +/- ##
=======================================
Coverage 72.27% 72.27%
=======================================
Files 37 37
Lines 3293 3293
=======================================
Hits 2380 2380
Misses 913 913
Continue to review full report at Codecov.
|
|
The change looks good, did you tested it on a few different devices/architectures, every-time a mac is changed it could tickle up some unexpected bug even in the hardware driver :-\ |
|
I quickly checked on YouHua WR1200JS and TP-Link WDR3600, please help with the testing as I will not have time to test further or on other routers. @G10h4ck @spiccinini |
ok that seems enough as those devices have quite different wifi stack |
|
This won't change the mac address seen by the other wireless stations right? sothe ipv6 link local addresses of the wireless interfaces won't change? Sorry if this seems a silly question |
I expect the macaddress of the wlanX-mesh interface so the one seen as station from the others and it's relative link local ipv6 address should not change. Instead I expect all of those properties change for the vlan interface. So just to make it explicit I expect wlanX-mesh_123 to get a different mac address and a different IPv6 link local address. Do we assume wlanX-mesh and wlanX-mesh_123 have same properties in some part of the stack? |
Thanks Gio!
Not that I am aware. |
In order to avoid the "received packet on bat0 with own address as source address" annoying (but harmless) message we are customizing the MAC address of the ethernet VLAN interface, see #726 (and #703 and #724).
But in some cases (see here) for some reason the wireless VLAN interface can be taken as MainIF by batman-adv and therefore needs to have a customized mac address also.
This PR simply moves the MAC customization code out of the
ifblock, with no further edit.