[dpdk-dev] [PATCH] net/i40e: add log when setting input set

Rybalchenko, Kirill kirill.rybalchenko at intel.com
Wed Mar 14 16:46:49 CET 2018



> -----Original Message-----
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Beilei Xing
> Sent: Sunday 11 February 2018 06:56
> To: Zhang, Qi Z <qi.z.zhang at intel.com>
> Cc: dev at dpdk.org
> Subject: [dpdk-dev] [PATCH] net/i40e: add log when setting input set
> 
> This patch adds log when setting input set since global configuration is
> changed.
> 
> Signed-off-by: Beilei Xing <beilei.xing at intel.com>
> ---
>  drivers/net/i40e/rte_pmd_i40e.c | 30 +++++++++++++++++++++---------
>  1 file changed, 21 insertions(+), 9 deletions(-)
> 
> diff --git a/drivers/net/i40e/rte_pmd_i40e.c
> b/drivers/net/i40e/rte_pmd_i40e.c index dae59e6..1288c51 100644
> --- a/drivers/net/i40e/rte_pmd_i40e.c
> +++ b/drivers/net/i40e/rte_pmd_i40e.c
> @@ -3071,6 +3071,7 @@ rte_pmd_i40e_inset_set(uint16_t port, uint8_t
> pctype,  {
>  	struct rte_eth_dev *dev;
>  	struct i40e_hw *hw;
> +	struct i40e_pf *pf;
>  	uint64_t inset_reg;
>  	uint32_t mask_reg[2];
>  	int i;
> @@ -3086,6 +3087,12 @@ rte_pmd_i40e_inset_set(uint16_t port, uint8_t
> pctype,
>  		return -EINVAL;
> 
>  	hw = I40E_DEV_PRIVATE_TO_HW(dev->data->dev_private);
> +	pf = I40E_DEV_PRIVATE_TO_PF(dev->data->dev_private);
> +
> +	if (pf->support_multi_driver) {
> +		PMD_DRV_LOG(ERR, "Input set configuration is not
> supported.");
> +		return -ENOTSUP;
> +	}
> 
>  	/* Clear mask first */
>  	for (i = 0; i < 2; i++)
> @@ -3098,14 +3105,17 @@ rte_pmd_i40e_inset_set(uint16_t port, uint8_t
> pctype,
> 
>  	switch (inset_type) {
>  	case INSET_HASH:
> -		i40e_check_write_reg(hw, I40E_GLQF_HASH_INSET(0,
> pctype),
> -				     (uint32_t)(inset_reg & UINT32_MAX));
> -		i40e_check_write_reg(hw, I40E_GLQF_HASH_INSET(1,
> pctype),
> -				     (uint32_t)((inset_reg >>
> -					      I40E_32_BIT_WIDTH) &
> UINT32_MAX));
> +		i40e_check_write_global_reg(hw,
> I40E_GLQF_HASH_INSET(0, pctype),
> +					    (uint32_t)(inset_reg &
> UINT32_MAX));
> +		i40e_check_write_global_reg(hw,
> I40E_GLQF_HASH_INSET(1, pctype),
> +					    (uint32_t)((inset_reg >>
> +					     I40E_32_BIT_WIDTH) &
> UINT32_MAX));
>  		for (i = 0; i < 2; i++)
> -			i40e_check_write_reg(hw, I40E_GLQF_HASH_MSK(i,
> pctype),
> -					     mask_reg[i]);
> +			i40e_check_write_global_reg(hw,
> +						  I40E_GLQF_HASH_MSK(i,
> pctype),
> +						  mask_reg[i]);
> +		i40e_global_cfg_warning(I40E_WARNING_HASH_INSET);
> +		i40e_global_cfg_warning(I40E_WARNING_HASH_MSK);
>  		break;
>  	case INSET_FDIR:
>  		i40e_check_write_reg(hw, I40E_PRTQF_FD_INSET(pctype,
> 0), @@ -3114,8 +3124,10 @@ rte_pmd_i40e_inset_set(uint16_t port, uint8_t
> pctype,
>  				     (uint32_t)((inset_reg >>
>  					      I40E_32_BIT_WIDTH) &
> UINT32_MAX));
>  		for (i = 0; i < 2; i++)
> -			i40e_check_write_reg(hw, I40E_GLQF_FD_MSK(i,
> pctype),
> -					     mask_reg[i]);
> +			i40e_check_write_global_reg(hw,
> +						    I40E_GLQF_FD_MSK(i,
> pctype),
> +						    mask_reg[i]);
> +		i40e_global_cfg_warning(I40E_WARNING_FD_MSK);
>  		break;
>  	case INSET_FDIR_FLX:
>  		i40e_check_write_reg(hw,
> I40E_PRTQF_FD_FLXINSET(pctype),
> --
> 2.5.5


Reviewed-by: Kirill Rybalchenko <kirill.rybalchenko at intel.com>





More information about the dev mailing list