[v2] net/i40e: solve the failure of vf vlan filtering

  When vf driver port promiscuous is turned on, the vlan filtering function
is invalid.
Through communication with PAE expert, this is a limitation of the i40e
chip. Before adding or removing VLANs, you must first disable unicast
promiscuous or multicast promiscuous, then operate the vlan, and finally
restore unicast promiscuous or multicast promiscuous state.

Signed-off-by: Qiming Chen <chenqiming_huawei@163.com>
---
 drivers/net/i40e/i40e_ethdev_vf.c | 23 +++++++++++++++++++++--
 1 file changed, 21 insertions(+), 2 deletions(-)
  

> -----Original Message-----
> From: dev <dev-bounces@dpdk.org> On Behalf Of Qiming Chen
> Sent: Tuesday, August 24, 2021 5:30 PM
> To: dev@dpdk.org
> Cc: Xing, Beilei <beilei.xing@intel.com>; Qiming Chen
> <chenqiming_huawei@163.com>
> Subject: [dpdk-dev] [PATCH v2] net/i40e: solve the failure of vf vlan filtering
> 
> When vf driver port promiscuous is turned on, the vlan filtering function is
> invalid.
> Through communication with PAE expert, this is a limitation of the i40e chip.
> Before adding or removing VLANs, you must first disable unicast
> promiscuous or multicast promiscuous, then operate the vlan, and finally
> restore unicast promiscuous or multicast promiscuous state.

Thanks for the patch.
But I heard from DPDK validation team that there's no vf vlan filter issue with
i40evf driver. Please refer to the test plan
https://doc.dpdk.org/dts/test_plans/kernelpf_iavf_test_plan.html.

So could you please detail the issue?
E.g. do you use kernel PF + DPDK VF or DPDK PF + DPDK VF? 
What's the driver version? And what's the step to reproduce with testpmd?

BTW, for the commit log, needn't to describe the details you communicated
with PAE, just describe what's the issue, the root cause, and how to fix it.
Besides, fix patch needs fix line. Please refer to other fix patches.

Beilei

> 
> Signed-off-by: Qiming Chen <chenqiming_huawei@163.com>
> ---
>  drivers/net/i40e/i40e_ethdev_vf.c | 23 +++++++++++++++++++++--
>  1 file changed, 21 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/net/i40e/i40e_ethdev_vf.c
> b/drivers/net/i40e/i40e_ethdev_vf.c
> index 12e69a3233..a099daae6b 100644
> --- a/drivers/net/i40e/i40e_ethdev_vf.c
> +++ b/drivers/net/i40e/i40e_ethdev_vf.c
> @@ -1900,11 +1900,30 @@ static int
>  i40evf_vlan_filter_set(struct rte_eth_dev *dev, uint16_t vlan_id, int on)  {
>  	int ret;
> +	struct rte_eth_conf *dev_conf = &dev->data->dev_conf;
> +	struct i40e_vf *vf = I40EVF_DEV_PRIVATE_TO_VF(dev->data-
> >dev_private);
> +	bool promisc_unicast_enabled = vf->promisc_unicast_enabled;
> +	bool promisc_multicast_enabled = vf->promisc_multicast_enabled;
> 
> -	if (on)
> +	if (promisc_unicast_enabled)
> +		i40evf_dev_promiscuous_disable(dev);
> +
> +	if (promisc_multicast_enabled)
> +		i40evf_dev_allmulticast_disable(dev);
> +
> +	if (on) {
>  		ret = i40evf_add_vlan(dev, vlan_id);
> -	else
> +		if ((dev_conf->rxmode.offloads &
> DEV_RX_OFFLOAD_VLAN_STRIP) == 0)
> +			i40evf_disable_vlan_strip(dev);
> +	} else {
>  		ret = i40evf_del_vlan(dev,vlan_id);
> +	}
> +
> +	if (promisc_unicast_enabled)
> +		i40evf_dev_promiscuous_enable(dev);
> +
> +	if (promisc_multicast_enabled)
> +		i40evf_dev_allmulticast_enable(dev);
> 
>  	return ret;
>  }
> --
> 2.30.1.windows.1
  

Thanks your reply.


I use kenerl PF + DPDK VF, version is follow:
DPDK: 19.11
Kernel PF i40e.ko: 2.7.12
Firmware: 6.01 0x800034a3 1.1747.0


