test suite reviews and discussions
 help / color / mirror / Atom feed
* [dts][PATCH V1] vf_vlan/vf_vlan_rx:modify code to adapt avx2 and avx512 on vlan offload
@ 2023-01-13 13:05 Zhimin Huang
  2023-01-13  7:58 ` lijuan.tu
  0 siblings, 1 reply; 2+ messages in thread
From: Zhimin Huang @ 2023-01-13 13:05 UTC (permalink / raw)
  To: dts; +Cc: Zhimin Huang

base on dts commit 92e12676d29341695d86ef8ee8e36d4014d56e98,missing
vf_vlan_rx test case modification.

add parameter "--enable-hw-vlan" in testpmd.
the dpdk commit 5cbfb386aa3f4c49b3cd9579e4e928cc5ab08d35 fixed avx2 not support this parameter.
the avx2 behavior is not appropriate,it has gap between avx2 and avx512,
this feature will be implemetned in the future.
so we add parameter "enable-hw-vlan" in testpmd,there is no gap in avx2 and avx512 to test vlan strip.

Signed-off-by: Zhimin Huang <zhiminx.huang@intel.com>
---
 test_plans/vf_vlan_test_plan.rst | 17 +++++++++--------
 tests/TestSuite_vf_vlan.py       |  3 ++-
 2 files changed, 11 insertions(+), 9 deletions(-)

diff --git a/test_plans/vf_vlan_test_plan.rst b/test_plans/vf_vlan_test_plan.rst
index 8612bd84..0cf946d0 100644
--- a/test_plans/vf_vlan_test_plan.rst
+++ b/test_plans/vf_vlan_test_plan.rst
@@ -68,7 +68,13 @@ Prerequisites
      testpmd> set verbose 1
      testpmd> start
 
- if test DCF, set VF port to dcf and start up::
+.. note::
+   according to dpdk commit 5cbfb386aa3f4c49b3cd9579e4e928cc5ab08d35,if not add parameter "enable-hw-vlan", the vlan
+   offload should be disable.the avx2 behavior is not appropriate, the avx2 and avx512 shouldn't have gap on vlan
+   offload.this feature will be implemetned in the future.so add parameter "--enable-hw-vlan" in testpmd to test vlan
+   strip.
+
+if test DCF, set VF port to dcf and start up::
 
    Enable kernel trust mode:
 
@@ -155,7 +161,7 @@ Test case4: VF tagged vlan rx
 
 1. Make sure port based vlan disabled on VF0 and VF1
 
-2. Start testpmd with rxonly mode::
+2. Start testpmd with rxonly mode and parameter "--enable-hw-vlan"::
 
      testpmd> set fwd rxonly
      testpmd> set verbose 1
@@ -186,12 +192,7 @@ Test case4: VF tagged vlan rx
 Test case5: VF Vlan strip test
 ==============================
 
-.. note::
-   according to dpdk commit 5cbfb386aa3f4c49b3cd9579e4e928cc5ab08d35,if not add parameter "enable-hw-vlan", the vlan
-   offload should be disable.the avx2 behavior is not appropriate, the avx2 and avx512 shouldn't have gap on vlan
-   offload. so add parameter "--enable-hw-vlan" in testpmd to test vlan strip.
-
-1. Start testpmd with mac forward mode::
+1. Start testpmd with mac forward mode and parameter "--enable-hw-vlan"::
 
      testpmd> set fwd mac
      testpmd> set verbose 1
diff --git a/tests/TestSuite_vf_vlan.py b/tests/TestSuite_vf_vlan.py
index 14c675c7..563d06b6 100644
--- a/tests/TestSuite_vf_vlan.py
+++ b/tests/TestSuite_vf_vlan.py
@@ -344,7 +344,8 @@ class TestVfVlan(TestCase):
         self.vm0_dut_ports = self.vm_dut_0.get_ports("any")
 
         self.vm0_testpmd = PmdOutput(self.vm_dut_0)
-        self.launch_testpmd(dcf_flag=self.dcf_mode)
+        param = "--enable-hw-vlan" if not self.dcf_mode else ""
+        self.launch_testpmd(dcf_flag=self.dcf_mode, param=param)
         self.vm0_testpmd.execute_cmd("set fwd rxonly")
         self.vm0_testpmd.execute_cmd("set verbose 1")
         self.vm0_testpmd.execute_cmd("vlan set strip on 0")
-- 
2.25.1


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2023-01-13  7:58 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-01-13 13:05 [dts][PATCH V1] vf_vlan/vf_vlan_rx:modify code to adapt avx2 and avx512 on vlan offload Zhimin Huang
2023-01-13  7:58 ` lijuan.tu

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).