[dts] [PATCH V1 1/4] tests/kernelpf_iavf: modify code to adapt avx512

[dts] [PATCH V1 1/4] tests/kernelpf_iavf: modify code to adapt avx512

[Xiang An]

Based on dpdk commit:4b64ccb328 modify dts code to adapt avx512.

Signed-off-by: Xiang An <xiangx.an@intel.com>
---
 tests/TestSuite_kernelpf_iavf.py | 12 +++++++++---
 1 file changed, 9 insertions(+), 3 deletions(-)

diff --git a/tests/TestSuite_kernelpf_iavf.py b/tests/TestSuite_kernelpf_iavf.py
index dde59e1..c5836dd 100644
--- a/tests/TestSuite_kernelpf_iavf.py
+++ b/tests/TestSuite_kernelpf_iavf.py
@@ -44,7 +44,7 @@ import time
 import framework.utils as utils
 from framework.packet import Packet
 from framework.pmd_output import PmdOutput
-from framework.settings import HEADER_SIZE
+from framework.settings import HEADER_SIZE, load_global_setting, DPDK_RXMODE_SETTING
 from framework.test_case import TestCase
 from framework.utils import RED
 from framework.virt_common import VM
@@ -774,7 +774,10 @@ class TestKernelpfIavf(TestCase):
         self.verify(bad_l4csum == 2, "Bad-ipcsum check error")
 
     def test_vf_hw_checksum_offload(self):
-        self.vm_testpmd.start_testpmd("all")
+        if load_global_setting(DPDK_RXMODE_SETTING) == 'default':
+            self.vm_testpmd.start_testpmd("all")
+        elif load_global_setting(DPDK_RXMODE_SETTING) == 'avx512':
+            self.vm_testpmd.start_testpmd("all", "--enable-rx-cksum")
         self.enable_hw_checksum()
         self.vm_testpmd.execute_cmd("port start all")
         self.vm_testpmd.execute_cmd("set verbose 1")
@@ -782,7 +785,10 @@ class TestKernelpfIavf(TestCase):
         self.checksum_verify()
 
     def test_vf_sw_checksum_offload(self):
-        self.vm_testpmd.start_testpmd("all")
+        if load_global_setting(DPDK_RXMODE_SETTING) == 'default':
+            self.vm_testpmd.start_testpmd("all")
+        elif load_global_setting(DPDK_RXMODE_SETTING) == 'avx512':
+            self.vm_testpmd.start_testpmd("all", "--enable-rx-cksum")
         self.enable_sw_checksum()
         self.vm_testpmd.execute_cmd("port start all")
         self.vm_testpmd.execute_cmd("start")
-- 
1.8.3.1

[dts] [PATCH V1 2/4] tests/vf_vlan: modify code to adapt avx512

[Xiang An]

Based on dpdk commit:4b64ccb328 modify dts code to adapt avx512.

Signed-off-by: Xiang An <xiangx.an@intel.com>
---
 tests/TestSuite_vf_vlan.py | 12 +++++++++---
 1 file changed, 9 insertions(+), 3 deletions(-)

diff --git a/tests/TestSuite_vf_vlan.py b/tests/TestSuite_vf_vlan.py
index 127eca3..3bbfa38 100644
--- a/tests/TestSuite_vf_vlan.py
+++ b/tests/TestSuite_vf_vlan.py
@@ -35,7 +35,7 @@ import time
 
 from framework.packet import Packet
 from framework.pmd_output import PmdOutput
-from framework.settings import get_nic_name
+from framework.settings import get_nic_name, load_global_setting, DPDK_RXMODE_SETTING
 from framework.test_case import TestCase
 from framework.virt_common import VM
 
@@ -346,7 +346,10 @@ class TestVfVlan(TestCase):
         self.vm0_dut_ports = self.vm_dut_0.get_ports('any')
 
         self.vm0_testpmd = PmdOutput(self.vm_dut_0)
-        self.vm0_testpmd.start_testpmd(VM_CORES_MASK)
+        if load_global_setting(DPDK_RXMODE_SETTING) == 'default':
+            self.vm0_testpmd.start_testpmd(VM_CORES_MASK)
+        elif load_global_setting(DPDK_RXMODE_SETTING) == 'avx512':
+            self.vm0_testpmd.start_testpmd(VM_CORES_MASK, '--enable-rx-cksum')
         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')
