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

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

Merge branch '10GbE' of git://git.kernel.org/pub/scm/linux/kernel/git/jkirsher/net-queue 



Jeff Kirsher says:

====================
Intel Wired LAN Driver Updates 2018-06-11

This series contains fixes to ixgbe IPsec and MACVLAN.

Alex provides the 5 fixes in this series, starting with fixing an issue
where num_rx_pools was not being populated until after the queues and
interrupts were reinitialized when enabling MACVLAN interfaces.  Updated
to use CONFIG_XFRM_OFFLOAD instead of CONFIG_XFRM, since the code
requires CONFIG_XFRM_OFFLOAD to be enabled.  Moved the IPsec
initialization function to be more consistent with the placement of
similar initialization functions and before the call to reset the
hardware, which will clean up any link issues that may have been
introduced.  Fixed the boolean logic that was testing for transmit OR
receive ready bits, when it should have been testing for transmit AND
receive ready bits.  Fixed the bit definitions for SECTXSTAT and SECRXSTAT
registers and ensure that if IPsec is disabled on the part, do not
enable it.
====================

Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parents 3f2d67b6 421d954c

drivers/net/ethernet/intel/ixgbe/ixgbe.h

+2 −2
Original line number Diff line number Diff line
@@ -760,9 +760,9 @@ struct ixgbe_adapter { 
#define IXGBE_RSS_KEY_SIZE     40  /* size of RSS Hash Key in bytes */

	u32 *rss_key;



#ifdef CONFIG_XFRM

#ifdef CONFIG_XFRM_OFFLOAD

	struct ixgbe_ipsec *ipsec;

#endif /* CONFIG_XFRM */

#endif /* CONFIG_XFRM_OFFLOAD */

};



static inline u8 ixgbe_max_rss_indices(struct ixgbe_adapter *adapter)

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

