Generalize setBridgePortLearningFDB()

Signed-off-by: Qi Luo <[email protected]>

Author: qiluo-msft

orchagent/orchdaemon.cpp

@@ -367,7 +367,7 @@ void OrchDaemon::start()
                     for (auto& pair: gPortsOrch->getAllPorts())
                     {
                         auto& port = pair.second;
-                        gPortsOrch->setBridgePortLearningFDB(port, false);
+                        gPortsOrch->setBridgePortLearningFDB(port, SAI_BRIDGE_PORT_FDB_LEARNING_MODE_DISABLE);
                     }
                     flush();
 

orchagent/portsorch.cpp

@@ -2312,7 +2312,7 @@ bool PortsOrch::addHostIntfs(Port &port, string alias, sai_object_id_t &host_int
     return true;
 }
 
-bool PortsOrch::setBridgePortLearningFDB(Port &port, bool enable)
+bool PortsOrch::setBridgePortLearningFDB(Port &port, sai_bridge_port_fdb_learning_mode_t mode)
 {
     // TODO: how to support 1D bridge?
     if (port.m_type != Port::PHY) return false;
@@ -2322,7 +2322,7 @@ bool PortsOrch::setBridgePortLearningFDB(Port &port, bool enable)
 
     sai_attribute_t bport_attr;
     bport_attr.id = SAI_BRIDGE_PORT_ATTR_FDB_LEARNING_MODE;
-    bport_attr.value.s32 = enable ? SAI_BRIDGE_PORT_FDB_LEARNING_MODE_HW : SAI_BRIDGE_PORT_FDB_LEARNING_MODE_DISABLE;
+    bport_attr.value.s32 = mode;
     auto status = sai_bridge_api->set_bridge_port_attribute(bridge_port_id, &bport_attr);
     if (status != SAI_STATUS_SUCCESS)
     {

orchagent/portsorch.h

@@ -59,7 +59,7 @@ class PortsOrch : public Orch, public Subject
     bool bake() override;
     void cleanPortTable(const vector<string>& keys);
     bool getBridgePort(sai_object_id_t id, Port &port);
-    bool setBridgePortLearningFDB(Port &port, bool enable);
+    bool setBridgePortLearningFDB(Port &port, sai_bridge_port_fdb_learning_mode_t mode);
     bool getPort(string alias, Port &port);
     bool getPort(sai_object_id_t id, Port &port);
     bool getPortByBridgePortId(sai_object_id_t bridge_port_id, Port &port);