ixgbe: fix inconsistent clearing of the multicast table
authorEmil Tantilov <emil.s.tantilov@intel.com>
Sat, 26 Oct 2013 08:13:20 +0000 (08:13 +0000)
committerJeff Kirsher <jeffrey.t.kirsher@intel.com>
Fri, 1 Nov 2013 13:27:38 +0000 (06:27 -0700)
This patch resolves an issue where the MTA table can be cleared when the
interface is reset while in promisc mode. As result IPv6 traffic between
VFs will be interrupted.

This patch makes the update of the MTA table unconditional to avoid the
inconsistent clearing on reset.

Signed-off-by: Emil Tantilov <emil.s.tantilov@intel.com>
Tested-by: Phil Schmitt <phillip.j.schmitt@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
drivers/net/ethernet/intel/ixgbe/ixgbe_main.c

index a7d1a1c43f129d75ee5ae9fdbddc668f21830ef9..5191b3ca9a2651df8c8d9f6caf8c7005aed87970 100644 (file)
@@ -3823,14 +3823,6 @@ void ixgbe_set_rx_mode(struct net_device *netdev)
                if (netdev->flags & IFF_ALLMULTI) {
                        fctrl |= IXGBE_FCTRL_MPE;
                        vmolr |= IXGBE_VMOLR_MPE;
-               } else {
-                       /*
-                        * Write addresses to the MTA, if the attempt fails
-                        * then we should just turn on promiscuous mode so
-                        * that we can at least receive multicast traffic
-                        */
-                       hw->mac.ops.update_mc_addr_list(hw, netdev);
-                       vmolr |= IXGBE_VMOLR_ROMPE;
                }
                ixgbe_vlan_filter_enable(adapter);
                hw->addr_ctrl.user_set_promisc = false;
@@ -3847,6 +3839,13 @@ void ixgbe_set_rx_mode(struct net_device *netdev)
                vmolr |= IXGBE_VMOLR_ROPE;
        }
 
+       /* Write addresses to the MTA, if the attempt fails
+        * then we should just turn on promiscuous mode so
+        * that we can at least receive multicast traffic
+        */
+       hw->mac.ops.update_mc_addr_list(hw, netdev);
+       vmolr |= IXGBE_VMOLR_ROMPE;
+
        if (adapter->num_vfs)
                ixgbe_restore_vf_multicasts(adapter);