@@ -417,7 +420,10 @@ class TestVfVlan(TestCase):
         if self.kdriver == "i40e":
             self.vm0_testpmd.start_testpmd(VM_CORES_MASK, '')
         else:
-            self.vm0_testpmd.start_testpmd(VM_CORES_MASK)
+            if load_global_setting(DPDK_RXMODE_SETTING) == 'default':
+                self.vm0_testpmd.start_testpmd(VM_CORES_MASK)
+            elif load_global_setting(DPDK_RXMODE_SETTING) == 'avx512':
+                self.vm0_testpmd.start_testpmd(VM_CORES_MASK, '--enable-rx-cksum')
         self.vm0_testpmd.execute_cmd('set fwd rxonly')
         self.vm0_testpmd.execute_cmd('set verbose 1')
         self.vm0_testpmd.execute_cmd('start')
-- 
1.8.3.1

[dts] [PATCH V1 3/4] tests/l3fwdacl: modify code to adapt avx512

[Xiang An]

Modify code to adapt rxmod avx512.

Signed-off-by: Xiang An <xiangx.an@intel.com>
---
 tests/TestSuite_l3fwdacl.py | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/tests/TestSuite_l3fwdacl.py b/tests/TestSuite_l3fwdacl.py
index 65c46b0..dc5d41f 100644
--- a/tests/TestSuite_l3fwdacl.py
+++ b/tests/TestSuite_l3fwdacl.py
@@ -40,6 +40,7 @@ import time
 import framework.packet as packet
 import framework.utils as utils
 from framework.test_case import TestCase
+from framework.settings import load_global_setting, DPDK_RXMODE_SETTING
 
 
 class TestL3fwdacl(TestCase):
@@ -643,7 +644,8 @@ class TestL3fwdacl(TestCase):
         print("Core mask: %s" % self.core_mask)
 
         if self.dut.dpdk_version >= '20.11.0':
