test suite reviews and discussions
 help / color / mirror / Atom feed
* [dts] [PATCH] TestSuite_dynamic_config.py : Modified send packet count to support vector mode
@ 2019-06-10  6:09 nareddy
  2019-06-12  2:51 ` Tu, Lijuan
  0 siblings, 1 reply; 2+ messages in thread
From: nareddy @ 2019-06-10  6:09 UTC (permalink / raw)
  To: dts; +Cc: fmasood, avijay, Praneeth Reddy

From: Praneeth Reddy <nareddy@marvell.com>

In vector mode 4 packets are required.
Modified packet count to 4 in all test cases.

Signed-off-by: Praneeth Reddy <nareddy@marvell.com>
---
 tests/TestSuite_dynamic_config.py | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/tests/TestSuite_dynamic_config.py b/tests/TestSuite_dynamic_config.py
index 012421c..301f5b7 100644
--- a/tests/TestSuite_dynamic_config.py
+++ b/tests/TestSuite_dynamic_config.py
@@ -105,7 +105,7 @@ class TestDynamicConfig(TestCase):
 
         self.tester.scapy_foreground()
         self.tester.scapy_append(
-            'sendp([Ether(dst="%s", src="52:00:00:00:00:00")/Raw(load="X"*26)], iface="%s")' % (destMac, itf))
+            'sendp([Ether(dst="%s", src="52:00:00:00:00:00")/Raw(load="X"*26)], iface="%s", count=4)' % (destMac, itf))
 
         self.tester.scapy_execute()
 
@@ -142,7 +142,7 @@ class TestDynamicConfig(TestCase):
 
         # check the packet increment
         self.verify(int(cur_rxpkt) == int(pre_rxpkt)
-                    + 1, "1st packet increment check error")
+                    + 4, "1st packet increment check error")
 
         # send one packet with the portid MAC address
         self.dynamic_config_send_packet(portid, self.dest)
@@ -153,7 +153,7 @@ class TestDynamicConfig(TestCase):
 
         # check the packet increment
         self.verify(int(cur_rxpkt) == int(pre_rxpkt)
-                    + 1, "2nd packet increment check error")
+                    + 4, "2nd packet increment check error")
 
     def test_dynamic_config_disable_promiscuous(self):
         """
@@ -180,7 +180,7 @@ class TestDynamicConfig(TestCase):
               "show port stats %d" % self.dut_ports[1], "testpmd> ")
         cur_rxpkt = utils.regexp(out, "TX-packets: ([0-9]+)")
         self.verify(int(cur_rxpkt) == int(
-              pre_rxpkt) + 1, "2nd packet increment error")
+              pre_rxpkt) + 4, "2nd packet increment error")
 
     def test_dynamic_config_broadcast(self):
         """
@@ -198,7 +198,7 @@ class TestDynamicConfig(TestCase):
         out = self.dut.send_expect("show port stats %d" % self.dut_ports[1], "testpmd> ")
           
         cur_rxpkt = utils.regexp(out, "TX-packets: ([0-9]+)")
-        self.verify(int(cur_rxpkt) == 1, "not received broadcast packet")
+        self.verify(int(cur_rxpkt) == 4, "not received broadcast packet")
           
         self.dut.send_expect("clear port stats all", "testpmd> ")
 
@@ -224,7 +224,7 @@ class TestDynamicConfig(TestCase):
         out = self.dut.send_expect("show port stats %d" % self.dut_ports[1], "testpmd> ")
             
         cur_rxpkt = utils.regexp(out, "TX-packets: ([0-9]+)")
-        self.verify(int(cur_rxpkt) == 1, "enable allmulti switch, not received allmulti packet")
+        self.verify(int(cur_rxpkt) == 4, "enable allmulti switch, not received allmulti packet")
            
         self.dut.send_expect("clear port stats all", "testpmd> ")
         self.dut.send_expect("set allmulti all off", "testpmd> ")
@@ -257,7 +257,7 @@ class TestDynamicConfig(TestCase):
 
         # check the packet increment
         self.verify(int(cur_rxpkt) == int(pre_rxpkt)
-                    + 1, "1st packet increment error")
+                    + 4, "1st packet increment error")
 
         # send one packet with the portid MAC address
         self.dynamic_config_send_packet(portid, self.dest)
@@ -268,7 +268,7 @@ class TestDynamicConfig(TestCase):
 
         # check the packet increment
         self.verify(int(cur_rxpkt) == int(pre_rxpkt)
-                    + 1, "2nd packet increment error")
+                    + 4, "2nd packet increment error")
 
         #self.dut.send_expect("quit", "# ", 30)
 
-- 
1.8.3.1


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

* Re: [dts] [PATCH] TestSuite_dynamic_config.py : Modified send packet count to support vector mode
  2019-06-10  6:09 [dts] [PATCH] TestSuite_dynamic_config.py : Modified send packet count to support vector mode nareddy
@ 2019-06-12  2:51 ` Tu, Lijuan
  0 siblings, 0 replies; 2+ messages in thread
From: Tu, Lijuan @ 2019-06-12  2:51 UTC (permalink / raw)
  To: nareddy, dts; +Cc: fmasood, avijay

Applied, thanks

