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

Commit 28c47fd2 authored by Slawomir Laba's avatar Slawomir Laba Committed by Greg Kroah-Hartman
Browse files

i40e: Fix ethtool rx-flow-hash setting for X722 



[ Upstream commit 54b5af5a438076082d482cab105b1bd484ab5074 ]

When enabling flow type for RSS hash via ethtool:

ethtool -N $pf rx-flow-hash tcp4|tcp6|udp4|udp6 s|d

the driver would fail to setup this setting on X722
device since it was using the mask on the register
dedicated for X710 devices.

Apply a different mask on the register when setting the
RSS hash for the X722 device.

When displaying the flow types enabled via ethtool:

ethtool -n $pf rx-flow-hash tcp4|tcp6|udp4|udp6

the driver would print wrong values for X722 device.

Fix this issue by testing masks for X722 device in
i40e_get_rss_hash_opts function.

Fixes: eb0dd6e4 ("i40e: Allow RSS Hash set with less than four parameters")
Signed-off-by: default avatarSlawomir Laba <slawomirx.laba@intel.com>
Signed-off-by: default avatarMichal Jaron <michalx.jaron@intel.com>
Signed-off-by: default avatarMateusz Palczewski <mateusz.palczewski@intel.com>
Tested-by: Gurucharan <gurucharanx.g@intel.com> (A Contingent worker at Intel)
Signed-off-by: default avatarJacob Keller <jacob.e.keller@intel.com>
Link: https://lore.kernel.org/r/20221024100526.1874914-1-jacob.e.keller@intel.com


Signed-off-by: default avatarJakub Kicinski <kuba@kernel.org>
Signed-off-by: default avatarSasha Levin <sashal@kernel.org>
parent d303dabe

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