-            self.eal_para += " --force-max-simd-bitwidth=0"
+            if load_global_setting(DPDK_RXMODE_SETTING) == 'default':
+                self.eal_para += " --force-max-simd-bitwidth=0"
 
         valid_ports = [port for port in ports if self.tester.get_local_port(port) != -1]
         self.verify(
-- 
1.8.3.1

[dts] [PATCH V1 4/4] tests/unit_tests_eal: modify code to adapt avx512

[Xiang An]

Modify code to adapt rxmod avx512.

Signed-off-by: Xiang An <xiangx.an@intel.com>
---
 tests/TestSuite_unit_tests_eal.py | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/tests/TestSuite_unit_tests_eal.py b/tests/TestSuite_unit_tests_eal.py
index 8096183..b5cb7a2 100644
--- a/tests/TestSuite_unit_tests_eal.py
+++ b/tests/TestSuite_unit_tests_eal.py
@@ -38,7 +38,7 @@ EAL autotest.
 
 import framework.utils as utils
 from framework.test_case import TestCase
-
+from framework.settings import load_global_setting, DPDK_RXMODE_SETTING
 #
 #
 # Test class.
@@ -462,7 +462,8 @@ class TestUnitTestsEal(TestCase):
         self.test_app_cmdline += "--no-pci"
 
         if self.dut.dpdk_version >= '20.11.0':
-            self.test_app_cmdline += " --force-max-simd-bitwidth=0"
+            if load_global_setting(DPDK_RXMODE_SETTING) == 'default':
+                self.test_app_cmdline += " --force-max-simd-bitwidth=0"
         self.dut.send_expect(self.test_app_cmdline, "R.*T.*E.*>.*>", self.start_test_time)
         out = self.dut.send_expect("acl_autotest", "RTE>>", self.run_cmd_time)
         self.dut.send_expect("quit", "# ")
-- 
1.8.3.1

Re: [dts] [PATCH V1 4/4] tests/unit_tests_eal: modify code to adapt avx512

[Ananyev, Konstantin]

Hi Xiang,

> Modify code to adapt rxmod avx512.
> 
> Signed-off-by: Xiang An <xiangx.an@intel.com>
> ---
>  tests/TestSuite_unit_tests_eal.py | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/tests/TestSuite_unit_tests_eal.py b/tests/TestSuite_unit_tests_eal.py
> index 8096183..b5cb7a2 100644
> --- a/tests/TestSuite_unit_tests_eal.py
> +++ b/tests/TestSuite_unit_tests_eal.py
> @@ -38,7 +38,7 @@ EAL autotest.
> 
>  import framework.utils as utils
>  from framework.test_case import TestCase
> -
> +from framework.settings import load_global_setting, DPDK_RXMODE_SETTING
>  #
>  #
>  # Test class.
> @@ -462,7 +462,8 @@ class TestUnitTestsEal(TestCase):
>          self.test_app_cmdline += "--no-pci"
> 
>          if self.dut.dpdk_version >= '20.11.0':
> -            self.test_app_cmdline += " --force-max-simd-bitwidth=0"
> +            if load_global_setting(DPDK_RXMODE_SETTING) == 'default':
> +                self.test_app_cmdline += " --force-max-simd-bitwidth=0"

Could you probably explain why it is needed here?

>          self.dut.send_expect(self.test_app_cmdline, "R.*T.*E.*>.*>", self.start_test_time)
>          out = self.dut.send_expect("acl_autotest", "RTE>>", self.run_cmd_time)
>          self.dut.send_expect("quit", "# ")
> --
> 1.8.3.1

Re: [dts] [PATCH V1 4/4] tests/unit_tests_eal: modify code to adapt avx512

[Huang, ZhiminX]

> -----Original Message-----
> From: dts <dts-bounces@dpdk.org> On Behalf Of Ananyev, Konstantin
> Sent: Tuesday, November 2, 2021 6:43 PM
> To: An, XiangX <xiangx.an@intel.com>; dts@dpdk.org
> Cc: An, XiangX <xiangx.an@intel.com>
> Subject: Re: [dts] [PATCH V1 4/4] tests/unit_tests_eal: modify code to adapt
> avx512
> 
> 
> Hi Xiang,
> 
> > Modify code to adapt rxmod avx512.
> >
> > Signed-off-by: Xiang An <xiangx.an@intel.com>
> > ---
> >  tests/TestSuite_unit_tests_eal.py | 5 +++--
> >  1 file changed, 3 insertions(+), 2 deletions(-)
> >
> > diff --git a/tests/TestSuite_unit_tests_eal.py
> b/tests/TestSuite_unit_tests_eal.py
> > index 8096183..b5cb7a2 100644
> > --- a/tests/TestSuite_unit_tests_eal.py
> > +++ b/tests/TestSuite_unit_tests_eal.py
> > @@ -38,7 +38,7 @@ EAL autotest.
> >
> >  import framework.utils as utils
> >  from framework.test_case import TestCase
> > -
> > +from framework.settings import load_global_setting,
> DPDK_RXMODE_SETTING
> >  #
> >  #
> >  # Test class.
> > @@ -462,7 +462,8 @@ class TestUnitTestsEal(TestCase):
> >          self.test_app_cmdline += "--no-pci"
> >
> >          if self.dut.dpdk_version >= '20.11.0':
> > -            self.test_app_cmdline += " --force-max-simd-bitwidth=0"
> > +            if load_global_setting(DPDK_RXMODE_SETTING) == 'default':
> > +                self.test_app_cmdline += "
> --force-max-simd-bitwidth=0"
> 
> Could you probably explain why it is needed here?
> 
> >          self.dut.send_expect(self.test_app_cmdline, "R.*T.*E.*>.*>",
> self.start_test_time)
> >          out = self.dut.send_expect("acl_autotest", "RTE>>",
> self.run_cmd_time)
> >          self.dut.send_expect("quit", "# ")
> > --
> > 1.8.3.1

Hi Konstantin,
We have add this case to test avx512 mode, if this judgment is not added, the cmdline will invalid.

Take a example:
x86_64-native-linuxapp-gcc/app/test/dpdk-test -l 1,2,3,4 -n 4 --file-prefix=dpdk_237965_20211103034106 --force-max-simd-bitwidth=512 --no-pci --force-max-simd-bitwidth=0

Regards,
HuangZhiMin

Re: [dts] [PATCH V1 4/4] tests/unit_tests_eal: modify code to adapt avx512

[Ananyev, Konstantin]

> > Hi Xiang,
> >
> > > Modify code to adapt rxmod avx512.
> > >
> > > Signed-off-by: Xiang An <xiangx.an@intel.com>
> > > ---
> > >  tests/TestSuite_unit_tests_eal.py | 5 +++--
> > >  1 file changed, 3 insertions(+), 2 deletions(-)
> > >
> > > diff --git a/tests/TestSuite_unit_tests_eal.py
> > b/tests/TestSuite_unit_tests_eal.py
> > > index 8096183..b5cb7a2 100644
> > > --- a/tests/TestSuite_unit_tests_eal.py
> > > +++ b/tests/TestSuite_unit_tests_eal.py
> > > @@ -38,7 +38,7 @@ EAL autotest.
> > >
> > >  import framework.utils as utils
> > >  from framework.test_case import TestCase
> > > -
> > > +from framework.settings import load_global_setting,
> > DPDK_RXMODE_SETTING
> > >  #
> > >  #
> > >  # Test class.
> > > @@ -462,7 +462,8 @@ class TestUnitTestsEal(TestCase):
> > >          self.test_app_cmdline += "--no-pci"
> > >
> > >          if self.dut.dpdk_version >= '20.11.0':
> > > -            self.test_app_cmdline += " --force-max-simd-bitwidth=0"
> > > +            if load_global_setting(DPDK_RXMODE_SETTING) == 'default':
> > > +                self.test_app_cmdline += "
> > --force-max-simd-bitwidth=0"
> >
> > Could you probably explain why it is needed here?
> >
> > >          self.dut.send_expect(self.test_app_cmdline, "R.*T.*E.*>.*>",
> > self.start_test_time)
> > >          out = self.dut.send_expect("acl_autotest", "RTE>>",
> > self.run_cmd_time)
> > >          self.dut.send_expect("quit", "# ")
> > > --
> > > 1.8.3.1
> 
> Hi Konstantin,
> We have add this case to test avx512 mode, if this judgment is not added, the cmdline will invalid.
> 
> Take a example:
> x86_64-native-linuxapp-gcc/app/test/dpdk-test -l 1,2,3,4 -n 4 --file-prefix=dpdk_237965_20211103034106 --force-max-simd-bitwidth=512
> --no-pci --force-max-simd-bitwidth=0
> 

So DPDK_RXMODE_SETTING controls whether '--force-max-simd-bitwidth' is set or not?
If so, can we then just override it here for this test-case?
I'd like this test-case to always run with '--force-max-simd-bitwidth=0'.
That way it provides best possible coverage.

Re: [dts] [PATCH V1 4/4] tests/unit_tests_eal: modify code to adapt avx512

[Huang, ZhiminX]

> -----Original Message-----
> From: Ananyev, Konstantin <konstantin.ananyev@intel.com>
> Sent: Tuesday, November 2, 2021 7:27 PM
> To: Huang, ZhiminX <zhiminx.huang@intel.com>; dts@dpdk.org
> Cc: An, XiangX <xiangx.an@intel.com>
> Subject: RE: [dts] [PATCH V1 4/4] tests/unit_tests_eal: modify code to adapt
> avx512
> 
> 
> 
> > > Hi Xiang,
> > >
> > > > Modify code to adapt rxmod avx512.
> > > >
> > > > Signed-off-by: Xiang An <xiangx.an@intel.com>
> > > > ---
> > > >  tests/TestSuite_unit_tests_eal.py | 5 +++--
> > > >  1 file changed, 3 insertions(+), 2 deletions(-)
> > > >
> > > > diff --git a/tests/TestSuite_unit_tests_eal.py
> > > b/tests/TestSuite_unit_tests_eal.py
> > > > index 8096183..b5cb7a2 100644
> > > > --- a/tests/TestSuite_unit_tests_eal.py
> > > > +++ b/tests/TestSuite_unit_tests_eal.py
> > > > @@ -38,7 +38,7 @@ EAL autotest.
> > > >
> > > >  import framework.utils as utils
> > > >  from framework.test_case import TestCase
> > > > -
> > > > +from framework.settings import load_global_setting,
> > > DPDK_RXMODE_SETTING
> > > >  #
> > > >  #
> > > >  # Test class.
> > > > @@ -462,7 +462,8 @@ class TestUnitTestsEal(TestCase):
> > > >          self.test_app_cmdline += "--no-pci"
> > > >
> > > >          if self.dut.dpdk_version >= '20.11.0':
> > > > -            self.test_app_cmdline += " --force-max-simd-bitwidth=0"
> > > > +            if load_global_setting(DPDK_RXMODE_SETTING) ==
> 'default':
> > > > +                self.test_app_cmdline += "
> > > --force-max-simd-bitwidth=0"
> > >
> > > Could you probably explain why it is needed here?
> > >
> > > >          self.dut.send_expect(self.test_app_cmdline,
> > > > "R.*T.*E.*>.*>",
> > > self.start_test_time)
> > > >          out = self.dut.send_expect("acl_autotest", "RTE>>",
> > > self.run_cmd_time)
> > > >          self.dut.send_expect("quit", "# ")
> > > > --
> > > > 1.8.3.1
> >
> > Hi Konstantin,
> > We have add this case to test avx512 mode, if this judgment is not added, the
> cmdline will invalid.
> >
> > Take a example:
> > x86_64-native-linuxapp-gcc/app/test/dpdk-test -l 1,2,3,4 -n 4
> > --file-prefix=dpdk_237965_20211103034106 --force-max-simd-bitwidth=512
> > --no-pci --force-max-simd-bitwidth=0
> >
> 
> So DPDK_RXMODE_SETTING controls whether '--force-max-simd-bitwidth' is
> set or not?
> If so, can we then just override it here for this test-case?
> I'd like this test-case to always run with '--force-max-simd-bitwidth=0'.
> That way it provides best possible coverage.
> 
> 
So according your decription, in any avx mode should be excuted '--force-max-simd-bitwidth=0'?
If so, I will re-submit dst patch to cover about acl test.

Re: [dts] [PATCH V1 4/4] tests/unit_tests_eal: modify code to adapt avx512

[Ananyev, Konstantin]

> >
> > > > Hi Xiang,
> > > >
> > > > > Modify code to adapt rxmod avx512.
> > > > >
> > > > > Signed-off-by: Xiang An <xiangx.an@intel.com>
> > > > > ---
> > > > >  tests/TestSuite_unit_tests_eal.py | 5 +++--
> > > > >  1 file changed, 3 insertions(+), 2 deletions(-)
> > > > >
> > > > > diff --git a/tests/TestSuite_unit_tests_eal.py
> > > > b/tests/TestSuite_unit_tests_eal.py
> > > > > index 8096183..b5cb7a2 100644
> > > > > --- a/tests/TestSuite_unit_tests_eal.py
> > > > > +++ b/tests/TestSuite_unit_tests_eal.py
> > > > > @@ -38,7 +38,7 @@ EAL autotest.
> > > > >
> > > > >  import framework.utils as utils
> > > > >  from framework.test_case import TestCase
> > > > > -
> > > > > +from framework.settings import load_global_setting,
> > > > DPDK_RXMODE_SETTING
> > > > >  #
> > > > >  #
> > > > >  # Test class.
> > > > > @@ -462,7 +462,8 @@ class TestUnitTestsEal(TestCase):
> > > > >          self.test_app_cmdline += "--no-pci"
> > > > >
> > > > >          if self.dut.dpdk_version >= '20.11.0':
> > > > > -            self.test_app_cmdline += " --force-max-simd-bitwidth=0"
> > > > > +            if load_global_setting(DPDK_RXMODE_SETTING) ==
> > 'default':
> > > > > +                self.test_app_cmdline += "
> > > > --force-max-simd-bitwidth=0"
> > > >
> > > > Could you probably explain why it is needed here?
> > > >
> > > > >          self.dut.send_expect(self.test_app_cmdline,
> > > > > "R.*T.*E.*>.*>",
> > > > self.start_test_time)
> > > > >          out = self.dut.send_expect("acl_autotest", "RTE>>",
> > > > self.run_cmd_time)
> > > > >          self.dut.send_expect("quit", "# ")
> > > > > --
> > > > > 1.8.3.1
> > >
> > > Hi Konstantin,
> > > We have add this case to test avx512 mode, if this judgment is not added, the
> > cmdline will invalid.
> > >
> > > Take a example:
> > > x86_64-native-linuxapp-gcc/app/test/dpdk-test -l 1,2,3,4 -n 4
> > > --file-prefix=dpdk_237965_20211103034106 --force-max-simd-bitwidth=512
> > > --no-pci --force-max-simd-bitwidth=0
> > >
> >
> > So DPDK_RXMODE_SETTING controls whether '--force-max-simd-bitwidth' is
> > set or not?
> > If so, can we then just override it here for this test-case?
> > I'd like this test-case to always run with '--force-max-simd-bitwidth=0'.
> > That way it provides best possible coverage.
> >
> >
> So according your decription, in any avx mode should be excuted '--force-max-simd-bitwidth=0'?

Yes, basically '--force-max-simd-bitwidth=0' says use max available SIMD on that box.
So on machine with avx512 it will test: avx512, avx2, sse, scalar.
On machine with avx2 only it will test: avx2, sse, scalar.
...

> If so, I will re-submit dst patch to cover about acl test.

Good, thanks.