test suite reviews and discussions
 help / color / mirror / Atom feed
* [dts] [PATCH] tests/TestSuite_uni_pkt: Enable cvl nic columbiaville to support uni pkt test case and delete time sync for DPDK15109
@ 2019-08-20  2:04 zhiwei.he
  2019-08-30  3:02 ` Tu, Lijuan
  0 siblings, 1 reply; 2+ messages in thread
From: zhiwei.he @ 2019-08-20  2:04 UTC (permalink / raw)
  To: dts; +Cc: zhiwei.he

From: "zhiwei.he" <zhiwei.he@intel.com>

diff --git a/tests/TestSuite_uni_pkt.py b/tests/TestSuite_uni_pkt.py
index 2b60320..bdc0c6f 100644
--- a/tests/TestSuite_uni_pkt.py
+++ b/tests/TestSuite_uni_pkt.py
@@ -93,14 +93,21 @@ class TestUniPacket(TestCase):
         """
         Check whether L2 packet can be detected"
         """
-        self.verify(("fortville" in self.nic or "fortpark_TLV" in self.nic),
+        self.verify(("fortville" in self.nic or "fortpark_TLV" in self.nic or "columbiaville" in self.nic),
                     "L2 packet detect only support by Fortville")
-        self.L2_types = {
-            "TIMESYNC": "L2_ETHER_TIMESYNC",
-            "ARP": "L2_ETHER_ARP",
-            "LLDP": "L2_ETHER_LLDP",
+       #Change this code start for DPDK-15109, the share code doest not support TIMESYNC, once supported then will enable
+        if  "columbiaville" in self.nic:
+            self.L2_types = {
+                 "ARP": "L2_ETHER_ARP",
+                 "LLDP": "L2_ETHER_LLDP",
+            }
+        else:
+            self.L2_types = {
+                 "TIMESYNC": "L2_ETHER_TIMESYNC",
+                 "ARP": "L2_ETHER_ARP",
+                 "LLDP": "L2_ETHER_LLDP",
         }
-
+       #Change this code end for DPDK-15109, the share code doest not support TIMESYNC, once supported then will enable
         for l2_type in self.L2_types.keys():
             pkt_name = self.L2_types[l2_type]
             pkt = Packet(pkt_type=l2_type)
@@ -115,7 +122,7 @@ class TestUniPacket(TestCase):
         """
         checked that whether L3 and L4 packet can be normally detected.
         """
-        if "fortville" in self.nic or "fortpark_TLV" in self.nic:
+        if "fortville" in self.nic or "fortpark_TLV" in self.nic or "columbiaville" in self.nic:
             outerL4Type = "L4_NONFRAG"
             ipv4_default_packet_type = ["L2_ETHER", "L3_IPV4_EXT_UNKNOWN"]
         elif "niantic" in self.nic.lower() or "powerville" in self.nic.lower() or "cavium" in self.nic.lower() or "twinpond" in self.nic.lower() or "twinville" in self.nic.lower() or "sageville" in self.nic.lower() or "sagepond" in self.nic.lower() or "springville" in self.nic.lower():
@@ -133,7 +140,7 @@ class TestUniPacket(TestCase):
         }
 
         # delete the unsupported packet based on nic type
-        if "fortville" in self.nic or "fortpark_TLV" in self.nic:
+        if "fortville" in self.nic or "fortpark_TLV" in self.nic or "columbiaville" in self.nic:
             pktType.pop("MAC_IPihl_PKT")
             pktType.pop("MAC_IPihl_SCTP_PKT")
         elif "niantic" in self.nic.lower() or "powerville" in self.nic.lower() or "cavium" in self.nic.lower() or "twinpond" in self.nic.lower() or "twinville" in self.nic.lower() or "sageville" in self.nic.lower() or "sagepond" in self.nic.lower() or "springville" in self.nic.lower():
@@ -146,7 +153,7 @@ class TestUniPacket(TestCase):
         """
         checked that whether IPv6 and L4 packet can be normally detected.
         """
-        if "fortville" in self.nic or "fortpark_TLV" in self.nic:
+        if "fortville" in self.nic or "fortpark_TLV" in self.nic or "columbiaville" in self.nic:
             outerL4Type = "L4_NONFRAG"
             ipv6_default_packet_type = ["L2_ETHER", "L3_IPV6_EXT_UNKNOWN"]
         elif "niantic" in self.nic.lower() or "powerville" in self.nic.lower() or "cavium" in self.nic.lower() or "twinpond" in self.nic.lower() or "twinville" in self.nic.lower() or "sageville" in self.nic.lower() or "sagepond" in self.nic.lower() or "springville" in self.nic.lower():
@@ -162,7 +169,7 @@ class TestUniPacket(TestCase):
         }
 
         # delete the unsupported packet based on nic type
-        if "fortville" in self.nic or "fortpark_TLV" in self.nic:
+        if "fortville" in self.nic or "fortpark_TLV" in self.nic or "columbiaville" in self.nic:
             pktType.pop("MAC_IPv6FRAG_PKT_N")
         elif "niantic" in self.nic.lower() or "powerville" in self.nic.lower() or "cavium" in self.nic.lower() or "twinpond" in self.nic.lower() or "twinville" in self.nic.lower() or "sageville" in self.nic.lower() or "sagepond" in self.nic.lower() or "springville" in self.nic.lower():
             pktType.pop("MAC_IPv6FRAG_PKT_F")
@@ -174,7 +181,7 @@ class TestUniPacket(TestCase):
         checked that whether IP in IPv4 tunnel packet can be normally
         detected by Fortville.
         """
-        self.verify(("fortville" in self.nic or "fortpark_TLV" in self.nic),
+        self.verify(("fortville" in self.nic or "fortpark_TLV" in self.nic or "columbiaville" in self.nic),
                     "IP in IPv4 tunnel packet type detect only support by Fortville")
         ipv4_in_ipv4_packet_type = ["L2_ETHER", "L3_IPV4_EXT_UNKNOWN", "TUNNEL_IP", "INNER_L3_IPV4_EXT_UNKNOWN"]
         ipv6_in_ipv4_packet_type = ["L2_ETHER", "L3_IPV4_EXT_UNKNOWN", "TUNNEL_IP", "INNER_L3_IPV6_EXT_UNKNOWN"]
@@ -201,7 +208,7 @@ class TestUniPacket(TestCase):
         checked that whether IPv4 in IPv6 tunnel packet can be normally
         detected.
         """
-        self.verify(self.nic in ["niantic", "fortville_eagle", "fortville_spirit","powerville", "fortpark_TLV",
+        self.verify(self.nic in ["niantic", "columbiaville_25g", "columbiaville_100g", "fortville_eagle", "fortville_spirit","powerville", "fortpark_TLV",
             "fortville_25g", "fortville_spirit_single"], "not support %s" % self.nic)
         pktType = {
             "MAC_IP_IPv6_PKT":            ["L2_ETHER", "L3_IPV4", "TUNNEL_IP",  "INNER_L3_IPV6"],
@@ -352,7 +359,7 @@ class TestUniPacket(TestCase):
         """
         checked that whether GRE tunnel packet can be normally detected by Fortville.
         """
-        self.verify(("fortville" in self.nic or "fortpark_TLV" in self.nic),
+        self.verify(("fortville" in self.nic or "fortpark_TLV" in self.nic or "columbiaville" in self.nic),
                     "GRE tunnel packet type detect only support by Fortville")
         base_packet_type = [" L2_ETHER", " L3_IPV4_EXT_UNKNOWN", "TUNNEL_GRENAT"]
         pktType = {
@@ -371,7 +378,7 @@ class TestUniPacket(TestCase):
         checked that whether Vxlan tunnel packet can be normally detected by
         Fortville.
         """
-        self.verify(("fortville" in self.nic or "fortpark_TLV" in self.nic),
+        self.verify(("fortville" in self.nic or "fortpark_TLV" in self.nic or "columbiaville" in self.nic),
                     "Vxlan tunnel packet type detect only support by Fortville")
 
         self.dut.send_expect("rx_vxlan_port add 4789 0", "testpmd>", 10)
-- 
1.8.3.1


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

* Re: [dts] [PATCH] tests/TestSuite_uni_pkt: Enable cvl nic columbiaville to support uni pkt test case and delete time sync for DPDK15109
  2019-08-20  2:04 [dts] [PATCH] tests/TestSuite_uni_pkt: Enable cvl nic columbiaville to support uni pkt test case and delete time sync for DPDK15109 zhiwei.he
@ 2019-08-30  3:02 ` Tu, Lijuan
  0 siblings, 0 replies; 2+ messages in thread
