[dpdk-dev] [PATCH v2] net/i40e: solve the failure of vf vlan filtering

[Qiming Chen]

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 at 163.com
|
签名由网易邮箱大师定制
On 8/25/2021 15:13，Xing, Beilei<beilei.xing at intel.com> wrote：


-----Original Message-----
From: dev <dev-bounces at dpdk.org> On Behalf Of Qiming Chen
Sent: Tuesday, August 24, 2021 5:30 PM
To: dev at dpdk.org
Cc: Xing, Beilei <beilei.xing at intel.com>; Qiming Chen
<chenqiming_huawei at 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 at 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