[orchagent]: Temporarily fix pre-allocated bridge_port_list size (son…

…ic-net#256) * Fix pre-allocated bridge_port_list size * Revert "[copporch]: Add UDLD entry in COPP (sonic-net#252)": now Mellanox implementation does not support this This reverts commit 04e3eed.
madhanmellanox · Jul 18, 2017 · 92533e3 · 92533e3
1 parent 17a6d61
commit 92533e3
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/orchagent/copporch.cpp b/orchagent/copporch.cpp
@@ -42,7 +42,6 @@ map<string, sai_hostif_trap_type_t> trap_id_map = {
     {"igmp_v2_report", SAI_HOSTIF_TRAP_TYPE_IGMP_TYPE_V2_REPORT},
     {"igmp_v3_report", SAI_HOSTIF_TRAP_TYPE_IGMP_TYPE_V3_REPORT},
     {"sample_packet", SAI_HOSTIF_TRAP_TYPE_SAMPLEPACKET},
-    {"udld", SAI_HOSTIF_TRAP_TYPE_UDLD},
     {"switch_cust_range", SAI_HOSTIF_TRAP_TYPE_SWITCH_CUSTOM_RANGE_BASE},
     {"arp_req", SAI_HOSTIF_TRAP_TYPE_ARP_REQUEST},
     {"arp_resp", SAI_HOSTIF_TRAP_TYPE_ARP_RESPONSE},

diff --git a/orchagent/portsorch.cpp b/orchagent/portsorch.cpp
@@ -177,7 +177,9 @@ void PortsOrch::removeDefaultVlanMembers()
 void PortsOrch::removeDefaultBridgePorts()
 {
     /* Get bridge ports in default 1Q bridge */
-    vector<sai_object_id_t> bridge_port_list(m_portCount);
+    // FIXME: Mellanox SAI implementation will response SAI_BRIDGE_ATTR_PORT_LIST
+    // all the front panel ports and CPU port. The CPU bug should be there by SAI spec.
+    vector<sai_object_id_t> bridge_port_list(m_portCount + 1);
 
     sai_attribute_t attr;
     attr.id = SAI_BRIDGE_ATTR_PORT_LIST;