Add extra check for warm boot discovered RIDs (sonic-net#502)

* Add extra check for warm boot discovered RIDs

* dump asic operations when rid2vid map missmatch

syncd/syncd_applyview.cpp

@@ -7632,6 +7632,8 @@ sai_status_t syncdApplyView()
  checkMap(current.ridToVid, "current R2V", current.vidToRid, "current V2R", temp.ridToVid, "temp R2V", temp.vidToRid, "temp V2R");
  checkMap(temp.ridToVid, "temp R2V", temp.vidToRid, "temp V2R", current.ridToVid, "current R2V", current.vidToRid, "current V2R");
 
+ current.dumpVidToAsicOperatioId();
+
  SWSS_LOG_THROW("wrong number of vid/rid items in map, forgot to translate? R2V: %zu:%zu, V2R: %zu:%zu, FIXME",
  current.ridToVid.size(),
  temp.ridToVid.size(),

syncd/syncd_hard_reinit.cpp

@@ -1223,6 +1223,40 @@ void hardReinit()
  checkAllIds();
 }
 
+void checkWarmBootDiscoveredRids(
+ _In_ std::shared_ptr<SaiSwitch> sw)
+{
+ SWSS_LOG_ENTER();
+
+ /*
+ * After switch was created, rid discovery method was called, and all
+ * discovered RIDs should be present in current RID2VID map in redis
+ * database. If any RID is missing, then ether there is bug in vendor code,
+ * and after warm boot some RID values changed or we have a bug and forgot
+ * to put rid/vid pair to redis.
+ *
+ * Assumption here is that during warm boot ASIC state will not change.
+ */
+
+ auto rid2vid = redisGetRidToVidMap();
+ bool success = true;
+
+ for (auto rid: sw->getDiscoveredRids())
+ {
+ if (rid2vid.find(rid) != rid2vid.end())
+ continue;
+
+ SWSS_LOG_ERROR("RID 0x%lx is missing from current RID2VID map after WARM boot!", rid);
+
+ success = false;
+ }
+
+ if (!success)
+ SWSS_LOG_THROW("FATAL, some discovered RIDs are not present in current RID2VID map, bug");
+
+ SWSS_LOG_NOTICE("all discovered RIDs are present in current RID2VID map");
+}
+
 void performWarmRestart()
 {
  SWSS_LOG_ENTER();
@@ -1329,5 +1363,7 @@ void performWarmRestart()
 
  gSwitchId = g_switch_rid;
 
+ checkWarmBootDiscoveredRids(sw);
+
  startDiagShell();
 }