test suite reviews and discussions
 help / color / mirror / Atom feed
* [dts] [PATCH V2 0/2] modify script to adapt changes in dpdk
@ 2023-03-20 18:05 Song Jiale
  2023-03-20 18:05 ` [dts] [PATCH V2 1/2] test_plans/ipv4_reassembly: " Song Jiale
  2023-03-20 18:05 ` [dts] [PATCH V2 2/2] tests/ipv4_reassembly: " Song Jiale
  0 siblings, 2 replies; 5+ messages in thread
From: Song Jiale @ 2023-03-20 18:05 UTC (permalink / raw)
  To: dts; +Cc: Song Jiale

DPDK increase default value for config parameter RTE_LIBRTE_IP_FRAG_MAX_FRAG from 4 to 8.
Modify the script to adapt to this change.

According to dpdk commit f8e0f8ce90303b("ip_frag: increase default maximum of fragments").

Song Jiale (2):
  test_plans/ipv4_reassembly: modify script to adapt changes in dpdk
  tests/ipv4_reassembly: modify script to adapt changes in dpdk

 test_plans/ipv4_reassembly_test_plan.rst |  5 ++-
 tests/TestSuite_ipv4_reassembly.py       | 40 ++++++++++++++++++------
 2 files changed, 32 insertions(+), 13 deletions(-)

-- 
2.25.1


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

* [dts] [PATCH V2 1/2] test_plans/ipv4_reassembly: modify script to adapt changes in dpdk
  2023-03-20 18:05 [dts] [PATCH V2 0/2] modify script to adapt changes in dpdk Song Jiale
@ 2023-03-20 18:05 ` Song Jiale
  2023-03-20 18:05 ` [dts] [PATCH V2 2/2] tests/ipv4_reassembly: " Song Jiale
  1 sibling, 0 replies; 5+ messages in thread
From: Song Jiale @ 2023-03-20 18:05 UTC (permalink / raw)
  To: dts; +Cc: Song Jiale

DPDK increase default value for config parameter RTE_LIBRTE_IP_FRAG_MAX_FRAG from 4 to 8.
Modify the script to adapt to this change.

According to dpdk commit f8e0f8ce90303b("ip_frag: increase default maximum of fragments").

Signed-off-by: Song Jiale <songx.jiale@intel.com>
---
 test_plans/ipv4_reassembly_test_plan.rst | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/test_plans/ipv4_reassembly_test_plan.rst b/test_plans/ipv4_reassembly_test_plan.rst
index 7f721123..43d18590 100644
--- a/test_plans/ipv4_reassembly_test_plan.rst
+++ b/test_plans/ipv4_reassembly_test_plan.rst
@@ -153,12 +153,11 @@ Sample command::
   ./<build_target>/examples/dpdk-ip_reassembly -c 0x2 -n 4 -- \
       -P -p 0x2 --config "(1,0,1)" --maxflows=1024 --flowttl=10s
 
-Sends 1 packet split in 5 fragments while the maximum number of supported
-fragments per packet is 4.
+Sends 1 frame split in into maximum fragments + 1 fragment, the packet can't be forwarded back.
 
 It expects:
 
-  - 5 IP packets to be sent to the DUT.
+  - (maximum fragments + 1 fragment) IP packets to be sent to the DUT.
   - 0 TCP packets being forwarded back to the TESTER.
   - 0 packets with a valid TCP checksum.
 
-- 
2.25.1


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

* [dts] [PATCH V2 2/2] tests/ipv4_reassembly: modify script to adapt changes in dpdk
  2023-03-20 18:05 [dts] [PATCH V2 0/2] modify script to adapt changes in dpdk Song Jiale
  2023-03-20 18:05 ` [dts] [PATCH V2 1/2] test_plans/ipv4_reassembly: " Song Jiale
@ 2023-03-20 18:05 ` Song Jiale
  2023-03-21  6:10   ` Peng, Yuan
  2023-03-28  1:04   ` lijuan.tu
  1 sibling, 2 replies; 5+ messages in thread
From: Song Jiale @ 2023-03-20 18:05 UTC (permalink / raw)
  To: dts; +Cc: Song Jiale

