test suite reviews and discussions
 help / color / mirror / Atom feed
* [dts] [PATCH V2 0/2] rxtx_offload: add test plan and tests for tx offload multi_segs setting
@ 2021-08-25  3:11 Yu Jiang
  2021-08-25  3:11 ` [dts] [PATCH V2 1/2] tests/rxtx_offload: add tx offload multi_segs setting case Yu Jiang
                   ` (2 more replies)
  0 siblings, 3 replies; 5+ messages in thread
From: Yu Jiang @ 2021-08-25  3:11 UTC (permalink / raw)
  To: dts; +Cc: Yu Jiang

vector path doesn't support multi-segment packet transmit,
only normal path supports multi-segment packet transmit.
Use 'port config 0 tx_offload multi_segs on' or
"--tx-offloads=0x00008000" parameter to enable multi segment offload.

Yu Jiang (2):
  tests/rxtx_offload: add tx offload multi_segs setting case
  test_plans/rxtx_offload: add tx offload multi_segs setting plan

 test_plans/rxtx_offload_test_plan.rst | 99 +++++++++++++++++++++++++++++++++++
 tests/TestSuite_rxtx_offload.py       | 66 +++++++++++++++++++++++
 2 files changed, 165 insertions(+)

-- 
2.7.4


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

* [dts] [PATCH V2 1/2] tests/rxtx_offload: add tx offload multi_segs setting case
  2021-08-25  3:11 [dts] [PATCH V2 0/2] rxtx_offload: add test plan and tests for tx offload multi_segs setting Yu Jiang
@ 2021-08-25  3:11 ` Yu Jiang
  2021-08-25  3:17   ` Jiang, YuX
  2021-08-25  3:11 ` [dts] [PATCH V2 2/2] test_plans/rxtx_offload: add tx offload multi_segs setting plan Yu Jiang
  2021-08-27  8:25 ` [dts] [PATCH V2 0/2] rxtx_offload: add test plan and tests for tx offload multi_segs setting Tu, Lijuan
  2 siblings, 1 reply; 5+ messages in thread
From: Yu Jiang @ 2021-08-25  3:11 UTC (permalink / raw)
  To: dts; +Cc: Yu Jiang

vector path doesn't support multi-segment packet transmit,
only normal path supports multi-segment packet transmit.
Use 'port config 0 tx_offload multi_segs on' or
"--tx-offloads=0x00008000" parameter to enable multi segment offload.

Signed-off-by: Yu Jiang <yux.jiang@intel.com>
---
 tests/TestSuite_rxtx_offload.py | 66 +++++++++++++++++++++++++++++++++++++++++
 1 file changed, 66 insertions(+)

diff --git a/tests/TestSuite_rxtx_offload.py b/tests/TestSuite_rxtx_offload.py
index 0e70526..a934026 100644
--- a/tests/TestSuite_rxtx_offload.py
+++ b/tests/TestSuite_rxtx_offload.py
@@ -209,6 +209,26 @@ class TestRxTx_Offload(TestCase):
                 acl_offload = offloads[offload[i]]
                 self.verify(acl_offload in queue_line[i], "Fail to configure offload by queue.")
             i = i + 1
