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

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

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



Jeff Kirsher says:

====================
40GbE Intel Wired LAN Driver Updates 2016-09-22

This series contains updates to i40e and i40evf only.

Sridhar fixes link state event handling by updating the carrier and
starts/stops the Tx queues based on the link state notification from PF.

Brady fixes an issue where a user defined RSS hash key was not being
set because a user defined indirection table is not supplied when changing
the hash key, so if an indirection table is not supplied now, then a
default one is created and the hash key is correctly set.  Also fixed
an issue where when NPAR was enabled, we were still using pf->mac_seid
to perform the dump port query. Instead, go through the VSI to determine
the correct ID to use in either case.

Mitch provides one fix where a conditional return code was reversed, so
he does a "switheroo" to fix the issue.

Carolyn has two fixes, first fixes an issue in the virt channel code,
where a return code was not checked for NULL when applicable.  Second,
fixes an issue where we were byte swapping the port parameter, then
byte swapping it again in function execution.

Colin Ian King fixes a potential NULL pointer dereference.

Bimmy changes up i40evf_up_complete() to be void since it always returns
success anyways, which allows cleaning up of code which checked the
return code from this function.

Alex fixed an issue where the driver was incorrectly assuming that we
would always be pulling no more than 1 descriptor from each fragment.
So to correct this, we just need to make certain to test all the way to
the end of the fragments as it is possible for us to span 2 descriptors
in the block before us so we need to guarantee that even the last 6
descriptors have enough data to fill a full frame.

v2: dropped patches 1-3, 10 and 12 from the original series since Or
    Gerlitz pointed out several areas of improvement in the implementation
    of the VF Port representor netdev.  Sridhar is re-working the series
    for later submission.
====================

Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parents 1ad0751d 903e6832

drivers/net/ethernet/intel/i40e/i40e.h

+2 −0
Original line number Diff line number Diff line
@@ -701,6 +701,8 @@ void i40e_do_reset_safe(struct i40e_pf *pf, u32 reset_flags); 
void i40e_do_reset(struct i40e_pf *pf, u32 reset_flags);

int i40e_config_rss(struct i40e_vsi *vsi, u8 *seed, u8 *lut, u16 lut_size);

int i40e_get_rss(struct i40e_vsi *vsi, u8 *seed, u8 *lut, u16 lut_size);

void i40e_fill_rss_lut(struct i40e_pf *pf, u8 *lut,

		       u16 rss_table_size, u16 rss_size);

struct i40e_vsi *i40e_find_vsi_from_id(struct i40e_pf *pf, u16 id);

void i40e_update_stats(struct i40e_vsi *vsi);

void i40e_update_eth_stats(struct i40e_vsi *vsi);

drivers/net/ethernet/intel/i40e/i40e_debugfs.c

+6 −1
Original line number Diff line number Diff line
@@ -1054,6 +1054,7 @@ static ssize_t i40e_dbg_command_write(struct file *filp, 
			struct i40e_dcbx_config *r_cfg =

						&pf->hw.remote_dcbx_config;

			int i, ret;

			u32 switch_id;



			bw_data = kzalloc(sizeof(

				    struct i40e_aqc_query_port_ets_config_resp),
@@ -1063,8 +1064,12 @@ static ssize_t i40e_dbg_command_write(struct file *filp, 
				goto command_write_done;

			}



			vsi = pf->vsi[pf->lan_vsi];

			switch_id =

				vsi->info.switch_id & I40E_AQ_VSI_SW_ID_MASK;



			ret = i40e_aq_query_port_ets_config(&pf->hw,

							    pf->mac_seid,

							    switch_id,

							    bw_data, NULL);

			if (ret) {

				dev_info(&pf->pdev->dev,

drivers/net/ethernet/intel/i40e/i40e_ethtool.c

+7 −5
Original line number Diff line number Diff line
@@ -2922,15 +2922,13 @@ static int i40e_set_rxfh(struct net_device *netdev, const u32 *indir, 
{

	struct i40e_netdev_priv *np = netdev_priv(netdev);

	struct i40e_vsi *vsi = np->vsi;

	struct i40e_pf *pf = vsi->back;

	u8 *seed = NULL;

	u16 i;



	if (hfunc != ETH_RSS_HASH_NO_CHANGE && hfunc != ETH_RSS_HASH_TOP)

		return -EOPNOTSUPP;



	if (!indir)

		return 0;



	if (key) {

		if (!vsi->rss_hkey_user) {

			vsi->rss_hkey_user = kzalloc(I40E_HKEY_ARRAY_SIZE,
@@ -2948,8 +2946,12 @@ static int i40e_set_rxfh(struct net_device *netdev, const u32 *indir, 
	}



	/* Each 32 bits pointed by 'indir' is stored with a lut entry */

	if (indir)

		for (i = 0; i < I40E_HLUT_ARRAY_SIZE; i++)

			vsi->rss_lut_user[i] = (u8)(indir[i]);

	else

		i40e_fill_rss_lut(pf, vsi->rss_lut_user, I40E_HLUT_ARRAY_SIZE,

				  vsi->rss_size);



	return i40e_config_rss(vsi, seed, vsi->rss_lut_user,

			       I40E_HLUT_ARRAY_SIZE);

drivers/net/ethernet/intel/i40e/i40e_main.c

+5 −7
Original line number Diff line number Diff line
@@ -57,8 +57,6 @@ static int i40e_setup_pf_switch(struct i40e_pf *pf, bool reinit); 
static int i40e_setup_misc_vector(struct i40e_pf *pf);

static void i40e_determine_queue_usage(struct i40e_pf *pf);

static int i40e_setup_pf_filter_control(struct i40e_pf *pf);

static void i40e_fill_rss_lut(struct i40e_pf *pf, u8 *lut,

			      u16 rss_table_size, u16 rss_size);

static void i40e_fdir_sb_setup(struct i40e_pf *pf);

static int i40e_veb_get_bw_info(struct i40e_veb *veb);
@@ -7156,7 +7154,7 @@ static void i40e_sync_udp_filters_subtask(struct i40e_pf *pf) 
			pf->pending_udp_bitmap &= ~BIT_ULL(i);

			port = pf->udp_ports[i].index;

			if (port)

				ret = i40e_aq_add_udp_tunnel(hw, ntohs(port),

				ret = i40e_aq_add_udp_tunnel(hw, port,

							pf->udp_ports[i].type,

							NULL, NULL);

			else
@@ -8244,7 +8242,7 @@ int i40e_get_rss(struct i40e_vsi *vsi, u8 *seed, u8 *lut, u16 lut_size) 
 * @rss_table_size: Lookup table size

 * @rss_size: Range of queue number for hashing

 */

static void i40e_fill_rss_lut(struct i40e_pf *pf, u8 *lut,

void i40e_fill_rss_lut(struct i40e_pf *pf, u8 *lut,

		       u16 rss_table_size, u16 rss_size)

{

	u16 i;

drivers/net/ethernet/intel/i40e/i40e_txrx.c

+2 −5
Original line number Diff line number Diff line
@@ -2621,9 +2621,7 @@ bool __i40e_chk_linearize(struct sk_buff *skb) 
		return false;



	/* We need to walk through the list and validate that each group

	 * of 6 fragments totals at least gso_size.  However we don't need

	 * to perform such validation on the last 6 since the last 6 cannot

	 * inherit any data from a descriptor after them.

	 * of 6 fragments totals at least gso_size.

	 */

	nr_frags -= I40E_MAX_BUFFER_TXD - 2;

	frag = &skb_shinfo(skb)->frags[0];
@@ -2654,8 +2652,7 @@ bool __i40e_chk_linearize(struct sk_buff *skb) 
		if (sum < 0)

			return true;



		/* use pre-decrement to avoid processing last fragment */

		if (!--nr_frags)

		if (!nr_frags--)

			break;



		sum -= skb_frag_size(stale++);