Sai switch handling new reachability group field

Summary:
As titled. Process reachability group list in sai switch.

Also need to sort the entries - at wb GET would return sorted order.

Reviewed By: jasmeetbagga

Differential Revision: D66029979

fbshipit-source-id: 2ae0ca3279c8d4888d5a694ef3821de620c5c88a

fboss/agent/hw/sai/switch/SaiSwitch.cpp

@@ -1338,11 +1338,10 @@ void SaiSwitch::processSwitchSettingsChangeSansDrainedEntryLocked(
   }
 
   {
-    const auto oldVal = oldSwitchSettings->getReachabilityGroupListSize();
-    const auto newVal = newSwitchSettings->getReachabilityGroupListSize();
+    auto oldVal = oldSwitchSettings->getReachabilityGroups();
+    auto newVal = newSwitchSettings->getReachabilityGroups();
     if (oldVal != newVal) {
-      managerTable_->switchManager().setReachabilityGroupList(
-          newVal.has_value() ? newVal.value() : 0);
+      managerTable_->switchManager().setReachabilityGroupList(newVal);
     }
   }
 

fboss/agent/hw/sai/switch/SaiSwitchManager.cpp

@@ -1009,16 +1009,14 @@ void SaiSwitchManager::setLocalCapsuleSwitchIds(
       SaiSwitchTraits::Attributes::MultiStageLocalSwitchIds{values});
 }
 
-void SaiSwitchManager::setReachabilityGroupList(int reachabilityGroupListSize) {
+void SaiSwitchManager::setReachabilityGroupList(
+    const std::vector<int>& reachabilityGroups) {
 #if defined(BRCM_SAI_SDK_DNX_GTE_12_0)
-  if (reachabilityGroupListSize > 0) {
-    std::vector<uint32_t> list;
-    for (int i = 0; i < reachabilityGroupListSize; i++) {
-      list.push_back(i + 1);
-    }
-    switch_->setOptionalAttribute(
-        SaiSwitchTraits::Attributes::ReachabilityGroupList{list});
-  }
+  std::vector<uint32_t> groupList(
+      reachabilityGroups.begin(), reachabilityGroups.end());
+  std::sort(groupList.begin(), groupList.end());
+  switch_->setOptionalAttribute(
+      SaiSwitchTraits::Attributes::ReachabilityGroupList{groupList});
 #endif
 }
 

fboss/agent/hw/sai/switch/SaiSwitchManager.h

@@ -102,7 +102,7 @@ class SaiSwitchManager {
   }
   void setLocalCapsuleSwitchIds(
       const std::map<SwitchID, int>& switchIdToNumCores);
-  void setReachabilityGroupList(int reachabilityGroupListSize);
+  void setReachabilityGroupList(const std::vector<int>& reachabilityGroups);
   void setSramGlobalFreePercentXoffTh(uint8_t sramFreePercentXoffThreshold);
   void setSramGlobalFreePercentXonTh(uint8_t sramFreePercentXonThreshold);
   void setLinkFlowControlCreditTh(uint16_t linkFlowControlThreshold);