+
+    def verify_packets_increasing(self, rxtx="tx", count=2):
+        # Verify RXTX-packets increasing on each ports, check count < 25
+        out1 = self.dut.send_expect("show port stats all", "testpmd> ")
+        i = 0
+        while i < count:
+            if rxtx == "tx":
+                pks_l1 = re.findall(r'TX-packets: (\w+)', out1)
+                time.sleep(15)
+                out1 = self.dut.send_expect("show port stats all", "testpmd> ")
+                pks_l2 = re.findall(r'TX-packets: (\w+)', out1)
+                self.logger.info("Times="+ str(i) + ", count=" + str(count) + ", pks2_cur=" + str(pks_l2) + ", pks1_before=" + str(pks_l1))
+                for index in range(len(pks_l2)):
+                    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:
+                    count += 1
+            i += 1
+            if count > 25:
+                self.verify(False, "Check count timeout")
+                break
  
     def get_queue_number(self, packet):
         """
@@ -716,6 +736,52 @@ class TestRxTx_Offload(TestCase):
         offload = ["mbuf_fast_free"]
         self.check_port_config("tx", offload)
 
+    def test_txoffload_port_multi_segs(self):
+        """
+        Tx offload multi_segs setting.
+        """
+        offload = ["multi_segs"]
+        # Start testpmd with "--tx-offloads=0x00008000" to enable multi_segs tx_offload
+        self.pmdout.start_testpmd("%s" % self.cores, "--tx-offloads=0x00008000")
+        for portid in range(len(self.dut_ports)):
+            self.check_port_config(rxtx="tx", offload=offload, port_id=portid)
+
+        # Set fwd to txonly, Set the length of each segment of the TX-ONLY packets, Set the split policy for TX packets, then start to send pkgs
+        self.dut.send_expect("set fwd txonly", "testpmd> ")
+        self.dut.send_expect("set txpkts 64,128,512,2000,64,128,512,2000", "testpmd> ")
+        self.dut.send_expect("set txsplit rand", "testpmd> ")
+        self.dut.send_expect("start", "testpmd> ")
+
+        # Check TX-packets will not hang and continue to increase
+        self.verify_packets_increasing(rxtx="tx")
+        self.dut.send_expect("stop", "testpmd> ")
+        self.dut.send_expect("quit", "# ")
+
+        # Start testpmd again without "--tx-offloads", check multi-segs is disabled by default
+        self.pmdout.start_testpmd("%s" % self.cores, " ")
+        for portid in range(len(self.dut_ports)):
+            outstring = self.dut.send_expect("show port %d tx_offload configuration" % portid, "testpmd> ")
+            self.verify("MULTI_SEGS" not in outstring, "multi-segs is not disabled by default")
+
+        self.dut.send_expect("port stop all", "testpmd> ")
+        for portid in range(len(self.dut_ports)):
+            cmd = "port config {} tx_offload multi_segs on".format(portid)
+            self.dut.send_expect(cmd, "testpmd> ")
+        self.dut.send_expect("port start all", "testpmd> ")
+        for portid in range(len(self.dut_ports)):
+              self.check_port_config(rxtx="tx", offload=offload, port_id=portid)
+
+        # Set fwd to txonly, Set the length of each segment of the TX-ONLY packets, Set the split policy for TX packets, then start to send pkgs
+        self.dut.send_expect("set fwd txonly", "testpmd> ")
+        self.dut.send_expect("set txpkts 64,128,256,512,64,128,256,512", "testpmd> ")
+        self.dut.send_expect("set txsplit rand", "testpmd> ")
+        self.dut.send_expect("start", "testpmd> ")
+
+        # Check TX-packets will not hang and continue to increase
+        self.verify_packets_increasing(rxtx="tx")
+        self.dut.send_expect("stop", "testpmd> ")
+        self.dut.send_expect("quit", "# ")
+
     def tear_down(self):
         """
         Run after each test case.
-- 
2.7.4


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

