diff mbox series

[V1] tests/distributor: modify script add novector path expect

Message ID 20220705081647.35533-1-linglix.chen@intel.com (mailing list archive)
State Accepted
Headers
Series [V1] tests/distributor: modify script add novector path expect |

Checks

Context Check Description
ci/Intel-dts-format-test success Testing OK
ci/Intel-dts-pylama-test success Testing OK
ci/Intel-dts-doc-test success Testing OK
ci/Intel-dts-suite-test success Testing OK

Commit Message

Lingli Chen July 5, 2022, 8:16 a.m. UTC 
  The “--force-max-simd-bitwidth=64” is relatively new to DPDK (novector code path) and severely limits the performance of the burst mode of the distributor.
So modify script to add novector path expect

Signed-off-by: Lingli Chen <linglix.chen@intel.com>
---
 test_plans/distributor_test_plan.rst |  4 ++++
 tests/TestSuite_distributor.py       | 12 +++++++++---
 2 files changed, 13 insertions(+), 3 deletions(-)

Comments

Tu, Lijuan July 6, 2022, 6:11 a.m. UTC | #1 
On Tue,  5 Jul 2022 04:16:47 -0400, Lingli Chen <linglix.chen@intel.com> wrote:
> The “--force-max-simd-bitwidth=64” is relatively new to DPDK (novector code path) and severely limits the performance of the burst mode of the distributor.
> So modify script to add novector path expect
> 
> Signed-off-by: Lingli Chen <linglix.chen@intel.com>


Applied, thanks
diff mbox series

Patch

diff --git a/test_plans/distributor_test_plan.rst b/test_plans/distributor_test_plan.rst
index 7e61bef4..12fd9e9f 100644
--- a/test_plans/distributor_test_plan.rst
+++ b/test_plans/distributor_test_plan.rst
@@ -40,6 +40,10 @@  Compared CPU cycles for normal distributor and burst API
 
 Verify burst distributor API cost much less cycles then normal version
 
+The novector path severely limits the performance of the burst mode of the distributor.
+so novector: expect single mode > burst mode
+vector: expect single mode > burst mode * 2
+
 Test case: Distribute with maximum workers
 ==========================================
 Start distributor sample with 63(0xeffffffffffffffff0) workers
diff --git a/tests/TestSuite_distributor.py b/tests/TestSuite_distributor.py
index 7b41eb01..6878edbd 100644
--- a/tests/TestSuite_distributor.py
+++ b/tests/TestSuite_distributor.py
@@ -65,9 +65,15 @@  class TestDistributor(TestCase):
         )
         self.dut.send_expect("quit", "# ")
         self.verify("Test OK" in out, "Test failed")
-        self.verify(
-            cycles_single > cycles_burst * 2, "Burst performance should be much better"
-        )
+        if "force-max-simd-bitwidth=64" in eal_para:
+            self.verify(
+                cycles_single > cycles_burst, "Burst performance should be much better"
+            )
+        else:
+            self.verify(
+                cycles_single > cycles_burst * 2,
+                "Burst performance should be much better",
+            )
 
     def test_perf_distributor(self):
         """