Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit c9771bfd authored by David S. Miller's avatar David S. Miller
Browse files

Merge branch 'master' of git://git.kernel.org/pub/scm/linux/kernel/git/jkirsher/net 



Jeff Kirsher says:

====================
Intel Wired LAN Driver Updates

This series contains updates to ixgbe and e1000e.

Jacob provides a ixgbe patch to fix the configure_rx patch to properly
disable RSC hardware logic when a user disables it.  Previously we only
disabled RSC in the queue settings, but this does not fully disable
hardware RSC logic which can lead to unexpected performance issues.

Emil provides three fixes for ixgbe.  First fixes the ethtool loopback
test when DCB is enabled, where the frames may be modified on Tx
(by adding VLAN tag) which will fail the check on receive.  Then a fix
for QSFP+ modules, limit the speed setting to advertise only one speed
at a time since the QSFP+ modules do not support auto negotiation.
Lastly, resolve an issue where the driver will display incorrect info
for QSFP+ modules that were inserted after the driver has been loaded.

David Ertman provides to fixes for e1000e, one removes a comparison to
the boolean value true where evaluating the lvalue will produce the
same result.  The other fixes an error in the calculation of the
rar_entry_count, which causes a write of unkown/undefined register
space in the MAC to unknown/undefined register space in the PHY.
====================

Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parents 89473129 c3a0dce3

drivers/net/ethernet/intel/e1000e/ethtool.c

+8 −0
Original line number Diff line number Diff line
@@ -922,6 +922,14 @@ static int e1000_reg_test(struct e1000_adapter *adapter, u64 *data) 
			else

				mask &= ~(1 << 30);

		}

		if (mac->type == e1000_pch2lan) {

			/* SHRAH[0,1,2] different than previous */

			if (i == 7)

				mask &= 0xFFF4FFFF;

			/* SHRAH[3] different than SHRAH[0,1,2] */

			if (i == 10)

				mask |= (1 << 30);

		}



		REG_PATTERN_TEST_ARRAY(E1000_RA, ((i << 1) + 1), mask,

				       0xFFFFFFFF);

drivers/net/ethernet/intel/e1000e/ich8lan.c