From: Tu, Lijuan @ 2019-08-30  3:02 UTC (permalink / raw)
  To: He, Zhiwei, dts; +Cc: He, Zhiwei

Applied failed, because of code base changed.

> -----Original Message-----
> From: dts [mailto:dts-bounces@dpdk.org] On Behalf Of zhiwei.he
> Sent: Tuesday, August 20, 2019 10:04 AM
> To: dts@dpdk.org
> Cc: He, Zhiwei <zhiwei.he@intel.com>
> Subject: [dts] [PATCH] tests/TestSuite_uni_pkt: Enable cvl nic columbiaville
> to support uni pkt test case and delete time sync for DPDK15109
> 
> From: "zhiwei.he" <zhiwei.he@intel.com>
> 
> diff --git a/tests/TestSuite_uni_pkt.py b/tests/TestSuite_uni_pkt.py index
> 2b60320..bdc0c6f 100644
> --- a/tests/TestSuite_uni_pkt.py
> +++ b/tests/TestSuite_uni_pkt.py
> @@ -93,14 +93,21 @@ class TestUniPacket(TestCase):
>          """
>          Check whether L2 packet can be detected"
>          """
> -        self.verify(("fortville" in self.nic or "fortpark_TLV" in self.nic),
> +        self.verify(("fortville" in self.nic or "fortpark_TLV" in
> + self.nic or "columbiaville" in self.nic),
>                      "L2 packet detect only support by Fortville")
> -        self.L2_types = {
> -            "TIMESYNC": "L2_ETHER_TIMESYNC",
> -            "ARP": "L2_ETHER_ARP",
> -            "LLDP": "L2_ETHER_LLDP",
> +       #Change this code start for DPDK-15109, the share code doest not
> support TIMESYNC, once supported then will enable
> +        if  "columbiaville" in self.nic:
> +            self.L2_types = {
> +                 "ARP": "L2_ETHER_ARP",
> +                 "LLDP": "L2_ETHER_LLDP",
> +            }
> +        else:
> +            self.L2_types = {
> +                 "TIMESYNC": "L2_ETHER_TIMESYNC",
> +                 "ARP": "L2_ETHER_ARP",
> +                 "LLDP": "L2_ETHER_LLDP",
>          }
> -
> +       #Change this code end for DPDK-15109, the share code doest not
> + support TIMESYNC, once supported then will enable
>          for l2_type in self.L2_types.keys():
>              pkt_name = self.L2_types[l2_type]
>              pkt = Packet(pkt_type=l2_type) @@ -115,7 +122,7 @@ class
> TestUniPacket(TestCase):
>          """
>          checked that whether L3 and L4 packet can be normally detected.
>          """
> -        if "fortville" in self.nic or "fortpark_TLV" in self.nic:
> +        if "fortville" in self.nic or "fortpark_TLV" in self.nic or "columbiaville" in
> self.nic:
>              outerL4Type = "L4_NONFRAG"
>              ipv4_default_packet_type = ["L2_ETHER", "L3_IPV4_EXT_UNKNOWN"]
>          elif "niantic" in self.nic.lower() or "powerville" in self.nic.lower() or
> "cavium" in self.nic.lower() or "twinpond" in self.nic.lower() or "twinville" in
> self.nic.lower() or "sageville" in self.nic.lower() or "sagepond" in
> self.nic.lower() or "springville" in self.nic.lower():
> @@ -133,7 +140,7 @@ class TestUniPacket(TestCase):
>          }
> 
>          # delete the unsupported packet based on nic type
> -        if "fortville" in self.nic or "fortpark_TLV" in self.nic:
> +        if "fortville" in self.nic or "fortpark_TLV" in self.nic or "columbiaville" in
> self.nic:
>              pktType.pop("MAC_IPihl_PKT")
>              pktType.pop("MAC_IPihl_SCTP_PKT")
>          elif "niantic" in self.nic.lower() or "powerville" in self.nic.lower() or
> "cavium" in self.nic.lower() or "twinpond" in self.nic.lower() or "twinville" in
> self.nic.lower() or "sageville" in self.nic.lower() or "sagepond" in
> self.nic.lower() or "springville" in self.nic.lower():
> @@ -146,7 +153,7 @@ class TestUniPacket(TestCase):
>          """
>          checked that whether IPv6 and L4 packet can be normally detected.
>          """
> -        if "fortville" in self.nic or "fortpark_TLV" in self.nic:
> +        if "fortville" in self.nic or "fortpark_TLV" in self.nic or "columbiaville" in
> self.nic:
>              outerL4Type = "L4_NONFRAG"
>              ipv6_default_packet_type = ["L2_ETHER", "L3_IPV6_EXT_UNKNOWN"]
>          elif "niantic" in self.nic.lower() or "powerville" in self.nic.lower() or
> "cavium" in self.nic.lower() or "twinpond" in self.nic.lower() or "twinville" in
> self.nic.lower() or "sageville" in self.nic.lower() or "sagepond" in
> self.nic.lower() or "springville" in self.nic.lower():
> @@ -162,7 +169,7 @@ class TestUniPacket(TestCase):
>          }
> 
>          # delete the unsupported packet based on nic type
> -        if "fortville" in self.nic or "fortpark_TLV" in self.nic:
> +        if "fortville" in self.nic or "fortpark_TLV" in self.nic or "columbiaville" in
> self.nic:
>              pktType.pop("MAC_IPv6FRAG_PKT_N")
>          elif "niantic" in self.nic.lower() or "powerville" in self.nic.lower() or
> "cavium" in self.nic.lower() or "twinpond" in self.nic.lower() or "twinville" in
> self.nic.lower() or "sageville" in self.nic.lower() or "sagepond" in
> self.nic.lower() or "springville" in self.nic.lower():
>              pktType.pop("MAC_IPv6FRAG_PKT_F") @@ -174,7 +181,7 @@ class
> TestUniPacket(TestCase):
>          checked that whether IP in IPv4 tunnel packet can be normally
>          detected by Fortville.
>          """
> -        self.verify(("fortville" in self.nic or "fortpark_TLV" in self.nic),
> +        self.verify(("fortville" in self.nic or "fortpark_TLV" in
> + self.nic or "columbiaville" in self.nic),
>                      "IP in IPv4 tunnel packet type detect only support by Fortville")
>          ipv4_in_ipv4_packet_type = ["L2_ETHER", "L3_IPV4_EXT_UNKNOWN",
> "TUNNEL_IP", "INNER_L3_IPV4_EXT_UNKNOWN"]
>          ipv6_in_ipv4_packet_type = ["L2_ETHER", "L3_IPV4_EXT_UNKNOWN",
> "TUNNEL_IP", "INNER_L3_IPV6_EXT_UNKNOWN"] @@ -201,7 +208,7 @@
> class TestUniPacket(TestCase):
>          checked that whether IPv4 in IPv6 tunnel packet can be normally
>          detected.
>          """
> -        self.verify(self.nic in ["niantic", "fortville_eagle",
> "fortville_spirit","powerville", "fortpark_TLV",
> +        self.verify(self.nic in ["niantic", "columbiaville_25g",
> + "columbiaville_100g", "fortville_eagle",
> + "fortville_spirit","powerville", "fortpark_TLV",
>              "fortville_25g", "fortville_spirit_single"], "not support %s" % self.nic)
>          pktType = {
>              "MAC_IP_IPv6_PKT":            ["L2_ETHER", "L3_IPV4", "TUNNEL_IP",
> "INNER_L3_IPV6"],
> @@ -352,7 +359,7 @@ class TestUniPacket(TestCase):
>          """
>          checked that whether GRE tunnel packet can be normally detected by
> Fortville.
>          """
> -        self.verify(("fortville" in self.nic or "fortpark_TLV" in self.nic),
> +        self.verify(("fortville" in self.nic or "fortpark_TLV" in
> + self.nic or "columbiaville" in self.nic),
>                      "GRE tunnel packet type detect only support by Fortville")
>          base_packet_type = [" L2_ETHER", " L3_IPV4_EXT_UNKNOWN",
> "TUNNEL_GRENAT"]
>          pktType = {
> @@ -371,7 +378,7 @@ class TestUniPacket(TestCase):
>          checked that whether Vxlan tunnel packet can be normally detected by
>          Fortville.
>          """
> -        self.verify(("fortville" in self.nic or "fortpark_TLV" in self.nic),
> +        self.verify(("fortville" in self.nic or "fortpark_TLV" in
> + self.nic or "columbiaville" in self.nic),
>                      "Vxlan tunnel packet type detect only support by Fortville")
> 
>          self.dut.send_expect("rx_vxlan_port add 4789 0", "testpmd>", 10)
> --
> 1.8.3.1


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

end of thread, other threads:[~2019-08-30  3:02 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-08-20  2:04 [dts] [PATCH] tests/TestSuite_uni_pkt: Enable cvl nic columbiaville to support uni pkt test case and delete time sync for DPDK15109 zhiwei.he
2019-08-30  3:02 ` 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).