test suite reviews and discussions
 help / color / mirror / Atom feed
* [PATCH v1] framework/flow: Remove items not supported by testpmd
@ 2022-02-07 16:00 ohilyard
  2022-02-11  3:18 ` Tu, Lijuan
  0 siblings, 1 reply; 2+ messages in thread
From: ohilyard @ 2022-02-07 16:00 UTC (permalink / raw)
  To: dts; +Cc: lijuan.tu, Owen Hilyard

From: Owen Hilyard <ohilyard@iol.unh.edu>

Testpmd does not have full support for rte_flow at this point, and the
rte_flow test suite has not been kept up to date with changes in
rte_flow. As such, the logic for generating the flows had to be updated
to disable some test suites to avoid false failures.

Signed-off-by: Owen Hilyard <ohilyard@iol.unh.edu>
---
 framework/flow/flow_pattern_items.py | 500 ++++++++++++++-------------
 1 file changed, 254 insertions(+), 246 deletions(-)

diff --git a/framework/flow/flow_pattern_items.py b/framework/flow/flow_pattern_items.py
index 3bdd25ce..0c46f2f3 100644
--- a/framework/flow/flow_pattern_items.py
+++ b/framework/flow/flow_pattern_items.py
@@ -136,54 +136,57 @@ class FlowItemArp_eth_ipv4(PatternFlowItem):
     - Default ``mask`` matches SHA, SPA, THA and TPA.
     """
     possible_properties = {
-        'hdr':
-            ('arp_eth_ipv4 hdr is 1',
-             frozenset({"Ether() / ARP(hwtype=1) / Raw('\\x00' * 64)"}),
-
-             frozenset({"Ether() / ARP(hwtype=2) / Raw('\\x00' * 64)",
-                        "Ether() / ARP(hwtype=3) / Raw('\\x00' * 64)",
-                        "Ether() / ARP(hwtype=6) / Raw('\\x00' * 64)",
-                        "Ether() / ARP(hwtype-15) / Raw('\\x00' * 64)"
-                        })),
-        'pro':
-            ('arp_eth_ipv4 pro is 0x0800',
-             frozenset({"Ether() / ARP(ptype=0x0800) / Raw('\\x00' * 64)"}),
-
-             frozenset({"Ether() / ARP(ptype=0x0800) / Raw('\\x00' * 64)",
-                        "Ether() / ARP(ptype=0x0842) / Raw('\\x00' * 64)",
-                        "Ether() / ARP(ptype=0x6004) / Raw('\\x00' * 64)",
-                        "Ether() / ARP(ptype=0x809b) / Raw('\\x00' * 64)"
-                        })),
-
-        'hln':
-            ('arp_eth_ipv4 hln is 6',
-             frozenset({"Ether() / ARP(hwlen=6) / Raw('\\x00' * 64)"}),
-
-             frozenset({"Ether() / ARP(hwlen=12) / Raw('\\x00' * 64)",
-                        "Ether() / ARP(hwlen=2) / Raw('\\x00' * 64)",
-                        "Ether() / ARP(hwlen=8) / Raw('\\x00' * 64)",
-                        "Ether() / ARP(hwlen=4) / Raw('\\x00' * 64)"
-                        })),
-
-        'pln':
-            ('arp_eth_ipv4 pln is 4',
-             frozenset({"Ether() / ARP(plen=4) / Raw('\\x00' * 64)"}),
-
-             frozenset({"Ether() / ARP(plen=6) / Raw('\\x00' * 64)",
-                        "Ether() / ARP(plen=2) / Raw('\\x00' * 64)",
-                        "Ether() / ARP(plen=8) / Raw('\\x00' * 64)",
-                        "Ether() / ARP(plen=12) / Raw('\\x00' * 64)"
-                        })),
-
-        'op':
-            ('arp_eth_ipv4 op is 1',
-             frozenset({"Ether() / ARP(op=1) / Raw('\\x00' * 64)"}),
-
-             frozenset({"Ether() / ARP(op=2) / Raw('\\x00' * 64)",
-                        "Ether() / ARP(op=3) / Raw('\\x00' * 64)",
-                        "Ether() / ARP(op=4) / Raw('\\x00' * 64)",
-                        "Ether() / ARP(op=5) / Raw('\\x00' * 64)"
-                        })),
+        # THE FOLLOWING PROPERTIES ARE UNSUPPORTED BY TESTPMD AT THE TIME OF WRITING.
+        # THEY CAN BE ENABLED ONCE TESTPMD SUPPORTS THEM
+        # 'hdr':
+        #     ('arp_eth_ipv4 hdr is 1',
+        #      frozenset({"Ether() / ARP(hwtype=1) / Raw('\\x00' * 64)"}),
+        #
+        #      frozenset({"Ether() / ARP(hwtype=2) / Raw('\\x00' * 64)",
+        #                 "Ether() / ARP(hwtype=3) / Raw('\\x00' * 64)",
+        #                 "Ether() / ARP(hwtype=6) / Raw('\\x00' * 64)",
+        #                 "Ether() / ARP(hwtype-15) / Raw('\\x00' * 64)"
+        #                 })),
+        # 'pro':
+        #     ('arp_eth_ipv4 pro is 0x0800',
+        #      frozenset({"Ether() / ARP(ptype=0x0800) / Raw('\\x00' * 64)"}),
+        #
+        #      frozenset({"Ether() / ARP(ptype=0x0800) / Raw('\\x00' * 64)",
+        #                 "Ether() / ARP(ptype=0x0842) / Raw('\\x00' * 64)",
+        #                 "Ether() / ARP(ptype=0x6004) / Raw('\\x00' * 64)",
+        #                 "Ether() / ARP(ptype=0x809b) / Raw('\\x00' * 64)"
+        #                 })),
+        #
+        # 'hln':
+        #     ('arp_eth_ipv4 hln is 6',
+        #      frozenset({"Ether() / ARP(hwlen=6) / Raw('\\x00' * 64)"}),
+        #
+        #      frozenset({"Ether() / ARP(hwlen=12) / Raw('\\x00' * 64)",
+        #                 "Ether() / ARP(hwlen=2) / Raw('\\x00' * 64)",
+        #                 "Ether() / ARP(hwlen=8) / Raw('\\x00' * 64)",
+        #                 "Ether() / ARP(hwlen=4) / Raw('\\x00' * 64)"
+        #                 })),
+        #
+        # 'pln':
+        #     ('arp_eth_ipv4 pln is 4',
+        #      frozenset({"Ether() / ARP(plen=4) / Raw('\\x00' * 64)"}),
+        #
+        #      frozenset({"Ether() / ARP(plen=6) / Raw('\\x00' * 64)",
+        #                 "Ether() / ARP(plen=2) / Raw('\\x00' * 64)",
+        #                 "Ether() / ARP(plen=8) / Raw('\\x00' * 64)",
+        #                 "Ether() / ARP(plen=12) / Raw('\\x00' * 64)"
+        #                 })),
+        #
+        # 'op':
+        #     ('arp_eth_ipv4 op is 1',
+        #      frozenset({"Ether() / ARP(op=1) / Raw('\\x00' * 64)"}),
+        #
+        #      frozenset({"Ether() / ARP(op=2) / Raw('\\x00' * 64)",
+        #                 "Ether() / ARP(op=3) / Raw('\\x00' * 64)",
+        #                 "Ether() / ARP(op=4) / Raw('\\x00' * 64)",
+        #                 "Ether() / ARP(op=5) / Raw('\\x00' * 64)"
+        #                 })),
+        # END UNSUPPORTED PROPERTIES
 
         'sha':
             ('arp_eth_ipv4 sha is 90:61:ae:fd:41:43',
@@ -282,11 +285,12 @@ class FlowItemGre(PatternFlowItem):
     """
     possible_properties = {
         'c_rsvd0_ver':
-           ('gre c_rsvd0_ver is 0',
-            frozenset({"Ether() / GRE(chksum_present=0, version=0) / Raw('\\x00' * 64)"}),
+            ('gre c_rsvd0_ver is 0',
+             frozenset({"Ether() / GRE(chksum_present=0, version=0) / Raw('\\x00' * 64)"}),
 
-           frozenset({"Ether() / GRE(chksum_present=1, version=0)) / Raw('\\x00' * 64)", #this is the only other option
-                      })),
+             frozenset({"Ether() / GRE(chksum_present=1, version=0)) / Raw('\\x00' * 64)",
+                        # this is the only other option
+                        })),
         'protocol':
             ('gre protocol is 0x0800',
              frozenset({"Ether() / GRE(proto=0x0800) / Raw('\\x00' * 64)"}),
@@ -315,7 +319,18 @@ class FlowItemIcmp(PatternFlowItem):
     - Default ``mask`` matches ICMP type and code only.
     """
     possible_properties = {
-
+        # THE FOLLOWING PROPERTIES ARE UNSUPPORTED BY TESTPMD AT THE TIME OF WRITING.
+        # THEY CAN BE ENABLED ONCE TESTPMD SUPPORTS THEM
+        # 'icmp_cksum':
+        #     ('icmp cksum is 0x0800',
+        #      frozenset({"Ether() / ICMP() / UDP() / Raw('\x00' * 64)"}),
+        #
+        #      frozenset({"Ether() / ICMP() / UDP() / Raw('\x00' * 64)",
+        #                 "Ether() / ICMP() / UDP() / Raw('\x00' * 64)",
+        #                 "Ether() / ICMP() / UDP() / Raw('\x00' * 64)",
+        #                 "Ether() / ICMP() / UDP() / Raw('\x00' * 64)"
+        #                 })),
+        # END UNSUPPORTED PROPERTIES
         'icmp_type':
             ('icmp type is 3',
              frozenset({"Ether() / ICMP(type=3) / Raw('\\x00' * 64)"}),
@@ -335,32 +350,23 @@ class FlowItemIcmp(PatternFlowItem):
                         "Ether() / ICMP(type=11, code=1) / Raw('\\x00' * 64)",
                         "Ether() / ICMP(type=12, code=2) / Raw('\\x00' * 64)"
                         })),
-        'icmp_cksum':
-           ('icmp cksum is 0x0800',
-            frozenset({"Ether() / ICMP() / UDP() / Raw('\x00' * 64)"}),
-
-            frozenset({"Ether() / ICMP() / UDP() / Raw('\x00' * 64)",
-                       "Ether() / ICMP() / UDP() / Raw('\x00' * 64)",
-                       "Ether() / ICMP() / UDP() / Raw('\x00' * 64)",
-                       "Ether() / ICMP() / UDP() / Raw('\x00' * 64)"
-                       })),
         'icmp_ident':
-           ('icmp ident is 0x0800',
-            frozenset({"Ether() / ICMP() / UDP() / Raw('\x00' * 64)"}),
-
-            frozenset({"Ether() / ICMP() / UDP() / Raw('\x00' * 64)",
-                       "Ether() / ICMP() / UDP() / Raw('\x00' * 64)",
-                       "Ether() / ICMP() / UDP() / Raw('\x00' * 64)",
-                       "Ether() / ICMP() / UDP() / Raw('\x00' * 64)"
-                       })),
-        'icmp_seq_nb':
-           ('icmp seq_nb is 0x0800',
-            frozenset({"Ether() / ICMP(proto=0x0800) / UDP() / Raw('\x00' * 64)"}),
-
-            frozenset({"Ether() / ICMP() / UDP() / Raw('\x00' * 64)",
-                       "Ether() / ICMP() / UDP() / Raw('\x00' * 64)",
-                       "Ether() / ICMP() / UDP() / Raw('\x00' * 64)",
-                       "Ether() / ICMP() / UDP() / Raw('\x00' * 64)"
+            ('icmp ident is 0x0800',
+             frozenset({"Ether() / ICMP() / UDP() / Raw('\x00' * 64)"}),
+
+             frozenset({"Ether() / ICMP() / UDP() / Raw('\x00' * 64)",
+                        "Ether() / ICMP() / UDP() / Raw('\x00' * 64)",
+                        "Ether() / ICMP() / UDP() / Raw('\x00' * 64)",
+                        "Ether() / ICMP() / UDP() / Raw('\x00' * 64)"
+                        })),
+        'icmp_seq':
+            ('icmp seq is 0x0800',
+             frozenset({"Ether() / ICMP(proto=0x0800) / UDP() / Raw('\x00' * 64)"}),
+
+             frozenset({"Ether() / ICMP() / UDP() / Raw('\x00' * 64)",
+                        "Ether() / ICMP() / UDP() / Raw('\x00' * 64)",
+                        "Ether() / ICMP() / UDP() / Raw('\x00' * 64)",
+                        "Ether() / ICMP() / UDP() / Raw('\x00' * 64)"
                         })),
     }
 
@@ -375,9 +381,19 @@ class FlowItemIcmp6(PatternFlowItem):
     - ``checksum``: ICMPv6 checksum.
     - Default ``mask`` matches ``type`` and ``code``.
     """
-    # ICMP6 NOT SUPPORTED BY TESTPMD.
-    # DO NOT UNCOMMENT THE FOLLOWING TEST CASES UNTIL IT IS SUPPORTED.
     possible_properties = {
+        # THE FOLLOWING PROPERTIES ARE UNSUPPORTED BY TESTPMD AT THE TIME OF WRITING.
+        # THEY CAN BE ENABLED ONCE TESTPMD SUPPORTS THEM
+        # 'checksum':
+        #     ('icmp6 cksum is 0x1234',
+        #      frozenset({"Ether() / ICMPv6DestUnreach(cksum=0x1234) / Raw('\\x00' * 64)"}),
+        #
+        #      frozenset({"Ether() / ICMPv6DestUnreach(cksum=0x4321) / Raw('\\x00' * 64)",
+        #                 "Ether() / ICMPv6DestUnreach(cksum=0xffff) / Raw('\\x00' * 64)",
+        #                 "Ether() / ICMPv6DestUnreach(cksum=0x1233) / Raw('\\x00' * 64)",
+        #                 "Ether() / ICMPv6DestUnreach(cksum=0x1010) / Raw('\\x00' * 64)"
+        #                 })),
+        # END UNSUPPORTED PROPERTIES
         'type':
             ('icmp6 type is 1',  # Destination Unreachable
              frozenset({"Ether() / ICMPv6DestUnreach(type=1) / Raw('\\x00' * 64)"}),
@@ -396,16 +412,6 @@ class FlowItemIcmp6(PatternFlowItem):
                         "Ether() / ICMPv6DestUnreach(code=3) / Raw('\\x00' * 64)",
                         "Ether() / ICMPv6DestUnreach(code=4) / Raw('\\x00' * 64)"
                         })),
-
-        'checksum':
-            ('icmp6 cksum is 0x1234',
-             frozenset({"Ether() / ICMPv6DestUnreach(cksum=0x1234) / Raw('\\x00' * 64)"}),
-
-             frozenset({"Ether() / ICMPv6DestUnreach(cksum=0x4321) / Raw('\\x00' * 64)",
-                        "Ether() / ICMPv6DestUnreach(cksum=0xffff) / Raw('\\x00' * 64)",
-                        "Ether() / ICMPv6DestUnreach(cksum=0x1233) / Raw('\\x00' * 64)",
-                        "Ether() / ICMPv6DestUnreach(cksum=0x1010) / Raw('\\x00' * 64)"
-                        })),
     }
 
 
@@ -496,25 +502,25 @@ class FlowItemIpv6(PatternFlowItem):
 
     possible_properties = {
         # THE FOLLOWING PROPERTIES ARE UNSUPPORTED BY TESTPMD AT THE TIME OF WRITING.
-        # They are still tested to future proof this test suite.
-        'vtc_flow':
-            ('ipv6 vtc_flow is 0x0',
-             frozenset({"Ether() / IPv6(tc=0, fl=0, version=0) / Raw('\\x00' * 64)"}),
-
-             frozenset({"Ether() / IPv6(tc=1, fl=0, version=0) / Raw('\\x00' * 64)",
-                        "Ether() / IPv6(tc=0, fl=0xABCD, version=0) / Raw('\\x00' * 64)",
-                        "Ether() / IPv6(tc=0, fl=0, version=1) / Raw('\\x00' * 64)",
-                        "Ether() / IPv6(tc=6, fl=0x9999, version=1) / Raw('\\x00' * 64)"
-                        })),
-        'payload_len':
-            ('ipv6 payload_len is 64',
-             frozenset({"Ether() / IPv6(plen=64) / Raw('\\x00' * 64)"}),
-
-             frozenset({"Ether() / IPv6(plen=32) / Raw('\\x00' * 64)",
-                        "Ether() / IPv6(plen=128) / Raw('\\x00' * 64)",
-                        "Ether() / IPv6(plen=5000) / Raw('\\x00' * 64)",
-                        "Ether() / IPv6(plen=4) / Raw('\\x00' * 64)"
-                        })),
+        # THEY CAN BE ENABLED ONCE TESTPMD SUPPORTS THEM
+        # 'vtc_flow':
+        #     ('ipv6 vtc_flow is 0x0',
+        #      frozenset({"Ether() / IPv6(tc=0, fl=0, version=0) / Raw('\\x00' * 64)"}),
+
+        #      frozenset({"Ether() / IPv6(tc=1, fl=0, version=0) / Raw('\\x00' * 64)",
+        #                 "Ether() / IPv6(tc=0, fl=0xABCD, version=0) / Raw('\\x00' * 64)",
+        #                 "Ether() / IPv6(tc=0, fl=0, version=1) / Raw('\\x00' * 64)",
+        #                 "Ether() / IPv6(tc=6, fl=0x9999, version=1) / Raw('\\x00' * 64)"
+        #                 })),
+        # 'payload_len':
+        #     ('ipv6 payload_len is 64',
+        #      frozenset({"Ether() / IPv6(plen=64) / Raw('\\x00' * 64)"}),
+
+        #      frozenset({"Ether() / IPv6(plen=32) / Raw('\\x00' * 64)",
+        #                 "Ether() / IPv6(plen=128) / Raw('\\x00' * 64)",
+        #                 "Ether() / IPv6(plen=5000) / Raw('\\x00' * 64)",
+        #                 "Ether() / IPv6(plen=4) / Raw('\\x00' * 64)"
+        #                 })),
         # END UNSUPPORTED PROPERTIES
         'tc':
             ('ipv6 tc is 0',
@@ -655,34 +661,34 @@ class FlowItemTcp(PatternFlowItem):
     """
     possible_properties = {
         # THE FOLLOWING PROPERTIES ARE UNSUPPORTED BY TESTPMD AT THE TIME OF WRITING.
-        # They are still tested to future proof this test suite.
-        'data_off':
-            ('tcp data_off is 0',
-             frozenset({"Ether() / IP() / TCP(dataofs=0) / Raw('\\x00' * 64)"}),
-
-             frozenset({"Ether() /  IP() / TCP(dataofs=1) / Raw('\\x00' * 64)",
-                        "Ether() /  IP() / TCP(dataofs=2) / Raw('\\x00' * 64)",
-                        "Ether() /  IP() / TCP(dataofs=3) / Raw('\\x00' * 64)",
-                        "Ether() /  IP() / TCP(dataofs=4) / Raw('\\x00' * 64)"
-                        })),
-        'rx_win':
-            ('tcp rx_win is 64',
-             frozenset({"Ether() /  IP() / TCP(window=64)/ Raw('\\x00' * 64)"}),
-
-             frozenset({"Ether() /  IP() / TCP(window=16)/ Raw('\\x00' * 64)",
-                        "Ether() /  IP() / TCP(window=128) / Raw('\\x00' * 64)",
-                        "Ether() /  IP() / TCP(window=32) / Raw('\\x00' * 64)",
-                        "Ether() /  IP() / TCP(window=255) / Raw('\\x00' * 64)"
-                        })),
-        'cksum':
-            ('tcp cksum is 0x1234',
-             frozenset({"Ether() /  IP() / TCP(chksum=0x1234) / Raw('\\x00' * 64)"}),
-
-             frozenset({"Ether() / IP() / TCP(chksum=0x4321) / Raw('\\x00' * 64)",
-                        "Ether() /  IP() / TCP(chksum=0xffff) / Raw('\\x00' * 64)",
-                        "Ether() /  IP() / TCP(chksum=0x9999) / Raw('\\x00' * 64)",
-                        "Ether() /  IP() / TCP(chksum=0x1233)  / Raw('\\x00' * 64)"
-                        })),
+        # THEY CAN BE ENABLED ONCE TESTPMD SUPPORTS THEM
+        # 'data_off':
+        #     ('tcp data_off is 0',
+        #      frozenset({"Ether() / IP() / TCP(dataofs=0) / Raw('\\x00' * 64)"}),
+
+        #      frozenset({"Ether() /  IP() / TCP(dataofs=1) / Raw('\\x00' * 64)",
+        #                 "Ether() /  IP() / TCP(dataofs=2) / Raw('\\x00' * 64)",
+        #                 "Ether() /  IP() / TCP(dataofs=3) / Raw('\\x00' * 64)",
+        #                 "Ether() /  IP() / TCP(dataofs=4) / Raw('\\x00' * 64)"
+        #                 })),
+        # 'rx_win':
+        #     ('tcp rx_win is 64',
+        #      frozenset({"Ether() /  IP() / TCP(window=64)/ Raw('\\x00' * 64)"}),
+
+        #      frozenset({"Ether() /  IP() / TCP(window=16)/ Raw('\\x00' * 64)",
+        #                 "Ether() /  IP() / TCP(window=128) / Raw('\\x00' * 64)",
+        #                 "Ether() /  IP() / TCP(window=32) / Raw('\\x00' * 64)",
+        #                 "Ether() /  IP() / TCP(window=255) / Raw('\\x00' * 64)"
+        #                 })),
+        # 'cksum':
+        #     ('tcp cksum is 0x1234',
+        #      frozenset({"Ether() /  IP() / TCP(chksum=0x1234) / Raw('\\x00' * 64)"}),
+
+        #      frozenset({"Ether() / IP() / TCP(chksum=0x4321) / Raw('\\x00' * 64)",
+        #                 "Ether() /  IP() / TCP(chksum=0xffff) / Raw('\\x00' * 64)",
+        #                 "Ether() /  IP() / TCP(chksum=0x9999) / Raw('\\x00' * 64)",
+        #                 "Ether() /  IP() / TCP(chksum=0x1233)  / Raw('\\x00' * 64)"
+        #                 })),
         # END UNSUPPORTED PROPERTIES
         'src':
             ('tcp src is 3838',
@@ -727,25 +733,25 @@ class FlowItemUdp(PatternFlowItem):
 
     possible_properties = {
         # THE FOLLOWING PROPERTIES ARE UNSUPPORTED BY TESTPMD AT THE TIME OF WRITING.
-        # They are still tested to future proof this test suite.
-        'dgram_len':
-            ('udp dgram_len is 64',
-             frozenset({"Ether() / IP() / UDP(len=64) / Raw('\\x00' * 64)"}),
-
-             frozenset({"Ether() /  IP() / UDP(len=128) / Raw('\\x00' * 64)",
-                        "Ether() /  IP() / UDP(len=32) / Raw('\\x00' * 64)",
-                        "Ether() /  IP() / UDP(len=16) / Raw('\\x00' * 64)",
-                        "Ether() /  IP() / UDP(len=255) / Raw('\\x00' * 64)"
-                        })),
-        'dgram_cksum':
-            ('udp dgram_cksum is 0x1234',
-             frozenset({"Ether() /  IP() / UDP(chksum=0x1234) / Raw('\\x00' * 64)"}),
-
-             frozenset({"Ether() / IP() / UDP(chksum=0x4321) / Raw('\\x00' * 64)",
-                        "Ether() /  IP() / UDP(chksum=0xffff) / Raw('\\x00' * 64)",
-                        "Ether() /  IP() / UDP(chksum=0x9999) / Raw('\\x00' * 64)",
-                        "Ether() /  IP() / UDP(chksum=0x1233)  / Raw('\\x00' * 64)"
-                        })),
+        # THEY MAY BE RE-ENABLED ONCE TESTPMD SUPPORTS THEIR USE
+        # 'dgram_len':
+        #     ('udp dgram_len is 64',
+        #      frozenset({"Ether() / IP() / UDP(len=64) / Raw('\\x00' * 64)"}),
+        #
+        #      frozenset({"Ether() /  IP() / UDP(len=128) / Raw('\\x00' * 64)",
+        #                 "Ether() /  IP() / UDP(len=32) / Raw('\\x00' * 64)",
+        #                 "Ether() /  IP() / UDP(len=16) / Raw('\\x00' * 64)",
+        #                 "Ether() /  IP() / UDP(len=255) / Raw('\\x00' * 64)"
+        #                 })),
+        # 'dgram_cksum':
+        #     ('udp dgram_cksum is 0x1234',
+        #      frozenset({"Ether() /  IP() / UDP(chksum=0x1234) / Raw('\\x00' * 64)"}),
+        #
+        #      frozenset({"Ether() / IP() / UDP(chksum=0x4321) / Raw('\\x00' * 64)",
+        #                 "Ether() /  IP() / UDP(chksum=0xffff) / Raw('\\x00' * 64)",
+        #                 "Ether() /  IP() / UDP(chksum=0x9999) / Raw('\\x00' * 64)",
+        #                 "Ether() /  IP() / UDP(chksum=0x1233)  / Raw('\\x00' * 64)"
+        #                 })),
         # END UNSUPPORTED PROPERTIES
 
         'src':
@@ -795,6 +801,18 @@ class FlowItemVlan(PatternFlowItem):
     tpid in testpmd = type in scapy
     """
     possible_properties = {
+        # THE FOLLOWING PROPERTIES ARE UNSUPPORTED BY TESTPMD AT THE TIME OF WRITING.
+        # THEY MAY BE RE-ENABLED ONCE TESTPMD SUPPORTS THEIR USE
+        # 'tpid':
+        #     ('vlan tpid is 0x8100',  # standard value
+        #      frozenset({"Ether() / Dot1Q(type=0x8100) / Raw('\\x00' * 64)"}),
+        #
+        #      frozenset({"Ether() /  Dot1Q(type=0x0800) / Raw('\\x00' * 64)",
+        #                 "Ether() /  Dot1Q(type=0x0842) / Raw('\\x00' * 64)",
+        #                 "Ether() /  Dot1Q(type=0x809b) / Raw('\\x00' * 64)",
+        #                 "Ether() /  Dot1Q(type=0x86dd) / Raw('\\x00' * 64)"
+        #                 })),
+        # END UNSUPPORTED PROPERTIES
 
         'tci':
             ('vlan tci is 0xaaaa',
@@ -831,16 +849,6 @@ class FlowItemVlan(PatternFlowItem):
                         "Ether() /  Dot1Q(vlan=0x1f5) / Raw('\\x00' * 64)",
                         "Ether() /  Dot1Q(vlan=0x999) / Raw('\\x00' * 64)"
                         })),
-
-        'tpid':
-            ('vlan tpid is 0x8100',  # standard value
-             frozenset({"Ether() / Dot1Q(type=0x8100) / Raw('\\x00' * 64)"}),
-
-             frozenset({"Ether() /  Dot1Q(type=0x0800) / Raw('\\x00' * 64)",
-                        "Ether() /  Dot1Q(type=0x0842) / Raw('\\x00' * 64)",
-                        "Ether() /  Dot1Q(type=0x809b) / Raw('\\x00' * 64)",
-                        "Ether() /  Dot1Q(type=0x86dd) / Raw('\\x00' * 64)"
-                        })),
     }
 
 
@@ -859,47 +867,47 @@ class FlowItemVxlan(PatternFlowItem):
     """
 
 
-possible_properties = {
-    # THE FOLLOWING PROPERTIES ARE UNSUPPORTED BY TESTPMD AT THE TIME OF WRITING.
-    # They are still tested to future proof this test suite.
-    'rsvd0':
-        ('vxlan rsvd0 is 0x000000',
-         frozenset({"Ether() / IP() / VXLAN(reserved0=0) / Raw('\\x00' * 64)"}),
-
-         frozenset({"Ether() /  IP() / VXLAN(reserved0=1) / Raw('\\x00' * 64)",
-                    "Ether() /  IP() /  VXLAN(reserved0=2) / Raw('\\x00' * 64)",
-                    "Ether() /  IP() /  VXLAN(reserved0=3) / Raw('\\x00' * 64)",
-                    "Ether() /  IP()  / VXLAN(reserved0=4) / Raw('\\x00' * 64)"
-                    })),
-    'rsvd1':
-        ('vxlan rsvd1 is 0x00',
-         frozenset({"Ether() /  IP() /  VXLAN(reserved0=0) / Raw('\\x00' * 64)"}),
-
-         frozenset({"Ether() / IP() /  VXLAN(reserved0=1) / Raw('\\x00' * 64)",
-                    "Ether() /  IP() /  VXLAN(reserved0=2) / Raw('\\x00' * 64)",
-                    "Ether() /  IP() / VXLAN(reserved0=3) / Raw('\\x00' * 64)",
-                    "Ether() /  IP() /  VXLAN(reserved0=4) / Raw('\\x00' * 64)"
-                    })),
-    'flags':
-        ('vxlan flags is 0x08',
-         frozenset({"Ether() /  IP() /  VXLAN(flags=0x08) / Raw('\\x00' * 64)"}),
-
-         frozenset({"Ether() / IP() /  VXLAN(flags=0x80) / Raw('\\x00' * 64)",
-                    "Ether() /  IP() /  VXLAN(flags=0x00) / Raw('\\x00' * 64)",
-                    "Ether() /  IP() / VXLAN(flags=0x99) / Raw('\\x00' * 64)",
-                    "Ether() /  IP() /  VXLAN(flags=0x01) / Raw('\\x00' * 64)"
-                    })),
-    # END UNSUPPORTED PROPERTIES
-    'vni':  # a 3-byte value
-        ('vxlan vni is 0x112233',
-         frozenset({"Ether() / IP() / VXLAN(vni=0x112233) / Raw('\\x00' * 64)"}),
-
-         frozenset({"Ether() / IP() / VXLAN(vni=0x112234) / Raw('\\x00' * 64)",
-                    "Ether() / IP() / VXLAN(vni=0x123456) / Raw('\\x00' * 64)",
-                    "Ether() / IP() / VXLAN(vni=0xaabbcc) / Raw('\\x00' * 64)",
-                    "Ether() / IP() / VXLAN(vni=0x999999) / Raw('\\x00' * 64)"
-                    })),
-}
+    possible_properties = {
+        # THE FOLLOWING PROPERTIES ARE UNSUPPORTED BY TESTPMD AT THE TIME OF WRITING.
+        # THEY CAN BE ENABLED ONCE TESTPMD SUPPORTS THEM
+        # 'rsvd0':
+        #     ('vxlan rsvd0 is 0x000000',
+        #      frozenset({"Ether() / IP() / VXLAN(reserved0=0) / Raw('\\x00' * 64)"}),
+        #
+        #      frozenset({"Ether() /  IP() / VXLAN(reserved0=1) / Raw('\\x00' * 64)",
+        #                 "Ether() /  IP() /  VXLAN(reserved0=2) / Raw('\\x00' * 64)",
+        #                 "Ether() /  IP() /  VXLAN(reserved0=3) / Raw('\\x00' * 64)",
+        #                 "Ether() /  IP()  / VXLAN(reserved0=4) / Raw('\\x00' * 64)"
+        #                 })),
+        # 'rsvd1':
+        #     ('vxlan rsvd1 is 0x00',
+        #      frozenset({"Ether() /  IP() /  VXLAN(reserved0=0) / Raw('\\x00' * 64)"}),
+        #
+        #      frozenset({"Ether() / IP() /  VXLAN(reserved0=1) / Raw('\\x00' * 64)",
+        #                 "Ether() /  IP() /  VXLAN(reserved0=2) / Raw('\\x00' * 64)",
+        #                 "Ether() /  IP() / VXLAN(reserved0=3) / Raw('\\x00' * 64)",
+        #                 "Ether() /  IP() /  VXLAN(reserved0=4) / Raw('\\x00' * 64)"
+        #                 })),
+        # 'flags':
+        #     ('vxlan flags is 0x08',
+        #      frozenset({"Ether() /  IP() /  VXLAN(flags=0x08) / Raw('\\x00' * 64)"}),
+        #
+        #      frozenset({"Ether() / IP() /  VXLAN(flags=0x80) / Raw('\\x00' * 64)",
+        #                 "Ether() /  IP() /  VXLAN(flags=0x00) / Raw('\\x00' * 64)",
+        #                 "Ether() /  IP() / VXLAN(flags=0x99) / Raw('\\x00' * 64)",
+        #                 "Ether() /  IP() /  VXLAN(flags=0x01) / Raw('\\x00' * 64)"
+        #                 })),
+        # END UNSUPPORTED PROPERTIES
+        'vni':  # a 3-byte value
+            ('vxlan vni is 0x112233',
+             frozenset({"Ether() / IP() / VXLAN(vni=0x112233) / Raw('\\x00' * 64)"}),
+
+             frozenset({"Ether() / IP() / VXLAN(vni=0x112234) / Raw('\\x00' * 64)",
+                        "Ether() / IP() / VXLAN(vni=0x123456) / Raw('\\x00' * 64)",
+                        "Ether() / IP() / VXLAN(vni=0xaabbcc) / Raw('\\x00' * 64)",
+                        "Ether() / IP() / VXLAN(vni=0x999999) / Raw('\\x00' * 64)"
+                        })),
+    }
 
 
 class FlowItemVxlan_gpe(PatternFlowItem):
@@ -917,37 +925,46 @@ class FlowItemVxlan_gpe(PatternFlowItem):
     NOT CURRENTLY SUPPORTED BY TESTPMD.
     """
 
-    # THE FOLLOWING PROPERTIES ARE UNSUPPORTED BY TESTPMD AT THE TIME OF WRITING.
-    # They are still tested to future proof this test suite.
     possible_properties = {
-
-        'rsvd0':
-            ('vxlan rsvd0 is 0x000000',
-             frozenset({"Ether() / IP() / VXLAN(reserved0=0) / Raw('\\x00' * 64)"}),
-
-             frozenset({"Ether() /  IP() / VXLAN(reserved0=1) / Raw('\\x00' * 64)",
-                        "Ether() /  IP() /  VXLAN(reserved0=2) / Raw('\\x00' * 64)",
-                        "Ether() /  IP() /  VXLAN(reserved0=3) / Raw('\\x00' * 64)",
-                        "Ether() /  IP()  / VXLAN(reserved0=4) / Raw('\\x00' * 64)"
-                        })),
-        'rsvd1':
-            ('vxlan rsvd1 is 0x00',
-             frozenset({"Ether() /  IP() /  VXLAN(reserved0=0) / Raw('\\x00' * 64)"}),
-
-             frozenset({"Ether() / IP() /  VXLAN(reserved0=1) / Raw('\\x00' * 64)",
-                        "Ether() /  IP() /  VXLAN(reserved0=2) / Raw('\\x00' * 64)",
-                        "Ether() /  IP() / VXLAN(reserved0=3) / Raw('\\x00' * 64)",
-                        "Ether() /  IP() /  VXLAN(reserved0=4) / Raw('\\x00' * 64)"
-                        })),
-        'flags':
-            ('vxlan flags is 0x08',
-             frozenset({"Ether() /  IP() /  VXLAN(flags=0x08) / Raw('\\x00' * 64)"}),
-
-             frozenset({"Ether() / IP() /  VXLAN(flags=0x80) / Raw('\\x00' * 64)",
-                        "Ether() /  IP() /  VXLAN(flags=0x00) / Raw('\\x00' * 64)",
-                        "Ether() /  IP() / VXLAN(flags=0x99) / Raw('\\x00' * 64)",
-                        "Ether() /  IP() /  VXLAN(flags=0x01) / Raw('\\x00' * 64)"
-                        })),
+        # THE FOLLOWING PROPERTIES ARE UNSUPPORTED BY TESTPMD AT THE TIME OF WRITING.
+        # THEY CAN BE ENABLED ONCE TESTPMD SUPPORTS THEM
+        # 'rsvd0':
+        #     ('vxlan rsvd0 is 0x000000',
+        #      frozenset({"Ether() / IP() / VXLAN(reserved0=0) / Raw('\\x00' * 64)"}),
+        #
+        #      frozenset({"Ether() /  IP() / VXLAN(reserved0=1) / Raw('\\x00' * 64)",
+        #                 "Ether() /  IP() /  VXLAN(reserved0=2) / Raw('\\x00' * 64)",
+        #                 "Ether() /  IP() /  VXLAN(reserved0=3) / Raw('\\x00' * 64)",
+        #                 "Ether() /  IP()  / VXLAN(reserved0=4) / Raw('\\x00' * 64)"
+        #                 })),
+        # 'rsvd1':
+        #     ('vxlan rsvd1 is 0x00',
+        #      frozenset({"Ether() /  IP() /  VXLAN(reserved0=0) / Raw('\\x00' * 64)"}),
+        #
+        #      frozenset({"Ether() / IP() /  VXLAN(reserved0=1) / Raw('\\x00' * 64)",
+        #                 "Ether() /  IP() /  VXLAN(reserved0=2) / Raw('\\x00' * 64)",
+        #                 "Ether() /  IP() / VXLAN(reserved0=3) / Raw('\\x00' * 64)",
+        #                 "Ether() /  IP() /  VXLAN(reserved0=4) / Raw('\\x00' * 64)"
+        #                 })),
+        # 'flags':
+        #     ('vxlan flags is 0x08',
+        #      frozenset({"Ether() /  IP() /  VXLAN(flags=0x08) / Raw('\\x00' * 64)"}),
+        #
+        #      frozenset({"Ether() / IP() /  VXLAN(flags=0x80) / Raw('\\x00' * 64)",
+        #                 "Ether() /  IP() /  VXLAN(flags=0x00) / Raw('\\x00' * 64)",
+        #                 "Ether() /  IP() / VXLAN(flags=0x99) / Raw('\\x00' * 64)",
+        #                 "Ether() /  IP() /  VXLAN(flags=0x01) / Raw('\\x00' * 64)"
+        #                 })),
+        # 'protocol':
+        #     ('vxlan protocol is 0x01',
+        #      frozenset({"Ether() / IP() / VXLAN(NextProtocol=0x01) / Raw('\\x00' * 64)"}),
+        #
+        #      frozenset({"Ether() / IP() / VXLAN(NextProtocol=0x01) / Raw('\\x00' * 64)",
+        #                 "Ether() / IP() / VXLAN(NextProtocol=0x11) / Raw('\\x00' * 64)",
+        #                 "Ether() / IP() / VXLAN(NextProtocol=0x22) / Raw('\\x00' * 64)",
+        #                 "Ether() / IP() / VXLAN(NextProtocol=0x33) / Raw('\\x00' * 64)"
+        #                 })),
+        # END UNSUPPORTED PROPERTIES
 
         'vni':  # a 3-byte value
             ('vxlan vni is 0x112233',
@@ -958,15 +975,6 @@ class FlowItemVxlan_gpe(PatternFlowItem):
                         "Ether() / IP() / VXLAN(vni=0xaabbcc) / Raw('\\x00' * 64)",
                         "Ether() / IP() / VXLAN(vni=0x999999) / Raw('\\x00' * 64)"
                         })),
-        'protocol':
-            ('vxlan protocol is 0x01',
-             frozenset({"Ether() / IP() / VXLAN(NextProtocol=0x01) / Raw('\\x00' * 64)"}),
-
-             frozenset({"Ether() / IP() / VXLAN(NextProtocol=0x01) / Raw('\\x00' * 64)",
-                        "Ether() / IP() / VXLAN(NextProtocol=0x11) / Raw('\\x00' * 64)",
-                        "Ether() / IP() / VXLAN(NextProtocol=0x22) / Raw('\\x00' * 64)",
-                        "Ether() / IP() / VXLAN(NextProtocol=0x33) / Raw('\\x00' * 64)"
-                        })),
     }
 
 
-- 
2.30.2


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

* RE: [PATCH v1] framework/flow: Remove items not supported by testpmd
  2022-02-07 16:00 [PATCH v1] framework/flow: Remove items not supported by testpmd ohilyard
@ 2022-02-11  3:18 ` Tu, Lijuan
  0 siblings, 0 replies; 2+ messages in thread
From: Tu, Lijuan @ 2022-02-11  3:18 UTC (permalink / raw)
  To: ohilyard, dts

> -----Original Message-----
> From: ohilyard@iol.unh.edu <ohilyard@iol.unh.edu>
> Sent: 2022年2月8日 0:01
> To: dts@dpdk.org
> Cc: Tu, Lijuan <lijuan.tu@intel.com>; Owen Hilyard <ohilyard@iol.unh.edu>
> Subject: [PATCH v1] framework/flow: Remove items not supported by testpmd
> 
> From: Owen Hilyard <ohilyard@iol.unh.edu>
> 
> Testpmd does not have full support for rte_flow at this point, and the rte_flow
> test suite has not been kept up to date with changes in rte_flow. As such, the
> logic for generating the flows had to be updated to disable some test suites to
> avoid false failures.
> 
> Signed-off-by: Owen Hilyard <ohilyard@iol.unh.edu>

Applied, thanks

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

end of thread, other threads:[~2022-02-11  3:18 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-02-07 16:00 [PATCH v1] framework/flow: Remove items not supported by testpmd ohilyard
2022-02-11  3:18 ` 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).