SDFAB-193 Fix routing of GTP End Marker packet-outs on fabric-v1model
This change introduces three fixes:
1) We have observed an issue with p4lang/PI and BMv2 where in presence of
multiple metadata fields, the PI implementation for BMv2 provides an
erroneous serialization of the packet-out header, hence affecting the
parsing/forwarding behavior. As a workaround, since we cannot control
the order of fields in the p4runtime.PacketOut message, we modify the
interpreter to only add one field, egress_port or do_forwarding. Both
fields are treated as mutually exclusive by the P4 pipeline, so the
operation is safe. This is against the P4Runtime spec (all fields should
be provided), but supported by BMv2 (unset fields are initialized to
zero).
2) CPU port was not initialized when calling Pipeliner.init()
3) GTP End Marker were being parsed as GTP-U packets with inner IPv4,
causing a parser error (packet too short).
Change-Id: I406870b4a9aa044b5d0b35a56b0bfde4e601a4f6
(cherry picked from commit ffa7fed9eb3b30221cbc5e23b58c9ac322f8b7f9)
diff --git a/pipelines/fabric/impl/src/main/resources/p4c-out/fabric-spgw/bmv2/default/bmv2.json b/pipelines/fabric/impl/src/main/resources/p4c-out/fabric-spgw/bmv2/default/bmv2.json
index 3150ab0a..df6af4c 100644
--- a/pipelines/fabric/impl/src/main/resources/p4c-out/fabric-spgw/bmv2/default/bmv2.json
+++ b/pipelines/fabric/impl/src/main/resources/p4c-out/fabric-spgw/bmv2/default/bmv2.json
@@ -5,12 +5,15 @@
"id" : 0,
"fields" : [
["tmp_0", 1, false],
+ ["tmp_2", 3, false],
+ ["tmp_4", 8, false],
["tmp_1", 16, false],
["tmp_3", 16, false],
["tmp_5", 4, false],
["tmp_6", 16, false],
- ["tmp_2", 32, false],
- ["tmp_4", 32, false],
+ ["tmp_7", 64, false],
+ ["tmp_8", 32, false],
+ ["tmp_9", 32, false],
["userMetadata._ip_eth_type0", 16, false],
["userMetadata._vlan_id1", 12, false],
["userMetadata._vlan_pri2", 3, false],
@@ -42,8 +45,7 @@
["userMetadata._spgw_notify_spgwc28", 1, false],
["userMetadata._spgw_needs_gtpu_encap29", 1, false],
["userMetadata._spgw_needs_gtpu_decap30", 1, false],
- ["userMetadata._spgw_skip_egress_pdr_ctr31", 1, false],
- ["_padding_0", 3, false]
+ ["userMetadata._spgw_skip_egress_pdr_ctr31", 1, false]
]
},
{
@@ -56,9 +58,24 @@
]
},
{
- "name" : "standard_metadata",
+ "name" : "gtpu_t",
"id" : 2,
"fields" : [
+ ["version", 3, false],
+ ["pt", 1, false],
+ ["spare", 1, false],
+ ["ex_flag", 1, false],
+ ["seq_flag", 1, false],
+ ["npdu_flag", 1, false],
+ ["msgtype", 8, false],
+ ["msglen", 16, false],
+ ["teid", 32, false]
+ ]
+ },
+ {
+ "name" : "standard_metadata",
+ "id" : 3,
+ "fields" : [
["ingress_port", 9, false],
["egress_spec", 9, false],
["egress_port", 9, false],
@@ -80,7 +97,7 @@
},
{
"name" : "ethernet_t",
- "id" : 3,
+ "id" : 4,
"fields" : [
["dst_addr", 48, false],
["src_addr", 48, false]
@@ -88,7 +105,7 @@
},
{
"name" : "vlan_tag_t",
- "id" : 4,
+ "id" : 5,
"fields" : [
["eth_type", 16, false],
["pri", 3, false],
@@ -98,14 +115,14 @@
},
{
"name" : "eth_type_t",
- "id" : 5,
+ "id" : 6,
"fields" : [
["value", 16, false]
]
},
{
"name" : "mpls_t",
- "id" : 6,
+ "id" : 7,
"fields" : [
["label", 20, false],
["tc", 3, false],
@@ -115,7 +132,7 @@
},
{
"name" : "ipv4_t",
- "id" : 7,
+ "id" : 8,
"fields" : [
["version", 4, false],
["ihl", 4, false],
@@ -134,7 +151,7 @@
},
{
"name" : "udp_t",
- "id" : 8,
+ "id" : 9,
"fields" : [
["sport", 16, false],
["dport", 16, false],
@@ -143,21 +160,6 @@
]
},
{
- "name" : "gtpu_t",
- "id" : 9,
- "fields" : [
- ["version", 3, false],
- ["pt", 1, false],
- ["spare", 1, false],
- ["ex_flag", 1, false],
- ["seq_flag", 1, false],
- ["npdu_flag", 1, false],
- ["msgtype", 8, false],
- ["msglen", 16, false],
- ["teid", 32, false]
- ]
- },
- {
"name" : "tcp_t",
"id" : 10,
"fields" : [
@@ -204,148 +206,155 @@
"pi_omit" : true
},
{
- "name" : "scalars",
+ "name" : "gtpu_0",
"id" : 1,
+ "header_type" : "gtpu_t",
+ "metadata" : false,
+ "pi_omit" : true
+ },
+ {
+ "name" : "scalars",
+ "id" : 2,
"header_type" : "scalars_0",
"metadata" : true,
"pi_omit" : true
},
{
"name" : "standard_metadata",
- "id" : 2,
+ "id" : 3,
"header_type" : "standard_metadata",
"metadata" : true,
"pi_omit" : true
},
{
"name" : "ethernet",
- "id" : 3,
+ "id" : 4,
"header_type" : "ethernet_t",
"metadata" : false,
"pi_omit" : true
},
{
"name" : "vlan_tag",
- "id" : 4,
- "header_type" : "vlan_tag_t",
- "metadata" : false,
- "pi_omit" : true
- },
- {
- "name" : "inner_vlan_tag",
"id" : 5,
"header_type" : "vlan_tag_t",
"metadata" : false,
"pi_omit" : true
},
{
- "name" : "eth_type",
+ "name" : "inner_vlan_tag",
"id" : 6,
+ "header_type" : "vlan_tag_t",
+ "metadata" : false,
+ "pi_omit" : true
+ },
+ {
+ "name" : "eth_type",
+ "id" : 7,
"header_type" : "eth_type_t",
"metadata" : false,
"pi_omit" : true
},
{
"name" : "mpls",
- "id" : 7,
+ "id" : 8,
"header_type" : "mpls_t",
"metadata" : false,
"pi_omit" : true
},
{
"name" : "gtpu_ipv4",
- "id" : 8,
+ "id" : 9,
"header_type" : "ipv4_t",
"metadata" : false,
"pi_omit" : true
},
{
"name" : "gtpu_udp",
- "id" : 9,
+ "id" : 10,
"header_type" : "udp_t",
"metadata" : false,
"pi_omit" : true
},
{
"name" : "outer_gtpu",
- "id" : 10,
- "header_type" : "gtpu_t",
- "metadata" : false,
- "pi_omit" : true
- },
- {
- "name" : "gtpu",
"id" : 11,
"header_type" : "gtpu_t",
"metadata" : false,
"pi_omit" : true
},
{
- "name" : "inner_ipv4",
+ "name" : "gtpu",
"id" : 12,
+ "header_type" : "gtpu_t",
+ "metadata" : false,
+ "pi_omit" : true
+ },
+ {
+ "name" : "inner_ipv4",
+ "id" : 13,
"header_type" : "ipv4_t",
"metadata" : false,
"pi_omit" : true
},
{
"name" : "inner_udp",
- "id" : 13,
+ "id" : 14,
"header_type" : "udp_t",
"metadata" : false,
"pi_omit" : true
},
{
"name" : "inner_tcp",
- "id" : 14,
+ "id" : 15,
"header_type" : "tcp_t",
"metadata" : false,
"pi_omit" : true
},
{
"name" : "inner_icmp",
- "id" : 15,
+ "id" : 16,
"header_type" : "icmp_t",
"metadata" : false,
"pi_omit" : true
},
{
"name" : "ipv4",
- "id" : 16,
+ "id" : 17,
"header_type" : "ipv4_t",
"metadata" : false,
"pi_omit" : true
},
{
"name" : "tcp",
- "id" : 17,
+ "id" : 18,
"header_type" : "tcp_t",
"metadata" : false,
"pi_omit" : true
},
{
"name" : "udp",
- "id" : 18,
+ "id" : 19,
"header_type" : "udp_t",
"metadata" : false,
"pi_omit" : true
},
{
"name" : "icmp",
- "id" : 19,
+ "id" : 20,
"header_type" : "icmp_t",
"metadata" : false,
"pi_omit" : true
},
{
"name" : "packet_out",
- "id" : 20,
+ "id" : 21,
"header_type" : "packet_out_header_t",
"metadata" : false,
"pi_omit" : true
},
{
"name" : "packet_in",
- "id" : 21,
+ "id" : 22,
"header_type" : "packet_in_header_t",
"metadata" : false,
"pi_omit" : true
@@ -1126,12 +1135,525 @@
}
],
"op" : "set"
+ },
+ {
+ "parameters" : [
+ {
+ "type" : "field",
+ "value" : ["scalars", "tmp_7"]
+ },
+ {
+ "type" : "lookahead",
+ "value" : [0, 64]
+ }
+ ],
+ "op" : "set"
+ },
+ {
+ "parameters" : [
+ {
+ "parameters" : [
+ {
+ "type" : "header",
+ "value" : "gtpu_0"
+ }
+ ],
+ "op" : "add_header"
+ }
+ ],
+ "op" : "primitive"
+ },
+ {
+ "parameters" : [
+ {
+ "type" : "field",
+ "value" : ["gtpu_0", "version"]
+ },
+ {
+ "type" : "expression",
+ "value" : {
+ "type" : "expression",
+ "value" : {
+ "op" : "&",
+ "left" : {
+ "type" : "expression",
+ "value" : {
+ "op" : "&",
+ "left" : {
+ "type" : "expression",
+ "value" : {
+ "op" : ">>",
+ "left" : {
+ "type" : "field",
+ "value" : ["scalars", "tmp_7"]
+ },
+ "right" : {
+ "type" : "hexstr",
+ "value" : "0x3d"
+ }
+ }
+ },
+ "right" : {
+ "type" : "hexstr",
+ "value" : "0xffffffffffffffff"
+ }
+ }
+ },
+ "right" : {
+ "type" : "hexstr",
+ "value" : "0x07"
+ }
+ }
+ }
+ }
+ ],
+ "op" : "set"
+ },
+ {
+ "parameters" : [
+ {
+ "type" : "field",
+ "value" : ["gtpu_0", "pt"]
+ },
+ {
+ "type" : "expression",
+ "value" : {
+ "type" : "expression",
+ "value" : {
+ "op" : "&",
+ "left" : {
+ "type" : "expression",
+ "value" : {
+ "op" : "&",
+ "left" : {
+ "type" : "expression",
+ "value" : {
+ "op" : ">>",
+ "left" : {
+ "type" : "field",
+ "value" : ["scalars", "tmp_7"]
+ },
+ "right" : {
+ "type" : "hexstr",
+ "value" : "0x3c"
+ }
+ }
+ },
+ "right" : {
+ "type" : "hexstr",
+ "value" : "0xffffffffffffffff"
+ }
+ }
+ },
+ "right" : {
+ "type" : "hexstr",
+ "value" : "0x01"
+ }
+ }
+ }
+ }
+ ],
+ "op" : "set"
+ },
+ {
+ "parameters" : [
+ {
+ "type" : "field",
+ "value" : ["gtpu_0", "spare"]
+ },
+ {
+ "type" : "expression",
+ "value" : {
+ "type" : "expression",
+ "value" : {
+ "op" : "&",
+ "left" : {
+ "type" : "expression",
+ "value" : {
+ "op" : "&",
+ "left" : {
+ "type" : "expression",
+ "value" : {
+ "op" : ">>",
+ "left" : {
+ "type" : "field",
+ "value" : ["scalars", "tmp_7"]
+ },
+ "right" : {
+ "type" : "hexstr",
+ "value" : "0x3b"
+ }
+ }
+ },
+ "right" : {
+ "type" : "hexstr",
+ "value" : "0xffffffffffffffff"
+ }
+ }
+ },
+ "right" : {
+ "type" : "hexstr",
+ "value" : "0x01"
+ }
+ }
+ }
+ }
+ ],
+ "op" : "set"
+ },
+ {
+ "parameters" : [
+ {
+ "type" : "field",
+ "value" : ["gtpu_0", "ex_flag"]
+ },
+ {
+ "type" : "expression",
+ "value" : {
+ "type" : "expression",
+ "value" : {
+ "op" : "&",
+ "left" : {
+ "type" : "expression",
+ "value" : {
+ "op" : "&",
+ "left" : {
+ "type" : "expression",
+ "value" : {
+ "op" : ">>",
+ "left" : {
+ "type" : "field",
+ "value" : ["scalars", "tmp_7"]
+ },
+ "right" : {
+ "type" : "hexstr",
+ "value" : "0x3a"
+ }
+ }
+ },
+ "right" : {
+ "type" : "hexstr",
+ "value" : "0xffffffffffffffff"
+ }
+ }
+ },
+ "right" : {
+ "type" : "hexstr",
+ "value" : "0x01"
+ }
+ }
+ }
+ }
+ ],
+ "op" : "set"
+ },
+ {
+ "parameters" : [
+ {
+ "type" : "field",
+ "value" : ["gtpu_0", "seq_flag"]
+ },
+ {
+ "type" : "expression",
+ "value" : {
+ "type" : "expression",
+ "value" : {
+ "op" : "&",
+ "left" : {
+ "type" : "expression",
+ "value" : {
+ "op" : "&",
+ "left" : {
+ "type" : "expression",
+ "value" : {
+ "op" : ">>",
+ "left" : {
+ "type" : "field",
+ "value" : ["scalars", "tmp_7"]
+ },
+ "right" : {
+ "type" : "hexstr",
+ "value" : "0x39"
+ }
+ }
+ },
+ "right" : {
+ "type" : "hexstr",
+ "value" : "0xffffffffffffffff"
+ }
+ }
+ },
+ "right" : {
+ "type" : "hexstr",
+ "value" : "0x01"
+ }
+ }
+ }
+ }
+ ],
+ "op" : "set"
+ },
+ {
+ "parameters" : [
+ {
+ "type" : "field",
+ "value" : ["gtpu_0", "npdu_flag"]
+ },
+ {
+ "type" : "expression",
+ "value" : {
+ "type" : "expression",
+ "value" : {
+ "op" : "&",
+ "left" : {
+ "type" : "expression",
+ "value" : {
+ "op" : "&",
+ "left" : {
+ "type" : "expression",
+ "value" : {
+ "op" : ">>",
+ "left" : {
+ "type" : "field",
+ "value" : ["scalars", "tmp_7"]
+ },
+ "right" : {
+ "type" : "hexstr",
+ "value" : "0x38"
+ }
+ }
+ },
+ "right" : {
+ "type" : "hexstr",
+ "value" : "0xffffffffffffffff"
+ }
+ }
+ },
+ "right" : {
+ "type" : "hexstr",
+ "value" : "0x01"
+ }
+ }
+ }
+ }
+ ],
+ "op" : "set"
+ },
+ {
+ "parameters" : [
+ {
+ "type" : "field",
+ "value" : ["gtpu_0", "msgtype"]
+ },
+ {
+ "type" : "expression",
+ "value" : {
+ "type" : "expression",
+ "value" : {
+ "op" : "&",
+ "left" : {
+ "type" : "expression",
+ "value" : {
+ "op" : "&",
+ "left" : {
+ "type" : "expression",
+ "value" : {
+ "op" : ">>",
+ "left" : {
+ "type" : "field",
+ "value" : ["scalars", "tmp_7"]
+ },
+ "right" : {
+ "type" : "hexstr",
+ "value" : "0x30"
+ }
+ }
+ },
+ "right" : {
+ "type" : "hexstr",
+ "value" : "0xffffffffffffffff"
+ }
+ }
+ },
+ "right" : {
+ "type" : "hexstr",
+ "value" : "0xff"
+ }
+ }
+ }
+ }
+ ],
+ "op" : "set"
+ },
+ {
+ "parameters" : [
+ {
+ "type" : "field",
+ "value" : ["gtpu_0", "msglen"]
+ },
+ {
+ "type" : "expression",
+ "value" : {
+ "type" : "expression",
+ "value" : {
+ "op" : "&",
+ "left" : {
+ "type" : "expression",
+ "value" : {
+ "op" : "&",
+ "left" : {
+ "type" : "expression",
+ "value" : {
+ "op" : ">>",
+ "left" : {
+ "type" : "field",
+ "value" : ["scalars", "tmp_7"]
+ },
+ "right" : {
+ "type" : "hexstr",
+ "value" : "0x20"
+ }
+ }
+ },
+ "right" : {
+ "type" : "hexstr",
+ "value" : "0xffffffffffffffff"
+ }
+ }
+ },
+ "right" : {
+ "type" : "hexstr",
+ "value" : "0xffff"
+ }
+ }
+ }
+ }
+ ],
+ "op" : "set"
+ },
+ {
+ "parameters" : [
+ {
+ "type" : "field",
+ "value" : ["gtpu_0", "teid"]
+ },
+ {
+ "type" : "expression",
+ "value" : {
+ "type" : "expression",
+ "value" : {
+ "op" : "&",
+ "left" : {
+ "type" : "field",
+ "value" : ["scalars", "tmp_7"]
+ },
+ "right" : {
+ "type" : "hexstr",
+ "value" : "0xffffffff"
+ }
+ }
+ }
+ }
+ ],
+ "op" : "set"
+ },
+ {
+ "parameters" : [
+ {
+ "type" : "field",
+ "value" : ["scalars", "tmp_2"]
+ },
+ {
+ "type" : "expression",
+ "value" : {
+ "type" : "expression",
+ "value" : {
+ "op" : "&",
+ "left" : {
+ "type" : "expression",
+ "value" : {
+ "op" : "&",
+ "left" : {
+ "type" : "expression",
+ "value" : {
+ "op" : ">>",
+ "left" : {
+ "type" : "field",
+ "value" : ["scalars", "tmp_7"]
+ },
+ "right" : {
+ "type" : "hexstr",
+ "value" : "0x3d"
+ }
+ }
+ },
+ "right" : {
+ "type" : "hexstr",
+ "value" : "0xffffffffffffffff"
+ }
+ }
+ },
+ "right" : {
+ "type" : "hexstr",
+ "value" : "0x07"
+ }
+ }
+ }
+ }
+ ],
+ "op" : "set"
+ },
+ {
+ "parameters" : [
+ {
+ "type" : "field",
+ "value" : ["scalars", "tmp_4"]
+ },
+ {
+ "type" : "expression",
+ "value" : {
+ "type" : "expression",
+ "value" : {
+ "op" : "&",
+ "left" : {
+ "type" : "expression",
+ "value" : {
+ "op" : "&",
+ "left" : {
+ "type" : "expression",
+ "value" : {
+ "op" : ">>",
+ "left" : {
+ "type" : "field",
+ "value" : ["scalars", "tmp_7"]
+ },
+ "right" : {
+ "type" : "hexstr",
+ "value" : "0x30"
+ }
+ }
+ },
+ "right" : {
+ "type" : "hexstr",
+ "value" : "0xffffffffffffffff"
+ }
+ }
+ },
+ "right" : {
+ "type" : "hexstr",
+ "value" : "0xff"
+ }
+ }
+ }
+ }
+ ],
+ "op" : "set"
}
],
"transitions" : [
{
"type" : "hexstr",
- "value" : "0x0868",
+ "value" : "0x086801ff",
"mask" : null,
"next_state" : "parse_gtpu"
},
@@ -1146,6 +1668,14 @@
{
"type" : "field",
"value" : ["udp", "dport"]
+ },
+ {
+ "type" : "field",
+ "value" : ["scalars", "tmp_2"]
+ },
+ {
+ "type" : "field",
+ "value" : ["scalars", "tmp_4"]
}
]
},
@@ -1289,7 +1819,7 @@
"id" : 0,
"source_info" : {
"filename" : "include/parser.p4",
- "line" : 283,
+ "line" : 285,
"column" : 8,
"source_fragment" : "FabricDeparser"
},
@@ -5715,7 +6245,7 @@
"parameters" : [
{
"type" : "field",
- "value" : ["scalars", "tmp_2"]
+ "value" : ["scalars", "tmp_8"]
},
{
"type" : "expression",
@@ -5751,7 +6281,7 @@
},
{
"type" : "field",
- "value" : ["scalars", "tmp_2"]
+ "value" : ["scalars", "tmp_8"]
}
],
"source_info" : {
@@ -5773,7 +6303,7 @@
"parameters" : [
{
"type" : "field",
- "value" : ["scalars", "tmp_4"]
+ "value" : ["scalars", "tmp_9"]
},
{
"type" : "expression",
@@ -5809,7 +6339,7 @@
},
{
"type" : "field",
- "value" : ["scalars", "tmp_4"]
+ "value" : ["scalars", "tmp_9"]
}
],
"source_info" : {