]> git.baikalelectronics.ru Git - kernel.git/commit
igb: Indicate failure on vf reset for empty mac address
authorAlexander Graf <agraf@suse.de>
Thu, 9 Oct 2014 05:33:55 +0000 (05:33 +0000)
committerJeff Kirsher <jeffrey.t.kirsher@intel.com>
Fri, 23 Jan 2015 02:10:23 +0000 (18:10 -0800)
commit84917a8c2ed833a119eb597fe5f917c94106e60b
treec19d635c0b4ab1dc28083db1b1068feb2d053ad7
parent91f768c532ae533b5ac996220a3faa590a32e857
igb: Indicate failure on vf reset for empty mac address

Commit 5f36417a changed the igb driver to expose a zero (empty) mac
address to the VF on reset rather than a random one.

However, that behavioral change also requires igbvf driver changes
which can be hard especially when we want to talk to proprietary
guest OSs.

Looking at the code previous to the commit in Linux that made igbvf
work with empty mac addresses (cda53ad), we can see that on reset
failure the driver will try to generate a new mac address with both
the old and the new code.

Furthermore, ixgbe does send reset failure when it detects an empty
mac address (ba68d06c6).

So I think it's safe to make igb behave the same. With this patch I
can successfully run a Windows 8.1 guest with an empty mac address
and an assigned igbvf device that has no mac address set by the host.

If anyone is aware of a guest driver that chokes on NACK returns of
VF RESET commands, please speak up.

Signed-off-by: Alexander Graf <agraf@suse.de>
Tested-by: Aaron Brown <aaron.f.brown@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
drivers/net/ethernet/intel/igb/igb_main.c