]> bbs.cooldavid.org Git - net-next-2.6.git/commit
ixgbe: fix inconsistent SFP/SFP+ failure results.
authorDon Skidmore <donald.c.skidmore@intel.com>
Thu, 2 Jul 2009 12:50:52 +0000 (12:50 +0000)
committerDavid S. Miller <davem@davemloft.net>
Sat, 4 Jul 2009 02:11:05 +0000 (19:11 -0700)
commit5da43c1a78cbc16518017dfb058da6ec4829781b
tree039a95aaedc5624904affa67ef1dfe8ed8b525ad
parent19343de279d28e4ed039c8ad00510d19dc4d6d12
ixgbe: fix inconsistent SFP/SFP+ failure results.

Currently if we loaded the driver, insert an unsupported module, and then
attempt to "ifconfig up" the device it will be brought down but the netdev
would not be unregistered. This behavior is different than all other
code paths. This patch corrects that by down'ing the device and then
scheduling the sfp_config_module_task tasklet. The tasklet will detect
this condition (like it does with other code paths) and do the
unregister_netdev().

I also removed the log message as this condition (an unsupported SFP+
module) will be logged in sfp_config_module_task.

Signed-off-by: Don Skidmore <donald.c.skidmore@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ixgbe/ixgbe_main.c