[V1,2/2] tests/rxtx_offload: optimize test steps to adapt test plan

Message ID 20230202025927.22506-2-weiyuanx.li@intel.com (mailing list archive)
State Accepted
Headers
Series [V1,1/2] test_plans/rxtx_offload: Modify the test plan clerical error |

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

Weiyuan Li Feb. 2, 2023, 2:59 a.m. UTC
  It takes too much time to check 100M TX-packets, according to the test
plan only 100K Tx-packets needs to be checked to optimize the test
steps.
 
Signed-off-by: Weiyuan Li <weiyuanx.li@intel.com>
---
 tests/TestSuite_rxtx_offload.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
  

Comments

Huang, ZhiminX Feb. 3, 2023, 3:11 a.m. UTC | #1
> -----Original Message-----
> From: Weiyuan Li <weiyuanx.li@intel.com>
> Sent: Thursday, February 2, 2023 10:59 AM
> To: dts@dpdk.org
> Cc: Li, WeiyuanX <weiyuanx.li@intel.com>
> Subject: [dts][PATCH V1 2/2] tests/rxtx_offload: optimize test steps to adapt
> test plan
> 
> It takes too much time to check 100M TX-packets, according to the test plan
> only 100K Tx-packets needs to be checked to optimize the test steps.
> 
> Signed-off-by: Weiyuan Li <weiyuanx.li@intel.com>
> ---
Acked-by: Zhimin Huang <zhiminx.huang@intel.com >
  
Tu, Lijuan March 7, 2023, 3:52 a.m. UTC | #2
On Thu,  2 Feb 2023 10:59:27 +0800, Weiyuan Li <weiyuanx.li@intel.com> wrote:
> It takes too much time to check 100M TX-packets, according to the test
> plan only 100K Tx-packets needs to be checked to optimize the test
> steps.
>  
> Signed-off-by: Weiyuan Li <weiyuanx.li@intel.com>


Series applied, thanks
  

Patch

diff --git a/tests/TestSuite_rxtx_offload.py b/tests/TestSuite_rxtx_offload.py
index c60bb68b..4c084abf 100644
--- a/tests/TestSuite_rxtx_offload.py
+++ b/tests/TestSuite_rxtx_offload.py
@@ -235,7 +235,7 @@  class TestRxTx_Offload(TestCase):
         while i < count:
             if rxtx == "tx":
                 pks_l1 = re.findall(r"TX-packets: (\w+)", out1)
-                time.sleep(15)
+                time.sleep(3)
                 out1 = self.dut.send_expect("show port stats all", "testpmd> ")
                 pks_l2 = re.findall(r"TX-packets: (\w+)", out1)
                 self.logger.info(
@@ -252,7 +252,7 @@  class TestRxTx_Offload(TestCase):
                     self.verify(
                         int(pks_l2[index]) > int(pks_l1[index]), "TX-packets hang"
                     )
-                if int(pks_l1[index]) < 100000000 or int(pks_l2[index]) < 100000000:
+                if int(pks_l1[index]) < 100000 or int(pks_l2[index]) < 100000:
                     count += 1
             i += 1
             if count > 25: