commit 8a715f84c597630dabf8d0f437d9d951d696d4b7
author Carmelo Cascone <carmelo@opennetworking.org> Mon Aug 20 23:16:27 2018 -0700
committer Carmelo Cascone <carmelo@opennetworking.org> Wed Aug 22 18:48:35 2018 +0000
tree 8d0c79d6f8a3a9b2879d0ffc695b6c7e899e3402
parent 9587dc99ca9f5c5397d32d15c5fa1df4f74c5e52

Add default VLAN 4094 tag to all untagged traffic in fabric.p4

Also fix nop action in ingres_port_vlan table not invoking direct
counter.

This allows processing LLDP packets before any SR-related flow rule is
pushed

Change-Id: Ic6c33ff0c74e643d9e7c853e029bff924ba6f2ac
(cherry picked from commit 0433a9d002c6a9533ffd55de709aba5a6ca26f04)

pipelines/fabric/src/main/resources/include/control/filtering.p4

diff --git a/pipelines/fabric/src/main/resources/include/control/filtering.p4 b/pipelines/fabric/src/main/resources/include/control/filtering.p4
index c5030d7..dcabea7 100644
--- a/pipelines/fabric/src/main/resources/include/control/filtering.p4
+++ b/pipelines/fabric/src/main/resources/include/control/filtering.p4
@@ -58,6 +58,11 @@
         ingress_port_vlan_counter.count();
     }
 
+    action nop_ingress_port_vlan() {
+        nop();
+        ingress_port_vlan_counter.count();
+    }
+
     table ingress_port_vlan {
         key = {
             standard_metadata.ingress_port: exact;
@@ -68,11 +73,11 @@
         actions = {
             push_internal_vlan;
             set_vlan;
-            @defaultonly nop;
             drop;
+            nop_ingress_port_vlan();
         }
 
-        const default_action = nop();
+        const default_action = push_internal_vlan(DEFAULT_VLAN_ID);
         counters = ingress_port_vlan_counter;
     }
 
@@ -111,7 +116,12 @@
     }
 
     apply {
-        ingress_port_vlan.apply();
-        fwd_classifier.apply();
+        if (ingress_port_vlan.apply().hit) {
+            fwd_classifier.apply();
+        } else {
+            // Packet from unconfigured port. Skip forwarding processing,
+            // except for ACL table in case we want to punt to cpu.
+            fabric_metadata.fwd_type = FWD_UNKNOWN;
+        }
     }
 }