Commit 58b8b042 authored by Alexander Duyck's avatar Alexander Duyck Committed by David S. Miller
Browse files

igb: check both function bits in status register in wol exception 



The ethtool code for enabling Wake on Lan was not correctly checking the
status register bits so as a result ports 0 and 2 were both being allowed
to set WOL to enabled even though it is only supported on the first port
for our adapters.

Signed-off-by: default avatarAlexander Duyck <alexander.h.duyck@intel.com>
Signed-off-by: default avatarJeff Kirsher <jeffrey.t.kirsher@intel.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent d405ea3e

drivers/net/igb/igb_ethtool.c

+1 −1
Original line number Diff line number Diff line
@@ -1795,7 +1795,7 @@ static int igb_wol_exclusion(struct igb_adapter *adapter, 
		/* dual port cards only support WoL on port A from now on

		 * unless it was enabled in the eeprom for port B

		 * so exclude FUNC_1 ports from having WoL enabled */

		if (rd32(E1000_STATUS) & E1000_STATUS_FUNC_1 &&

		if ((rd32(E1000_STATUS) & E1000_STATUS_FUNC_MASK) &&

		    !adapter->eeprom_wol) {

			wol->supported = 0;

			break;