NoAction in ACL should be mapped to NOP
Change-Id: Ia3c9038eb726379ed2f95635c28ddf5794e646de
diff --git a/pipelines/fabric/src/main/resources/p4c-out/fabric-spgw/bmv2/default/bmv2.json b/pipelines/fabric/src/main/resources/p4c-out/fabric-spgw/bmv2/default/bmv2.json
index c0adbca..285ac8c 100644
--- a/pipelines/fabric/src/main/resources/p4c-out/fabric-spgw/bmv2/default/bmv2.json
+++ b/pipelines/fabric/src/main/resources/p4c-out/fabric-spgw/bmv2/default/bmv2.json
@@ -928,7 +928,7 @@
"id" : 0,
"source_info" : {
"filename" : "include/parser.p4",
- "line" : 222,
+ "line" : 223,
"column" : 8,
"source_fragment" : "FabricDeparser"
},
@@ -1102,7 +1102,7 @@
"id" : 1,
"source_info" : {
"filename" : "include/spgw.p4",
- "line" : 238,
+ "line" : 237,
"column" : 8,
"source_fragment" : "update_checksum(gtpu_ipv4.isValid(), ..."
},
@@ -1283,14 +1283,8 @@
"primitives" : []
},
{
- "name" : "nop",
- "id" : 10,
- "runtime_data" : [],
- "primitives" : []
- },
- {
"name" : "drop_now",
- "id" : 11,
+ "id" : 10,
"runtime_data" : [],
"primitives" : [
{
@@ -1317,7 +1311,7 @@
},
{
"name" : "FabricIngress.spgw_ingress.gtpu_decap",
- "id" : 12,
+ "id" : 11,
"runtime_data" : [],
"primitives" : [
{
@@ -1369,7 +1363,7 @@
},
{
"name" : "FabricIngress.spgw_ingress.set_dl_sess_info",
- "id" : 13,
+ "id" : 12,
"runtime_data" : [
{
"name" : "teid",
@@ -1446,7 +1440,7 @@
},
{
"name" : "FabricIngress.filtering.drop",
- "id" : 14,
+ "id" : 13,
"runtime_data" : [],
"primitives" : [
{
@@ -1463,7 +1457,7 @@
},
{
"name" : "FabricIngress.filtering.set_vlan",
- "id" : 15,
+ "id" : 14,
"runtime_data" : [
{
"name" : "new_vlan_id",
@@ -1494,7 +1488,7 @@
},
{
"name" : "FabricIngress.filtering.push_internal_vlan",
- "id" : 16,
+ "id" : 15,
"runtime_data" : [
{
"name" : "new_vlan_id",
@@ -1645,7 +1639,7 @@
},
{
"name" : "FabricIngress.filtering.set_forwarding_type",
- "id" : 17,
+ "id" : 16,
"runtime_data" : [
{
"name" : "fwd_type",
@@ -1676,7 +1670,7 @@
},
{
"name" : "FabricIngress.forwarding.set_next_id_bridging",
- "id" : 18,
+ "id" : 17,
"runtime_data" : [
{
"name" : "next_id",
@@ -1707,7 +1701,7 @@
},
{
"name" : "FabricIngress.forwarding.pop_mpls_and_next",
- "id" : 19,
+ "id" : 18,
"runtime_data" : [
{
"name" : "next_id",
@@ -1753,7 +1747,7 @@
},
{
"name" : "FabricIngress.forwarding.set_next_id_unicast_v4",
- "id" : 20,
+ "id" : 19,
"runtime_data" : [
{
"name" : "next_id",
@@ -1784,7 +1778,7 @@
},
{
"name" : "FabricIngress.forwarding.set_next_id_acl",
- "id" : 21,
+ "id" : 20,
"runtime_data" : [
{
"name" : "next_id",
@@ -1815,7 +1809,7 @@
},
{
"name" : "FabricIngress.forwarding.punt_to_cpu",
- "id" : 22,
+ "id" : 21,
"runtime_data" : [],
"primitives" : [
{
@@ -1851,7 +1845,7 @@
},
{
"name" : "FabricIngress.forwarding.clone_to_cpu",
- "id" : 23,
+ "id" : 22,
"runtime_data" : [],
"primitives" : [
{
@@ -1887,7 +1881,7 @@
},
{
"name" : "FabricIngress.forwarding.drop",
- "id" : 24,
+ "id" : 23,
"runtime_data" : [],
"primitives" : [
{
@@ -1903,6 +1897,12 @@
]
},
{
+ "name" : "FabricIngress.forwarding.nop_acl",
+ "id" : 24,
+ "runtime_data" : [],
+ "primitives" : []
+ },
+ {
"name" : "FabricIngress.next.set_vlan",
"id" : 25,
"runtime_data" : [
@@ -3565,7 +3565,7 @@
],
"source_info" : {
"filename" : "include/spgw.p4",
- "line" : 154,
+ "line" : 153,
"column" : 12,
"source_fragment" : "return"
}
@@ -3621,7 +3621,7 @@
],
"source_info" : {
"filename" : "include/spgw.p4",
- "line" : 171,
+ "line" : 170,
"column" : 8,
"source_fragment" : "spgw_meta.ipv4_len = ipv4.total_len"
}
@@ -4140,7 +4140,7 @@
],
"source_info" : {
"filename" : "include/spgw.p4",
- "line" : 186,
+ "line" : 185,
"column" : 8,
"source_fragment" : "gtpu_ipv4.setValid()"
}
@@ -4159,7 +4159,7 @@
],
"source_info" : {
"filename" : "include/spgw.p4",
- "line" : 187,
+ "line" : 186,
"column" : 8,
"source_fragment" : "gtpu_ipv4.version = 4"
}
@@ -4197,7 +4197,7 @@
],
"source_info" : {
"filename" : "include/spgw.p4",
- "line" : 189,
+ "line" : 188,
"column" : 8,
"source_fragment" : "gtpu_ipv4.dscp = 0"
}
@@ -4216,7 +4216,7 @@
],
"source_info" : {
"filename" : "include/spgw.p4",
- "line" : 190,
+ "line" : 189,
"column" : 8,
"source_fragment" : "gtpu_ipv4.ecn = 0"
}
@@ -4258,7 +4258,7 @@
],
"source_info" : {
"filename" : "include/spgw.p4",
- "line" : 191,
+ "line" : 190,
"column" : 8,
"source_fragment" : "gtpu_ipv4.total_len = ipv4.total_len ..."
}
@@ -4277,7 +4277,7 @@
],
"source_info" : {
"filename" : "include/spgw.p4",
- "line" : 193,
+ "line" : 192,
"column" : 8,
"source_fragment" : "gtpu_ipv4.identification = 0x1513"
}
@@ -4296,7 +4296,7 @@
],
"source_info" : {
"filename" : "include/spgw.p4",
- "line" : 194,
+ "line" : 193,
"column" : 8,
"source_fragment" : "gtpu_ipv4.flags = 0"
}
@@ -4315,7 +4315,7 @@
],
"source_info" : {
"filename" : "include/spgw.p4",
- "line" : 195,
+ "line" : 194,
"column" : 8,
"source_fragment" : "gtpu_ipv4.frag_offset = 0"
}
@@ -4372,7 +4372,7 @@
],
"source_info" : {
"filename" : "include/spgw.p4",
- "line" : 198,
+ "line" : 197,
"column" : 8,
"source_fragment" : "gtpu_ipv4.dst_addr = spgw_meta.s1u_enb_addr"
}
@@ -4391,7 +4391,7 @@
],
"source_info" : {
"filename" : "include/spgw.p4",
- "line" : 199,
+ "line" : 198,
"column" : 8,
"source_fragment" : "gtpu_ipv4.src_addr = spgw_meta.s1u_sgw_addr"
}
@@ -4410,7 +4410,7 @@
],
"source_info" : {
"filename" : "include/spgw.p4",
- "line" : 200,
+ "line" : 199,
"column" : 8,
"source_fragment" : "gtpu_ipv4.hdr_checksum = 0"
}
@@ -4425,7 +4425,7 @@
],
"source_info" : {
"filename" : "include/spgw.p4",
- "line" : 202,
+ "line" : 201,
"column" : 8,
"source_fragment" : "gtpu_udp.setValid()"
}
@@ -4444,7 +4444,7 @@
],
"source_info" : {
"filename" : "include/spgw.p4",
- "line" : 203,
+ "line" : 202,
"column" : 8,
"source_fragment" : "gtpu_udp.src_port = 2152"
}
@@ -4463,7 +4463,7 @@
],
"source_info" : {
"filename" : "include/spgw.p4",
- "line" : 204,
+ "line" : 203,
"column" : 8,
"source_fragment" : "gtpu_udp.dst_port = 2152"
}
@@ -4505,7 +4505,7 @@
],
"source_info" : {
"filename" : "include/spgw.p4",
- "line" : 205,
+ "line" : 204,
"column" : 8,
"source_fragment" : "gtpu_udp.len = spgw_meta.ipv4_len ..."
}
@@ -4524,7 +4524,7 @@
],
"source_info" : {
"filename" : "include/spgw.p4",
- "line" : 207,
+ "line" : 206,
"column" : 8,
"source_fragment" : "gtpu_udp.checksum = 0"
}
@@ -4539,7 +4539,7 @@
],
"source_info" : {
"filename" : "include/spgw.p4",
- "line" : 209,
+ "line" : 208,
"column" : 8,
"source_fragment" : "gtpu.setValid()"
}
@@ -4558,7 +4558,7 @@
],
"source_info" : {
"filename" : "include/spgw.p4",
- "line" : 210,
+ "line" : 209,
"column" : 8,
"source_fragment" : "gtpu.version = 0x01"
}
@@ -4577,7 +4577,7 @@
],
"source_info" : {
"filename" : "include/spgw.p4",
- "line" : 211,
+ "line" : 210,
"column" : 8,
"source_fragment" : "gtpu.pt = 0x01"
}
@@ -4596,7 +4596,7 @@
],
"source_info" : {
"filename" : "include/spgw.p4",
- "line" : 212,
+ "line" : 211,
"column" : 8,
"source_fragment" : "gtpu.spare = 0"
}
@@ -4615,7 +4615,7 @@
],
"source_info" : {
"filename" : "include/spgw.p4",
- "line" : 213,
+ "line" : 212,
"column" : 8,
"source_fragment" : "gtpu.ex_flag = 0"
}
@@ -4634,7 +4634,7 @@
],
"source_info" : {
"filename" : "include/spgw.p4",
- "line" : 214,
+ "line" : 213,
"column" : 8,
"source_fragment" : "gtpu.seq_flag = 0"
}
@@ -4653,7 +4653,7 @@
],
"source_info" : {
"filename" : "include/spgw.p4",
- "line" : 215,
+ "line" : 214,
"column" : 8,
"source_fragment" : "gtpu.npdu_flag = 0"
}
@@ -4672,7 +4672,7 @@
],
"source_info" : {
"filename" : "include/spgw.p4",
- "line" : 216,
+ "line" : 215,
"column" : 8,
"source_fragment" : "gtpu.msgtype = 0xff"
}
@@ -4691,7 +4691,7 @@
],
"source_info" : {
"filename" : "include/spgw.p4",
- "line" : 217,
+ "line" : 216,
"column" : 8,
"source_fragment" : "gtpu.msglen = spgw_meta.ipv4_len"
}
@@ -4710,7 +4710,7 @@
],
"source_info" : {
"filename" : "include/spgw.p4",
- "line" : 218,
+ "line" : 217,
"column" : 8,
"source_fragment" : "gtpu.teid = spgw_meta.teid"
}
@@ -5107,14 +5107,14 @@
"with_counters" : false,
"support_timeout" : false,
"direct_meters" : null,
- "action_ids" : [11],
+ "action_ids" : [10],
"actions" : ["drop_now"],
"base_default_next" : "tbl_act_8",
"next_tables" : {
"drop_now" : "tbl_act_8"
},
"default_entry" : {
- "action_id" : 11,
+ "action_id" : 10,
"action_const" : true,
"action_data" : [],
"action_entry_const" : true
@@ -5153,14 +5153,14 @@
"with_counters" : false,
"support_timeout" : false,
"direct_meters" : null,
- "action_ids" : [12],
+ "action_ids" : [11],
"actions" : ["FabricIngress.spgw_ingress.gtpu_decap"],
"base_default_next" : "node_27",
"next_tables" : {
"FabricIngress.spgw_ingress.gtpu_decap" : "node_27"
},
"default_entry" : {
- "action_id" : 12,
+ "action_id" : 11,
"action_const" : true,
"action_data" : [],
"action_entry_const" : true
@@ -5189,7 +5189,7 @@
"with_counters" : true,
"support_timeout" : false,
"direct_meters" : null,
- "action_ids" : [13, 0],
+ "action_ids" : [12, 0],
"actions" : ["FabricIngress.spgw_ingress.set_dl_sess_info", "NoAction"],
"base_default_next" : null,
"next_tables" : {
@@ -5353,7 +5353,7 @@
"with_counters" : true,
"support_timeout" : false,
"direct_meters" : null,
- "action_ids" : [16, 15, 8, 14],
+ "action_ids" : [15, 14, 8, 13],
"actions" : ["FabricIngress.filtering.push_internal_vlan", "FabricIngress.filtering.set_vlan", "nop", "FabricIngress.filtering.drop"],
"base_default_next" : "FabricIngress.filtering.fwd_classifier",
"next_tables" : {
@@ -5404,14 +5404,14 @@
"with_counters" : true,
"support_timeout" : false,
"direct_meters" : null,
- "action_ids" : [17],
+ "action_ids" : [16],
"actions" : ["FabricIngress.filtering.set_forwarding_type"],
"base_default_next" : "node_31",
"next_tables" : {
"FabricIngress.filtering.set_forwarding_type" : "node_31"
},
"default_entry" : {
- "action_id" : 17,
+ "action_id" : 16,
"action_const" : true,
"action_data" : ["0x0"],
"action_entry_const" : true
@@ -5446,7 +5446,7 @@
"with_counters" : true,
"support_timeout" : false,
"direct_meters" : null,
- "action_ids" : [18, 2],
+ "action_ids" : [17, 2],
"actions" : ["FabricIngress.forwarding.set_next_id_bridging", "NoAction"],
"base_default_next" : "FabricIngress.forwarding.acl",
"next_tables" : {
@@ -5483,7 +5483,7 @@
"with_counters" : true,
"support_timeout" : false,
"direct_meters" : null,
- "action_ids" : [19, 3],
+ "action_ids" : [18, 3],
"actions" : ["FabricIngress.forwarding.pop_mpls_and_next", "NoAction"],
"base_default_next" : "tbl_act_14",
"next_tables" : {
@@ -5543,7 +5543,7 @@
"with_counters" : true,
"support_timeout" : false,
"direct_meters" : null,
- "action_ids" : [20, 4],
+ "action_ids" : [19, 4],
"actions" : ["FabricIngress.forwarding.set_next_id_unicast_v4", "NoAction"],
"base_default_next" : "FabricIngress.forwarding.acl",
"next_tables" : {
@@ -5562,7 +5562,7 @@
"id" : 25,
"source_info" : {
"filename" : "include/control/forwarding.p4",
- "line" : 127,
+ "line" : 131,
"column" : 10,
"source_fragment" : "acl"
},
@@ -5646,18 +5646,18 @@
"with_counters" : true,
"support_timeout" : false,
"direct_meters" : null,
- "action_ids" : [21, 22, 23, 24, 9],
- "actions" : ["FabricIngress.forwarding.set_next_id_acl", "FabricIngress.forwarding.punt_to_cpu", "FabricIngress.forwarding.clone_to_cpu", "FabricIngress.forwarding.drop", "nop"],
+ "action_ids" : [20, 21, 22, 23, 24],
+ "actions" : ["FabricIngress.forwarding.set_next_id_acl", "FabricIngress.forwarding.punt_to_cpu", "FabricIngress.forwarding.clone_to_cpu", "FabricIngress.forwarding.drop", "FabricIngress.forwarding.nop_acl"],
"base_default_next" : "tbl_act_15",
"next_tables" : {
"FabricIngress.forwarding.set_next_id_acl" : "tbl_act_15",
"FabricIngress.forwarding.punt_to_cpu" : "tbl_act_15",
"FabricIngress.forwarding.clone_to_cpu" : "tbl_act_15",
"FabricIngress.forwarding.drop" : "tbl_act_15",
- "nop" : "tbl_act_15"
+ "FabricIngress.forwarding.nop_acl" : "tbl_act_15"
},
"default_entry" : {
- "action_id" : 9,
+ "action_id" : 24,
"action_const" : true,
"action_data" : [],
"action_entry_const" : true
@@ -5709,7 +5709,7 @@
"with_counters" : true,
"support_timeout" : false,
"direct_meters" : null,
- "action_ids" : [25, 10],
+ "action_ids" : [25, 9],
"actions" : ["FabricIngress.next.set_vlan", "nop"],
"base_default_next" : "FabricIngress.next.simple",
"next_tables" : {
@@ -5717,7 +5717,7 @@
"nop" : "FabricIngress.next.simple"
},
"default_entry" : {
- "action_id" : 10,
+ "action_id" : 9,
"action_const" : false,
"action_data" : [],
"action_entry_const" : false
@@ -6292,7 +6292,7 @@
"id" : 8,
"source_info" : {
"filename" : "include/control/forwarding.p4",
- "line" : 231,
+ "line" : 235,
"column" : 11,
"source_fragment" : "fabric_metadata.fwd_type == FWD_BRIDGING"
},
@@ -6318,7 +6318,7 @@
"id" : 9,
"source_info" : {
"filename" : "include/control/forwarding.p4",
- "line" : 232,
+ "line" : 236,
"column" : 17,
"source_fragment" : "fabric_metadata.fwd_type == FWD_MPLS"
},
@@ -6344,7 +6344,7 @@
"id" : 10,
"source_info" : {
"filename" : "include/control/forwarding.p4",
- "line" : 246,
+ "line" : 250,
"column" : 17,
"source_fragment" : "fabric_metadata.fwd_type == FWD_IPV4_UNICAST"
},
@@ -6952,7 +6952,7 @@
"id" : 23,
"source_info" : {
"filename" : "include/spgw.p4",
- "line" : 222,
+ "line" : 221,
"column" : 12,
"source_fragment" : "spgw_meta.direction == SPGW_DIR_DOWNLINK"
},