* [dts] [PATCH V2 2/2] test_plans/rxtx_offload: add tx offload multi_segs setting plan
  2021-08-25  3:11 [dts] [PATCH V2 0/2] rxtx_offload: add test plan and tests for tx offload multi_segs setting Yu Jiang
  2021-08-25  3:11 ` [dts] [PATCH V2 1/2] tests/rxtx_offload: add tx offload multi_segs setting case Yu Jiang
@ 2021-08-25  3:11 ` Yu Jiang
  2021-08-27  8:25 ` [dts] [PATCH V2 0/2] rxtx_offload: add test plan and tests for tx offload multi_segs setting Tu, Lijuan
  2 siblings, 0 replies; 5+ messages in thread
From: Yu Jiang @ 2021-08-25  3:11 UTC (permalink / raw)
  To: dts; +Cc: Yu Jiang

vector path doesn't support multi-segment packet transmit,
only normal path supports multi-segment packet transmit.
Use 'port config 0 tx_offload multi_segs on' or
"--tx-offloads=0x00008000" parameter to enable multi segment offload.

Signed-off-by: Yu Jiang <yux.jiang@intel.com>
---
 test_plans/rxtx_offload_test_plan.rst | 99 +++++++++++++++++++++++++++++++++++
 1 file changed, 99 insertions(+)

diff --git a/test_plans/rxtx_offload_test_plan.rst b/test_plans/rxtx_offload_test_plan.rst
index 246e1e1..d008d43 100644
--- a/test_plans/rxtx_offload_test_plan.rst
+++ b/test_plans/rxtx_offload_test_plan.rst
@@ -677,3 +677,102 @@ Test case: FVL Tx offload per-queue setting
 
    Note 1: there is no tx_offload per_queue parameter in ixgbe driver,
    so this case is just only for i40e.
+
+Test case: Tx offload multi_segs setting
+======================================================
+
+1. Start testpmd with "--tx-offloads=0x00008000" to enable tx_offload multi_segs ::
+
+    ./testpmd -c 0xf -n 4  -- -i --tx-offloads==0x00008000
+    testpmd> show port 0 tx_offload configuration
+    Tx Offloading Configuration of port 0 :
+      Port : MULTI_SEGS
+      Queue[ 0] : MULTI_SEGS
+
+2. Set fwd to txonly, Set the length of each segment of the TX-ONLY packets, Set the split policy for TX packets, then start to send pkgs::
+
+    testpmd> set fwd txonly
+    testpmd> set txpkts 64,128,512,2000,64,128,512,2000
+    testpmd> set txsplit rand
+    testpmd> start
+
+3. Check TX-packets will not hang and continue to increase::
+    Wait 30s or more, check TX-packets will continue to increase and can be more than 100K
+
+    testpmd> show port stats all
+        ######################## NIC statistics for port 0  ########################
+        RX-packets: 0         RX-missed: 0          RX-bytes:  0
+        RX-errors: 0
+        RX-nombuf:  0
+        TX-packets: 102628493  TX-errors: 0          TX-bytes:  139709164375
+
+        Throughput (since last show)
+        Rx-pps:            0          Rx-bps:            0
+        Tx-pps:       563539          Tx-bps:   9892394768
+        ############################################################################
+
+        ######################## NIC statistics for port 1  ########################
+        RX-packets: 0         RX-missed: 0          RX-bytes:  0
+        RX-errors: 0
+        RX-nombuf:  0
+        TX-packets: 102627429  TX-errors: 0          TX-bytes:  139709724215
+
+        Throughput (since last show)
+        Rx-pps:            0          Rx-bps:            0
+        Tx-pps:       563708          Tx-bps:   9892375000
+        ############################################################################
+
+    testpmd> stop
+    testpmd> quit
+
+4. Start testpmd again without "--tx-offloads", check multi-segs is disabled by default::
+
+    ./testpmd -c 0xf -n 4  -- -i
+    testpmd> show port 0 tx_offload configuration
+    No MULTI_SEGS in Tx Offloading Configuration of ports
+
+5. Enable tx_offload multi_segs ::
+    testpmd> port stop all
+    testpmd> port config 0 tx_offload multi_segs on
+    testpmd> port config 1 tx_offload multi_segs on
+    testpmd> port start all
+    testpmd> show port 0 tx_offload configuration
+    Tx Offloading Configuration of port 0 :
+      Port : MULTI_SEGS
+      Queue[ 0] : MULTI_SEGS
+
+6. Set fwd to txonly, Set the length of each segment of the TX-ONLY packets, Set the split policy for TX packets, then start to send pkgs::
+
+    testpmd> set fwd txonly
+    testpmd> set txpkts 64,128,256,512,64,128,256,512
+    testpmd> set txsplit rand
+    testpmd> start
+
+7. Check TX-packets will not hang and continue to increase::
+    Wait 30s or more, check TX-packets will continue to increase and can be more than 100K
+
+    testpmd> show port stats all
+        ######################## NIC statistics for port 0  ########################
+        RX-packets: 0         RX-missed: 0          RX-bytes:  0
+        RX-errors: 0
+        RX-nombuf:  0
+        TX-packets: 101266875  TX-errors: 0          TX-bytes:  136721429135
+
+        Throughput (since last show)
+        Rx-pps:            0          Rx-bps:            0
+        Tx-pps:       563293          Tx-bps:   9892438256
+        ############################################################################
+
+        ######################## NIC statistics for port 1  ########################
+        RX-packets: 0         RX-missed: 0          RX-bytes:  0
+        RX-errors: 0
+        RX-nombuf:  0
+        TX-packets: 101265405  TX-errors: 0          TX-bytes:  136721996771
+
+        Throughput (since last show)
+        Rx-pps:            0          Rx-bps:            0
+        Tx-pps:       564392          Tx-bps:   9892193416
+        ############################################################################
+
+    testpmd> stop
+    testpmd> quit
-- 
2.7.4


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

* Re: [dts] [PATCH V2 1/2] tests/rxtx_offload: add tx offload multi_segs setting case
  2021-08-25  3:11 ` [dts] [PATCH V2 1/2] tests/rxtx_offload: add tx offload multi_segs setting case Yu Jiang