+24 −10
Original line number Diff line number Diff line
@@ -158,7 +158,16 @@ static void ixgbe_ipsec_stop_data(struct ixgbe_adapter *adapter) 
	reg |= IXGBE_SECRXCTRL_RX_DIS;

	IXGBE_WRITE_REG(hw, IXGBE_SECRXCTRL, reg);



	IXGBE_WRITE_FLUSH(hw);

	/* If both Tx and Rx are ready there are no packets

	 * that we need to flush so the loopback configuration

	 * below is not necessary.

	 */

	t_rdy = IXGBE_READ_REG(hw, IXGBE_SECTXSTAT) &

		IXGBE_SECTXSTAT_SECTX_RDY;

	r_rdy = IXGBE_READ_REG(hw, IXGBE_SECRXSTAT) &

		IXGBE_SECRXSTAT_SECRX_RDY;

	if (t_rdy && r_rdy)

		return;



	/* If the tx fifo doesn't have link, but still has data,

	 * we can't clear the tx sec block.  Set the MAC loopback
@@ -185,7 +194,7 @@ static void ixgbe_ipsec_stop_data(struct ixgbe_adapter *adapter) 
			IXGBE_SECTXSTAT_SECTX_RDY;

		r_rdy = IXGBE_READ_REG(hw, IXGBE_SECRXSTAT) &

			IXGBE_SECRXSTAT_SECRX_RDY;

	} while (!t_rdy && !r_rdy && limit--);

	} while (!(t_rdy && r_rdy) && limit--);



	/* undo loopback if we played with it earlier */

	if (!link) {
@@ -966,10 +975,22 @@ void ixgbe_ipsec_rx(struct ixgbe_ring *rx_ring, 
 **/

void ixgbe_init_ipsec_offload(struct ixgbe_adapter *adapter)

{

	struct ixgbe_hw *hw = &adapter->hw;

	struct ixgbe_ipsec *ipsec;

	u32 t_dis, r_dis;

	size_t size;



	if (adapter->hw.mac.type == ixgbe_mac_82598EB)

	if (hw->mac.type == ixgbe_mac_82598EB)

		return;



	/* If there is no support for either Tx or Rx offload

	 * we should not be advertising support for IPsec.

	 */

	t_dis = IXGBE_READ_REG(hw, IXGBE_SECTXSTAT) &

		IXGBE_SECTXSTAT_SECTX_OFF_DIS;

	r_dis = IXGBE_READ_REG(hw, IXGBE_SECRXSTAT) &

		IXGBE_SECRXSTAT_SECRX_OFF_DIS;

	if (t_dis || r_dis)

		return;



	ipsec = kzalloc(sizeof(*ipsec), GFP_KERNEL);
@@ -1001,13 +1022,6 @@ void ixgbe_init_ipsec_offload(struct ixgbe_adapter *adapter) 


	adapter->netdev->xfrmdev_ops = &ixgbe_xfrmdev_ops;



#define IXGBE_ESP_FEATURES	(NETIF_F_HW_ESP | \

				 NETIF_F_HW_ESP_TX_CSUM | \

				 NETIF_F_GSO_ESP)



	adapter->netdev->features |= IXGBE_ESP_FEATURES;

	adapter->netdev->hw_enc_features |= IXGBE_ESP_FEATURES;



	return;



err2:

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

+8 −0
Original line number Diff line number Diff line
@@ -593,6 +593,14 @@ static bool ixgbe_set_sriov_queues(struct ixgbe_adapter *adapter) 
	}



#endif

	/* To support macvlan offload we have to use num_tc to

	 * restrict the queues that can be used by the device.

	 * By doing this we can avoid reporting a false number of

	 * queues.

	 */

	if (vmdq_i > 1)

		netdev_set_num_tc(adapter->netdev, 1);



	/* populate TC0 for use by pool 0 */

	netdev_set_tc_queue(adapter->netdev, 0,

			    adapter->num_rx_queues_per_pool, 0);

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

+10 −11
Original line number Diff line number Diff line
@@ -6117,6 +6117,7 @@ static int ixgbe_sw_init(struct ixgbe_adapter *adapter, 
#ifdef CONFIG_IXGBE_DCB

	ixgbe_init_dcb(adapter);

#endif

	ixgbe_init_ipsec_offload(adapter);



	/* default flow control settings */

	hw->fc.requested_mode = ixgbe_fc_full;
@@ -8822,14 +8823,6 @@ int ixgbe_setup_tc(struct net_device *dev, u8 tc) 
	} else {

		netdev_reset_tc(dev);



		/* To support macvlan offload we have to use num_tc to

		 * restrict the queues that can be used by the device.

		 * By doing this we can avoid reporting a false number of

		 * queues.

		 */

		if (!tc && adapter->num_rx_pools > 1)

			netdev_set_num_tc(dev, 1);



		if (adapter->hw.mac.type == ixgbe_mac_82598EB)

			adapter->hw.fc.requested_mode = adapter->last_lfc_mode;
@@ -9904,7 +9897,7 @@ ixgbe_features_check(struct sk_buff *skb, struct net_device *dev, 
	 * the TSO, so it's the exception.

	 */

	if (skb->encapsulation && !(features & NETIF_F_TSO_MANGLEID)) {

#ifdef CONFIG_XFRM

#ifdef CONFIG_XFRM_OFFLOAD

		if (!skb->sp)

#endif

			features &= ~NETIF_F_TSO;
@@ -10437,6 +10430,14 @@ static int ixgbe_probe(struct pci_dev *pdev, const struct pci_device_id *ent) 
	if (hw->mac.type >= ixgbe_mac_82599EB)

		netdev->features |= NETIF_F_SCTP_CRC;



#ifdef CONFIG_XFRM_OFFLOAD

#define IXGBE_ESP_FEATURES	(NETIF_F_HW_ESP | \

				 NETIF_F_HW_ESP_TX_CSUM | \

				 NETIF_F_GSO_ESP)



	if (adapter->ipsec)

		netdev->features |= IXGBE_ESP_FEATURES;

#endif

	/* copy netdev features into list of user selectable features */

	netdev->hw_features |= netdev->features |

			       NETIF_F_HW_VLAN_CTAG_FILTER |
@@ -10499,8 +10500,6 @@ static int ixgbe_probe(struct pci_dev *pdev, const struct pci_device_id *ent) 
					 NETIF_F_FCOE_MTU;

	}

#endif /* IXGBE_FCOE */

	ixgbe_init_ipsec_offload(adapter);



	if (adapter->flags2 & IXGBE_FLAG2_RSC_CAPABLE)

		netdev->hw_features |= NETIF_F_LRO;

	if (adapter->flags2 & IXGBE_FLAG2_RSC_ENABLED)

drivers/net/ethernet/intel/ixgbe/ixgbe_type.h

+4 −2
Original line number Diff line number Diff line
@@ -599,13 +599,15 @@ struct ixgbe_nvm_version { 
#define IXGBE_SECTXCTRL_STORE_FORWARD   0x00000004



#define IXGBE_SECTXSTAT_SECTX_RDY       0x00000001

#define IXGBE_SECTXSTAT_ECC_TXERR       0x00000002

#define IXGBE_SECTXSTAT_SECTX_OFF_DIS   0x00000002

#define IXGBE_SECTXSTAT_ECC_TXERR       0x00000004



#define IXGBE_SECRXCTRL_SECRX_DIS       0x00000001

#define IXGBE_SECRXCTRL_RX_DIS          0x00000002



#define IXGBE_SECRXSTAT_SECRX_RDY       0x00000001

#define IXGBE_SECRXSTAT_ECC_RXERR       0x00000002

#define IXGBE_SECRXSTAT_SECRX_OFF_DIS   0x00000002

#define IXGBE_SECRXSTAT_ECC_RXERR       0x00000004



/* LinkSec (MacSec) Registers */

#define IXGBE_LSECTXCAP         0x08A00