> -----Original Message-----
> From: dts [mailto:dts-bounces@dpdk.org] On Behalf Of
> nareddy@marvell.com
> Sent: Monday, June 10, 2019 2:09 PM
> To: dts@dpdk.org
> Cc: fmasood@marvell.com; avijay@marvell.com; Praneeth Reddy
> <nareddy@marvell.com>
> Subject: [dts] [PATCH] TestSuite_dynamic_config.py : Modified send packet
> count to support vector mode
> 
> From: Praneeth Reddy <nareddy@marvell.com>
> 
> In vector mode 4 packets are required.
> Modified packet count to 4 in all test cases.
> 
> Signed-off-by: Praneeth Reddy <nareddy@marvell.com>
> ---
>  tests/TestSuite_dynamic_config.py | 16 ++++++++--------
>  1 file changed, 8 insertions(+), 8 deletions(-)
> 
> diff --git a/tests/TestSuite_dynamic_config.py
> b/tests/TestSuite_dynamic_config.py
> index 012421c..301f5b7 100644
> --- a/tests/TestSuite_dynamic_config.py
> +++ b/tests/TestSuite_dynamic_config.py
> @@ -105,7 +105,7 @@ class TestDynamicConfig(TestCase):
> 
>          self.tester.scapy_foreground()
>          self.tester.scapy_append(
> -            'sendp([Ether(dst="%s", src="52:00:00:00:00:00")/Raw(load="X"*26)],
> iface="%s")' % (destMac, itf))
> +            'sendp([Ether(dst="%s",
> + src="52:00:00:00:00:00")/Raw(load="X"*26)], iface="%s", count=4)' %
> + (destMac, itf))
> 
>          self.tester.scapy_execute()
> 
> @@ -142,7 +142,7 @@ class TestDynamicConfig(TestCase):
> 
>          # check the packet increment
>          self.verify(int(cur_rxpkt) == int(pre_rxpkt)
> -                    + 1, "1st packet increment check error")
> +                    + 4, "1st packet increment check error")
> 
>          # send one packet with the portid MAC address
>          self.dynamic_config_send_packet(portid, self.dest) @@ -153,7 +153,7
> @@ class TestDynamicConfig(TestCase):
> 
>          # check the packet increment
>          self.verify(int(cur_rxpkt) == int(pre_rxpkt)
> -                    + 1, "2nd packet increment check error")
> +                    + 4, "2nd packet increment check error")
> 
>      def test_dynamic_config_disable_promiscuous(self):
>          """
> @@ -180,7 +180,7 @@ class TestDynamicConfig(TestCase):
>                "show port stats %d" % self.dut_ports[1], "testpmd> ")
>          cur_rxpkt = utils.regexp(out, "TX-packets: ([0-9]+)")
>          self.verify(int(cur_rxpkt) == int(
> -              pre_rxpkt) + 1, "2nd packet increment error")
> +              pre_rxpkt) + 4, "2nd packet increment error")
> 
>      def test_dynamic_config_broadcast(self):
>          """
> @@ -198,7 +198,7 @@ class TestDynamicConfig(TestCase):
>          out = self.dut.send_expect("show port stats %d" % self.dut_ports[1],
> "testpmd> ")
> 
>          cur_rxpkt = utils.regexp(out, "TX-packets: ([0-9]+)")
> -        self.verify(int(cur_rxpkt) == 1, "not received broadcast packet")
> +        self.verify(int(cur_rxpkt) == 4, "not received broadcast
> + packet")
> 
>          self.dut.send_expect("clear port stats all", "testpmd> ")
> 
> @@ -224,7 +224,7 @@ class TestDynamicConfig(TestCase):
>          out = self.dut.send_expect("show port stats %d" % self.dut_ports[1],
> "testpmd> ")
> 
>          cur_rxpkt = utils.regexp(out, "TX-packets: ([0-9]+)")
> -        self.verify(int(cur_rxpkt) == 1, "enable allmulti switch, not received
> allmulti packet")
> +        self.verify(int(cur_rxpkt) == 4, "enable allmulti switch, not
> + received allmulti packet")
> 
>          self.dut.send_expect("clear port stats all", "testpmd> ")
>          self.dut.send_expect("set allmulti all off", "testpmd> ") @@ -257,7
> +257,7 @@ class TestDynamicConfig(TestCase):
> 
>          # check the packet increment
>          self.verify(int(cur_rxpkt) == int(pre_rxpkt)
> -                    + 1, "1st packet increment error")
> +                    + 4, "1st packet increment error")
> 
>          # send one packet with the portid MAC address
>          self.dynamic_config_send_packet(portid, self.dest) @@ -268,7 +268,7
> @@ class TestDynamicConfig(TestCase):
> 
>          # check the packet increment
>          self.verify(int(cur_rxpkt) == int(pre_rxpkt)
> -                    + 1, "2nd packet increment error")
> +                    + 4, "2nd packet increment error")
> 
>          #self.dut.send_expect("quit", "# ", 30)
> 
> --
> 1.8.3.1


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

end of thread, other threads:[~2019-06-12  2:51 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-06-10  6:09 [dts] [PATCH] TestSuite_dynamic_config.py : Modified send packet count to support vector mode nareddy
2019-06-12  2:51 ` 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).