+23 −8
Original line number Original line Diff line number Diff line
@@ -3082,10 +3082,17 @@ static int i40e_get_rss_hash_opts(struct i40e_pf *pf, struct ethtool_rxnfc *cmd) 




		if (cmd->flow_type == TCP_V4_FLOW ||

		if (cmd->flow_type == TCP_V4_FLOW ||

		    cmd->flow_type == UDP_V4_FLOW) {

		    cmd->flow_type == UDP_V4_FLOW) {

			if (hw->mac.type == I40E_MAC_X722) {

				if (i_set & I40E_X722_L3_SRC_MASK)

					cmd->data |= RXH_IP_SRC;

				if (i_set & I40E_X722_L3_DST_MASK)

					cmd->data |= RXH_IP_DST;

			} else {

				if (i_set & I40E_L3_SRC_MASK)

				if (i_set & I40E_L3_SRC_MASK)

					cmd->data |= RXH_IP_SRC;

					cmd->data |= RXH_IP_SRC;

				if (i_set & I40E_L3_DST_MASK)

				if (i_set & I40E_L3_DST_MASK)

					cmd->data |= RXH_IP_DST;

					cmd->data |= RXH_IP_DST;

			}

		} else if (cmd->flow_type == TCP_V6_FLOW ||

		} else if (cmd->flow_type == TCP_V6_FLOW ||

			  cmd->flow_type == UDP_V6_FLOW) {

			  cmd->flow_type == UDP_V6_FLOW) {

			if (i_set & I40E_L3_V6_SRC_MASK)

			if (i_set & I40E_L3_V6_SRC_MASK)
@@ -3392,12 +3399,15 @@ static int i40e_get_rxnfc(struct net_device *netdev, struct ethtool_rxnfc *cmd, 




/**

/**

 * i40e_get_rss_hash_bits - Read RSS Hash bits from register

 * i40e_get_rss_hash_bits - Read RSS Hash bits from register

 * @hw: hw structure

 * @nfc: pointer to user request

 * @nfc: pointer to user request

 * @i_setc: bits currently set

 * @i_setc: bits currently set

 *

 *

 * Returns value of bits to be set per user request

 * Returns value of bits to be set per user request

 **/

 **/

static u64 i40e_get_rss_hash_bits(struct ethtool_rxnfc *nfc, u64 i_setc)

static u64 i40e_get_rss_hash_bits(struct i40e_hw *hw,

				  struct ethtool_rxnfc *nfc,

				  u64 i_setc)

{

{

	u64 i_set = i_setc;

	u64 i_set = i_setc;

	u64 src_l3 = 0, dst_l3 = 0;

	u64 src_l3 = 0, dst_l3 = 0;
@@ -3416,8 +3426,13 @@ static u64 i40e_get_rss_hash_bits(struct ethtool_rxnfc *nfc, u64 i_setc) 
		dst_l3 = I40E_L3_V6_DST_MASK;

		dst_l3 = I40E_L3_V6_DST_MASK;

	} else if (nfc->flow_type == TCP_V4_FLOW ||

	} else if (nfc->flow_type == TCP_V4_FLOW ||

		  nfc->flow_type == UDP_V4_FLOW) {

		  nfc->flow_type == UDP_V4_FLOW) {

		if (hw->mac.type == I40E_MAC_X722) {

			src_l3 = I40E_X722_L3_SRC_MASK;

			dst_l3 = I40E_X722_L3_DST_MASK;

		} else {

			src_l3 = I40E_L3_SRC_MASK;

			src_l3 = I40E_L3_SRC_MASK;

			dst_l3 = I40E_L3_DST_MASK;

			dst_l3 = I40E_L3_DST_MASK;

		}

	} else {

	} else {

		/* Any other flow type are not supported here */

		/* Any other flow type are not supported here */

		return i_set;

		return i_set;
@@ -3532,7 +3547,7 @@ static int i40e_set_rss_hash_opt(struct i40e_pf *pf, struct ethtool_rxnfc *nfc) 
					       flow_pctype)) |

					       flow_pctype)) |

			((u64)i40e_read_rx_ctl(hw, I40E_GLQF_HASH_INSET(1,

			((u64)i40e_read_rx_ctl(hw, I40E_GLQF_HASH_INSET(1,

					       flow_pctype)) << 32);

					       flow_pctype)) << 32);

		i_set = i40e_get_rss_hash_bits(nfc, i_setc);

		i_set = i40e_get_rss_hash_bits(&pf->hw, nfc, i_setc);

		i40e_write_rx_ctl(hw, I40E_GLQF_HASH_INSET(0, flow_pctype),

		i40e_write_rx_ctl(hw, I40E_GLQF_HASH_INSET(0, flow_pctype),

				  (u32)i_set);

				  (u32)i_set);

		i40e_write_rx_ctl(hw, I40E_GLQF_HASH_INSET(1, flow_pctype),

		i40e_write_rx_ctl(hw, I40E_GLQF_HASH_INSET(1, flow_pctype),

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

+4 −0
Original line number Original line Diff line number Diff line
@@ -1470,6 +1470,10 @@ struct i40e_lldp_variables { 
#define I40E_PFQF_CTL_0_HASHLUTSIZE_512	0x00010000

#define I40E_PFQF_CTL_0_HASHLUTSIZE_512	0x00010000





/* INPUT SET MASK for RSS, flow director, and flexible payload */

/* INPUT SET MASK for RSS, flow director, and flexible payload */

#define I40E_X722_L3_SRC_SHIFT		49

#define I40E_X722_L3_SRC_MASK		(0x3ULL << I40E_X722_L3_SRC_SHIFT)

#define I40E_X722_L3_DST_SHIFT		41

#define I40E_X722_L3_DST_MASK		(0x3ULL << I40E_X722_L3_DST_SHIFT)

#define I40E_L3_SRC_SHIFT		47

#define I40E_L3_SRC_SHIFT		47

#define I40E_L3_SRC_MASK		(0x3ULL << I40E_L3_SRC_SHIFT)

#define I40E_L3_SRC_MASK		(0x3ULL << I40E_L3_SRC_SHIFT)

#define I40E_L3_V6_SRC_SHIFT		43

#define I40E_L3_V6_SRC_SHIFT		43