> -----Original Message----- > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Beilei Xing > Sent: Sunday 11 February 2018 06:56 > To: Zhang, Qi Z <qi.z.zh...@intel.com> > Cc: dev@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.x...@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.rybalche...@intel.com>