igb: Don't give VFs random MAC addresses
authorMitch A Williams <mitch.a.williams@intel.com>
Fri, 18 Jan 2013 08:57:20 +0000 (08:57 +0000)
committerJeff Kirsher <jeffrey.t.kirsher@intel.com>
Mon, 28 Jan 2013 11:17:57 +0000 (03:17 -0800)
If the user has not assigned a MAC address to a VM, then don't give it a
random one. Instead, just give it zeros and let it figure out what to do
with them.

Signed-off-by: Mitch Williams <mitch.a.williams@intel.com>
CC: Andy Gospodarek <andy@greyhouse.net>
CC: Stefan Assmann <sassmann@kpanic.de>
Tested-by: Aaron Brown <aaron.f.brown@intel.com>
Tested-by: Stefan Assmann <sassmann@redhat.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
drivers/net/ethernet/intel/igb/igb_main.c

index b81a953709142de448786c33f17309bd6beb8e63..a59e63040f316052ef47c40ec2b21f825591680c 100644 (file)
@@ -5197,7 +5197,7 @@ static int igb_vf_configure(struct igb_adapter *adapter, int vf)
 {
        unsigned char mac_addr[ETH_ALEN];
 
-       eth_random_addr(mac_addr);
+       eth_zero_addr(mac_addr);
        igb_set_vf_mac(adapter, vf, mac_addr);
 
        return 0;
@@ -5550,9 +5550,9 @@ static void igb_vf_reset_event(struct igb_adapter *adapter, u32 vf)
 {
        unsigned char *vf_mac = adapter->vf_data[vf].vf_mac_addresses;
 
-       /* generate a new mac address as we were hotplug removed/added */
+       /* clear mac address as we were hotplug removed/added */
        if (!(adapter->vf_data[vf].flags & IGB_VF_FLAG_PF_SET_MAC))
-               eth_random_addr(vf_mac);
+               eth_zero_addr(vf_mac);
 
        /* process remaining reset events */
        igb_vf_reset(adapter, vf);