From: Paul Greenwalt <paul.greenw...@intel.com>

Remove is_zero_ether_add() check when setting the VF default LAN address.
This check assumed that the address had been delete and zeroed before
calling ice_vc_add_mac_addr(). Now the default LAN address will be set
to the last unicast MAC address added by the VF.

The default LAN address is reported by the PF via ndo_get_vf_config.

Signed-off-by: Paul Greenwalt <paul.greenw...@intel.com>
Tested-by: Andrew Bowers <andrewx.bow...@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirs...@intel.com>
---
 drivers/net/ethernet/intel/ice/ice_virtchnl_pf.c | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/drivers/net/ethernet/intel/ice/ice_virtchnl_pf.c 
b/drivers/net/ethernet/intel/ice/ice_virtchnl_pf.c
index 9b09a111321c..efd54299a220 100644
--- a/drivers/net/ethernet/intel/ice/ice_virtchnl_pf.c
+++ b/drivers/net/ethernet/intel/ice/ice_virtchnl_pf.c
@@ -2862,9 +2862,11 @@ ice_vc_add_mac_addr(struct ice_vf *vf, struct ice_vsi 
*vsi, u8 *mac_addr)
                return -EIO;
        }
 
-       /* only set dflt_lan_addr once */
-       if (is_zero_ether_addr(vf->dflt_lan_addr.addr) &&
-           is_unicast_ether_addr(mac_addr))
+       /* Set the default LAN address to the latest unicast MAC address added
+        * by the VF. The default LAN address is reported by the PF via
+        * ndo_get_vf_config.
+        */
+       if (is_unicast_ether_addr(mac_addr))
                ether_addr_copy(vf->dflt_lan_addr.addr, mac_addr);
 
        vf->num_mac++;
-- 
2.26.2

Reply via email to