+8 −5
Original line number Diff line number Diff line
@@ -1371,7 +1371,10 @@ static void e1000_rar_set_pch2lan(struct e1000_hw *hw, u8 *addr, u32 index) 
		return;

	}



	if (index < hw->mac.rar_entry_count) {

	/* RAR[1-6] are owned by manageability.  Skip those and program the

	 * next address into the SHRA register array.

	 */

	if (index < (u32)(hw->mac.rar_entry_count - 6)) {

		s32 ret_val;



		ret_val = e1000_acquire_swflag_ich8lan(hw);
@@ -1962,8 +1965,8 @@ void e1000_copy_rx_addrs_to_phy_ich8lan(struct e1000_hw *hw) 
	if (ret_val)

		goto release;



	/* Copy both RAL/H (rar_entry_count) and SHRAL/H (+4) to PHY */

	for (i = 0; i < (hw->mac.rar_entry_count + 4); i++) {

	/* Copy both RAL/H (rar_entry_count) and SHRAL/H to PHY */

	for (i = 0; i < (hw->mac.rar_entry_count); i++) {

		mac_reg = er32(RAL(i));

		hw->phy.ops.write_reg_page(hw, BM_RAR_L(i),

					   (u16)(mac_reg & 0xFFFF));
@@ -2007,10 +2010,10 @@ s32 e1000_lv_jumbo_workaround_ich8lan(struct e1000_hw *hw, bool enable) 
		return ret_val;



	if (enable) {

		/* Write Rx addresses (rar_entry_count for RAL/H, +4 for

		/* Write Rx addresses (rar_entry_count for RAL/H, and

		 * SHRAL/H) and initial CRC values to the MAC

		 */

		for (i = 0; i < (hw->mac.rar_entry_count + 4); i++) {

		for (i = 0; i < hw->mac.rar_entry_count; i++) {

			u8 mac_addr[ETH_ALEN] = { 0 };

			u32 addr_high, addr_low;

drivers/net/ethernet/intel/e1000e/ich8lan.h

+1 −1
Original line number Diff line number Diff line
@@ -98,7 +98,7 @@ 
#define PCIE_ICH8_SNOOP_ALL	PCIE_NO_SNOOP_ALL



#define E1000_ICH_RAR_ENTRIES	7

#define E1000_PCH2_RAR_ENTRIES	5	/* RAR[0], SHRA[0-3] */

#define E1000_PCH2_RAR_ENTRIES	11      /* RAR[0-6], SHRA[0-3] */

#define E1000_PCH_LPT_RAR_ENTRIES	12	/* RAR[0], SHRA[0-10] */



#define PHY_PAGE_SHIFT		5

drivers/net/ethernet/intel/e1000e/netdev.c

+1 −1
Original line number Diff line number Diff line
@@ -4868,7 +4868,7 @@ static void e1000_watchdog_task(struct work_struct *work) 
			 */

			if ((hw->phy.type == e1000_phy_igp_3 ||

			     hw->phy.type == e1000_phy_bm) &&

			    (hw->mac.autoneg == true) &&

			    hw->mac.autoneg &&

			    (adapter->link_speed == SPEED_10 ||

			     adapter->link_speed == SPEED_100) &&

			    (adapter->link_duplex == HALF_DUPLEX)) {

drivers/net/ethernet/intel/ixgbe/ixgbe_ethtool.c

+25 −0
Original line number Diff line number Diff line
@@ -160,6 +160,13 @@ static int ixgbe_get_settings(struct net_device *netdev, 
	bool autoneg = false;

	bool link_up;



	/* SFP type is needed for get_link_capabilities */

	if (hw->phy.media_type & (ixgbe_media_type_fiber |

				  ixgbe_media_type_fiber_qsfp)) {

		if (hw->phy.sfp_type == ixgbe_sfp_type_not_present)

				hw->phy.ops.identify_sfp(hw);

	}



	hw->mac.ops.get_link_capabilities(hw, &supported_link, &autoneg);



	/* set the supported link speeds */
@@ -186,6 +193,11 @@ static int ixgbe_get_settings(struct net_device *netdev, 
			ecmd->advertising |= ADVERTISED_1000baseT_Full;

		if (supported_link & IXGBE_LINK_SPEED_100_FULL)

			ecmd->advertising |= ADVERTISED_100baseT_Full;



		if (hw->phy.multispeed_fiber && !autoneg) {

			if (supported_link & IXGBE_LINK_SPEED_10GB_FULL)

				ecmd->advertising = ADVERTISED_10000baseT_Full;

		}

	}



	if (autoneg) {
@@ -314,6 +326,14 @@ static int ixgbe_set_settings(struct net_device *netdev, 
		if (ecmd->advertising & ~ecmd->supported)

			return -EINVAL;



		/* only allow one speed at a time if no autoneg */

		if (!ecmd->autoneg && hw->phy.multispeed_fiber) {

			if (ecmd->advertising ==

			    (ADVERTISED_10000baseT_Full |

			     ADVERTISED_1000baseT_Full))

				return -EINVAL;

		}



		old = hw->phy.autoneg_advertised;

		advertised = 0;

		if (ecmd->advertising & ADVERTISED_10000baseT_Full)
@@ -1805,6 +1825,10 @@ static int ixgbe_run_loopback_test(struct ixgbe_adapter *adapter) 
	unsigned int size = 1024;

	netdev_tx_t tx_ret_val;

	struct sk_buff *skb;

	u32 flags_orig = adapter->flags;



	/* DCB can modify the frames on Tx */

	adapter->flags &= ~IXGBE_FLAG_DCB_ENABLED;



	/* allocate test skb */

	skb = alloc_skb(size, GFP_KERNEL);
@@ -1857,6 +1881,7 @@ static int ixgbe_run_loopback_test(struct ixgbe_adapter *adapter) 


	/* free the original skb */

	kfree_skb(skb);

	adapter->flags = flags_orig;



	return ret_val;

}