@ 2021-08-25  3:17   ` Jiang, YuX
  0 siblings, 0 replies; 5+ messages in thread
From: Jiang, YuX @ 2021-08-25  3:17 UTC (permalink / raw)
  To: dts

[-- Attachment #1: Type: text/plain, Size: 624 bytes --]

> -----Original Message-----
> From: Jiang, YuX <yux.jiang@intel.com>
> Sent: Wednesday, August 25, 2021 11:12 AM
> To: dts@dpdk.org
> Cc: Jiang, YuX <yux.jiang@intel.com>
> Subject: [dts][PATCH V2 1/2] tests/rxtx_offload: add tx offload multi_segs
> setting case
> 
> vector path doesn't support multi-segment packet transmit, only normal
> path supports multi-segment packet transmit.
> Use 'port config 0 tx_offload multi_segs on' or "--tx-offloads=0x00008000"
> parameter to enable multi segment offload.
> 
> Signed-off-by: Yu Jiang <yux.jiang@intel.com>
> ---
Tested-by: Yu Jiang <yux.jiang@intel.com>

[-- Attachment #2: TestRxTx_Offload.log --]
[-- Type: application/octet-stream, Size: 2415456 bytes --]

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

* Re: [dts] [PATCH V2 0/2] rxtx_offload: add test plan and tests for tx offload multi_segs setting
  2021-08-25  3:11 [dts] [PATCH V2 0/2] rxtx_offload: add test plan and tests for tx offload multi_segs setting Yu Jiang
  2021-08-25  3:11 ` [dts] [PATCH V2 1/2] tests/rxtx_offload: add tx offload multi_segs setting case Yu Jiang
  2021-08-25  3:11 ` [dts] [PATCH V2 2/2] test_plans/rxtx_offload: add tx offload multi_segs setting plan Yu Jiang
@ 2021-08-27  8:25 ` Tu, Lijuan
  2 siblings, 0 replies; 5+ messages in thread
From: Tu, Lijuan @ 2021-08-27  8:25 UTC (permalink / raw)
  To: Jiang, YuX, dts; +Cc: Jiang, YuX



> -----Original Message-----
> From: dts <dts-bounces@dpdk.org> On Behalf Of Yu Jiang
> Sent: 2021年8月25日 11:12
> To: dts@dpdk.org
> Cc: Jiang, YuX <yux.jiang@intel.com>
> Subject: [dts] [PATCH V2 0/2] rxtx_offload: add test plan and tests for tx offload
> multi_segs setting
> 
> vector path doesn't support multi-segment packet transmit, only normal path
> supports multi-segment packet transmit.
> Use 'port config 0 tx_offload multi_segs on' or "--tx-offloads=0x00008000"
> parameter to enable multi segment offload.
> 
> Yu Jiang (2):
>   tests/rxtx_offload: add tx offload multi_segs setting case
>   test_plans/rxtx_offload: add tx offload multi_segs setting plan
> 
>  test_plans/rxtx_offload_test_plan.rst | 99
> +++++++++++++++++++++++++++++++++++
>  tests/TestSuite_rxtx_offload.py       | 66 +++++++++++++++++++++++
>  2 files changed, 165 insertions(+)

Applied, thanks
> 
> --
> 2.7.4


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

end of thread, other threads:[~2021-08-27  8:25 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-08-25  3:11 [dts] [PATCH V2 0/2] rxtx_offload: add test plan and tests for tx offload multi_segs setting Yu Jiang
2021-08-25  3:11 ` [dts] [PATCH V2 1/2] tests/rxtx_offload: add tx offload multi_segs setting case Yu Jiang
2021-08-25  3:17   ` Jiang, YuX
2021-08-25  3:11 ` [dts] [PATCH V2 2/2] test_plans/rxtx_offload: add tx offload multi_segs setting plan Yu Jiang
2021-08-27  8:25 ` [dts] [PATCH V2 0/2] rxtx_offload: add test plan and tests for tx offload multi_segs setting Tu, Lijuan

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).