commit 4df0eb4b448dd1d8aa000e2b87be37031998b271
author Andy Bavier <andy@opennetworking.org> Tue Apr 23 15:54:17 2019 -0700
committer Andy Bavier <andy@opennetworking.org> Wed Apr 24 09:20:32 2019 -0700
tree ad4dc850b6d84f13d951df6a3d18ee07855b9870
parent ea42371247ecb00342be46cd69f009602e74bc57

SEBA-607 Add EAPOL trap flow without VLAN_VID match

Change-Id: Ie4f4c76ae9f0fe4d46bae24109b9ed2cfa90e514

voltha/adapters/ponsim_olt/ponsim_olt.py

diff --git a/voltha/adapters/ponsim_olt/ponsim_olt.py b/voltha/adapters/ponsim_olt/ponsim_olt.py
index 4bbfac5..144f1a8 100644
--- a/voltha/adapters/ponsim_olt/ponsim_olt.py
+++ b/voltha/adapters/ponsim_olt/ponsim_olt.py
@@ -759,6 +759,7 @@
         dhcp_upstream_flows = {}
         eapol_flows = {}
         secondary_flows = []
+        eapol_flow_without_vlan = False
 
         for flow in flows:
             classifier_info = {}
@@ -811,12 +812,28 @@
                     self.to_controller(flow)
                     if VLAN_VID in classifier_info:
                         eapol_flows[classifier_info[VLAN_VID]] = flow
+                    else:
+                        eapol_flow_without_vlan = True
 
             self.log.info('out_port', out_port=fd.get_out_port(flow))
 
         flows.extend(self.create_secondary_flows(dhcp_upstream_flows, flows, "DHCP"))
         flows.extend(self.create_secondary_flows(eapol_flows, flows, "EAPOL"))
 
+        # The OLT app is now adding EAPOL flows with VLAN_VID=4091 but Ponsim can't
+        # properly handle this because it uses VLAN_VID to encode the UNI port ID.
+        # Add an EAPOL trap flow with no VLAN_VID match if we see the 4091 match.
+        if 4091 in eapol_flows and not eapol_flow_without_vlan:
+            new_eapol_flow = [
+                fd.mk_flow_stat(
+                    priority=10000,
+                    match_fields=[fd.in_port(1), fd.eth_type(EAP_ETH_TYPE)],
+                    actions=[fd.output(ofp.OFPP_CONTROLLER)]
+                )
+            ]
+            flows.extend(new_eapol_flow)
+            self.log.info('add eapol flow with no VLAN_VID match')
+
         self.log.debug('ctag_map', ctag_map=self.ctag_map)
 
         stub.UpdateFlowTable(FlowTable(