DPDK increase default value for config parameter RTE_LIBRTE_IP_FRAG_MAX_FRAG from 4 to 8.
Modify the script to adapt to this change.

According to dpdk commit f8e0f8ce90303b("ip_frag: increase default maximum of fragments").

Signed-off-by: Song Jiale <songx.jiale@intel.com>
---
 tests/TestSuite_ipv4_reassembly.py | 40 ++++++++++++++++++++++--------
 1 file changed, 30 insertions(+), 10 deletions(-)

V2:
-modify incorrect parameters.

diff --git a/tests/TestSuite_ipv4_reassembly.py b/tests/TestSuite_ipv4_reassembly.py
index 58b3fe3e..1e9f7f4e 100644
--- a/tests/TestSuite_ipv4_reassembly.py
+++ b/tests/TestSuite_ipv4_reassembly.py
@@ -10,6 +10,7 @@ Test the IP reassembly feature
 """
 
 import os
+import re
 import time
 
 from scapy.layers.inet import IP, TCP, Ether, fragment
@@ -92,7 +93,7 @@ class TestIpReassembly(TestCase):
     # Test cases.
     #
 
-    def set_max_num_of_fragments(self, num_of_fragments=4):
+    def set_max_num_of_fragments(self, num_of_fragments):
         """
         Changes the maximum number of frames by modifying the example app code.
         """
@@ -104,6 +105,22 @@ class TestIpReassembly(TestCase):
         self.dut.send_expect("rm -rf %s" % self.target, "# ", 5)
         self.dut.build_install_dpdk(self.target)
 
+    def get_dpdk_default_config(self, config_params, config_file=""):
+        if config_file == "":
+            config_file = "config/rte_config.h"
+        out = self.dut.send_expect(
+            "cat %s | grep %s" % (config_file, config_params), "[~|~\]]# "
+        )
+        if len(out) == 0:
+            self.verify(False, "Not Found Parameter %s !!!" % config_params)
+        regex = r"#define\s+%s\s+(\d+)" % config_params
+        m = re.match(regex, out)
+        if m:
+            default_value = m.group(1)
+            return int(default_value)
+        else:
+            self.verify(False, "Not Get Parameter %s Value!!!" % config_params)
+
     def set_tester_iface_mtu(self, iface, mtu=1500):
         """
         Set the interface MTU value.
@@ -393,6 +410,9 @@ class TestIpReassembly(TestCase):
         dut_ports = self.dut.get_ports(self.nic)
         dut_port = dut_ports[0]
         self.destination_mac = self.dut.get_mac_address(dut_port)
+        self.default_max_fragments = self.get_dpdk_default_config(
+            "RTE_LIBRTE_IP_FRAG_MAX_FRAG"
+        )
 
     def test_send_1K_frames_split_in_4_and_1K_maxflows(self):
         """
@@ -442,14 +462,10 @@ class TestIpReassembly(TestCase):
             self.verify_all()
             self.dut.send_expect("^C", "# ")
             time.sleep(5)
-            self.set_max_num_of_fragments(4)
-            time.sleep(5)
 
         except Exception as e:
             self.dut.send_expect("^C", "# ")
             time.sleep(2)
-            self.set_max_num_of_fragments()
-            self.compile_example_app()
             raise e
 
     def test_packets_are_forwarded_after_ttl_timeout(self):
@@ -500,12 +516,14 @@ class TestIpReassembly(TestCase):
 
     def test_send_more_fragments_than_supported(self):
         """