I did not use testpmd to test vlan filter, but wrote Demo for testing based on the following deployment:
1) x710 nic, use 2 PFs, each PF virtualizes 1 VF
2) 2 pf connected with fiber optic cable
3) 2 vf are hard to pass through to the VM
4) In vm, dpdk takes over the vf port,
5) One port is used as the sending port, and the other port is used as the receiving port, e.g. xmit portid is 0, rx portid is 1


Use the default configuration for port 0 as the sender, and configure port 1 as the receiving port as follows:
1) rte_eth_promiscuous_enable(1)
2) rte_eth_dev_set_vlan_offload(1, ETH_VLAN_FILTER_OFFLOAD)
3) rte_eth_dev_vlan_filter(1, 100, 1)


Do the following 2 scene tests:
1) Demo constructs a message with vlan 100 to be sent from port 0, and the test is OK
2) Demo constructs a message with VLAN 200 to send from port 0, test NOK, the message can also be received, VLAN filtering does not take effect


| |
Qiming Chen
|
|
chenqiming_huawei@163.com
|
签名由网易邮箱大师定制
On 8/25/2021 15:13，Xing, Beilei<beilei.xing@intel.com> wrote：


-----Original Message-----
From: dev <dev-bounces@dpdk.org> On Behalf Of Qiming Chen
Sent: Tuesday, August 24, 2021 5:30 PM
To: dev@dpdk.org
Cc: Xing, Beilei <beilei.xing@intel.com>; Qiming Chen
<chenqiming_huawei@163.com>
Subject: [dpdk-dev] [PATCH v2] net/i40e: solve the failure of vf vlan filtering

When vf driver port promiscuous is turned on, the vlan filtering function is
invalid.
Through communication with PAE expert, this is a limitation of the i40e chip.
Before adding or removing VLANs, you must first disable unicast
promiscuous or multicast promiscuous, then operate the vlan, and finally
restore unicast promiscuous or multicast promiscuous state.

Thanks for the patch.
But I heard from DPDK validation team that there's no vf vlan filter issue with
i40evf driver. Please refer to the test plan
https://doc.dpdk.org/dts/test_plans/kernelpf_iavf_test_plan.html.

So could you please detail the issue?
E.g. do you use kernel PF + DPDK VF or DPDK PF + DPDK VF?
What's the driver version? And what's the step to reproduce with testpmd?

BTW, for the commit log, needn't to describe the details you communicated
with PAE, just describe what's the issue, the root cause, and how to fix it.
Besides, fix patch needs fix line. Please refer to other fix patches.

Beilei


Signed-off-by: Qiming Chen <chenqiming_huawei@163.com>
---
drivers/net/i40e/i40e_ethdev_vf.c | 23 +++++++++++++++++++++--
1 file changed, 21 insertions(+), 2 deletions(-)

diff --git a/drivers/net/i40e/i40e_ethdev_vf.c
b/drivers/net/i40e/i40e_ethdev_vf.c
index 12e69a3233..a099daae6b 100644
--- a/drivers/net/i40e/i40e_ethdev_vf.c
+++ b/drivers/net/i40e/i40e_ethdev_vf.c
@@ -1900,11 +1900,30 @@ static int
i40evf_vlan_filter_set(struct rte_eth_dev *dev, uint16_t vlan_id, int on)  {
int ret;
+  struct rte_eth_conf *dev_conf = &dev->data->dev_conf;
+  struct i40e_vf *vf = I40EVF_DEV_PRIVATE_TO_VF(dev->data-
dev_private);
+  bool promisc_unicast_enabled = vf->promisc_unicast_enabled;
+  bool promisc_multicast_enabled = vf->promisc_multicast_enabled;

-  if (on)
+  if (promisc_unicast_enabled)
+    i40evf_dev_promiscuous_disable(dev);
+
+  if (promisc_multicast_enabled)
+    i40evf_dev_allmulticast_disable(dev);
+
+  if (on) {
ret = i40evf_add_vlan(dev, vlan_id);
-  else
+    if ((dev_conf->rxmode.offloads &
DEV_RX_OFFLOAD_VLAN_STRIP) == 0)
+      i40evf_disable_vlan_strip(dev);
+  } else {
ret = i40evf_del_vlan(dev,vlan_id);
+  }
+
+  if (promisc_unicast_enabled)
+    i40evf_dev_promiscuous_enable(dev);
+
+  if (promisc_multicast_enabled)
+    i40evf_dev_allmulticast_enable(dev);

return ret;
}
--
2.30.1.windows.1