On Mon, Mar 20, 2017 at 5:33 PM, Govindarajulu Varadarajan <gvara...@cisco.com> wrote: > On Mon, 20 Mar 2017, PJ Waskiewicz wrote: > >> From: PJ Waskiewicz <pjwaskiew...@gmail.com> >> >> The permanent MAC address is useful to store for things like ethtool, >> and when bonding with modes such as active/passive or LACP. > > > Hi Peter, > > Is this patch fixing an issue with bonding drive on enic?
We noticed that running ethtool -P <eth> on an enic, even on 4.9, returned nothing. This has fallout when using bonding, where LACP or Active/Passive overrides the LAA on one of the slaves, one can't figure out what the physical MAC address is of each slave. So not a problem with bonding directly, it's more secondary as a result of the driver not reporting the actual permanent address. > >> This follows the model of other Ethernet drivers, such as ixgbe. >> > > While other drivers set netdev->perm_addr, doesn't this actually come free > in > register_netdevice(). I thought it did as well, but in 4.9 when we tested it wasn't working. Hence the patch. :-) Cheers, -PJ