-        Sends 1 frame split in 5 fragments. Since the max number of
-        fragments is set to 4 by default, the packet can't be forwarded back.
+        Sends 1 frame split in into maximum fragments + 1 fragment, the packet can't be forwarded back.
         """
 
         self.test_config = IpReassemblyTestConfig(
-            self, number_of_frames=1, frags_per_frame=5, payload_size=180
+            self,
+            number_of_frames=1,
+            frags_per_frame=self.default_max_fragments + 1,
+            payload_size=(self.default_max_fragments + 1) * 40 - 20,
         )
         self.execute_example_app()
 
@@ -527,7 +545,6 @@ class TestIpReassembly(TestCase):
         self.test_config = IpReassemblyTestConfig(
             self, number_of_frames=1, flowttl="3s"
         )
-        self.compile_example_app()
         self.execute_example_app()
         self.tcpdump_start_sniffing()
 
@@ -563,7 +580,6 @@ class TestIpReassembly(TestCase):
         )
         try:
             self.set_tester_iface_mtu(self.test_config.tester_iface, mtu)
-            self.compile_example_app()
             self.execute_example_app()
             self.send_n_siff_packets()
             self.verify_all()
@@ -602,6 +618,10 @@ class TestIpReassembly(TestCase):
         """
 
         self.dut.send_expect("^C", "# ")
+        self.max_fragments = self.get_dpdk_default_config("RTE_LIBRTE_IP_FRAG_MAX_FRAG")
+        if self.max_fragments != self.default_max_fragments:
+            self.set_max_num_of_fragments(self.default_max_fragments)
+            self.compile_example_app()
 
     def tear_down_all(self):
         """
-- 
2.25.1


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

* RE: [dts] [PATCH V2 2/2] tests/ipv4_reassembly: modify script to adapt changes in dpdk
  2023-03-20 18:05 ` [dts] [PATCH V2 2/2] tests/ipv4_reassembly: " Song Jiale
@ 2023-03-21  6:10   ` Peng, Yuan
  2023-03-28  1:04   ` lijuan.tu
  1 sibling, 0 replies; 5+ messages in thread
From: Peng, Yuan @ 2023-03-21  6:10 UTC (permalink / raw)
  To: Jiale, SongX, dts; +Cc: Jiale, SongX



> -----Original Message-----
> From: Song Jiale <songx.jiale@intel.com>
> Sent: Tuesday, March 21, 2023 2:05 AM
> To: dts@dpdk.org
> Cc: Jiale, SongX <songx.jiale@intel.com>
> Subject: [dts] [PATCH V2 2/2] tests/ipv4_reassembly: modify script to adapt
> changes in dpdk
> 
> DPDK increase default value for config parameter
> RTE_LIBRTE_IP_FRAG_MAX_FRAG from 4 to 8.
> Modify the script to adapt to this change.
> 
> According to dpdk commit f8e0f8ce90303b("ip_frag: increase default
> maximum of fragments").
> 
> Signed-off-by: Song Jiale <songx.jiale@intel.com>
> ---

Acked-by: Yuan Peng <yuan.peng@intel.com>

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

* [dts] [PATCH V2 2/2] tests/ipv4_reassembly: modify script to adapt changes in dpdk
  2023-03-20 18:05 ` [dts] [PATCH V2 2/2] tests/ipv4_reassembly: " Song Jiale
  2023-03-21  6:10   ` Peng, Yuan
@ 2023-03-28  1:04   ` lijuan.tu
  1 sibling, 0 replies; 5+ messages in thread
From: lijuan.tu @ 2023-03-28  1:04 UTC (permalink / raw)
  To: dts, Song Jiale; +Cc: Song Jiale

On Mon, 20 Mar 2023 18:05:03 +0000, Song Jiale <songx.jiale@intel.com> wrote:
> DPDK increase default value for config parameter RTE_LIBRTE_IP_FRAG_MAX_FRAG from 4 to 8.
> Modify the script to adapt to this change.
> 
> According to dpdk commit f8e0f8ce90303b("ip_frag: increase default maximum of fragments").
> 
> Signed-off-by: Song Jiale <songx.jiale@intel.com>

Reviewed-by: Lijuan Tu <lijuan.tu@intel.com>
Series applied, thanks

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

end of thread, other threads:[~2023-03-28  1:04 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-03-20 18:05 [dts] [PATCH V2 0/2] modify script to adapt changes in dpdk Song Jiale
2023-03-20 18:05 ` [dts] [PATCH V2 1/2] test_plans/ipv4_reassembly: " Song Jiale
2023-03-20 18:05 ` [dts] [PATCH V2 2/2] tests/ipv4_reassembly: " Song Jiale
2023-03-21  6:10   ` Peng, Yuan
2023-03-28  1:04   ` 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).