[Mellanox|FFB]: Add support for Mellanox fast-fast boot in syncd

syncd/scripts/syncd_init_common.sh

@@ -24,12 +24,20 @@ fi
 # Use temporary view between init and apply
 CMD_ARGS+=" -u"
 
-case "$(cat /proc/cmdline)" in
-  *fast-reboot*)
+BOOT_TYPE="$(cat /proc/cmdline | grep -o 'SONIC_BOOT_TYPE=\S*' | cut -d'=' -f2)"
+
+case "$BOOT_TYPE" in
+  fast-reboot)
      FAST_REBOOT='yes'
     ;;
+  fast-fast)
+    if [ -e /var/warmboot/issu_started ]; then
+        FAST_FAST_REBOOT='yes'
+    fi
+    ;;
   *)
      FAST_REBOOT='no'
+     FAST_FAST_REBOOT='no'
     ;;
 esac
 
@@ -55,6 +63,8 @@ function set_start_type()
         CMD_ARGS+=" -t warm"
     elif [ $FAST_REBOOT == "yes" ]; then
         CMD_ARGS+=" -t fast"
+    elif [ $FAST_FAST_REBOOT == "yes" ]; then
+        CMD_ARGS+=" -t fast-fast"
     fi
 }
 

syncd/syncd.cpp

@@ -2919,7 +2919,7 @@ void printUsage()
 {
     SWSS_LOG_ENTER();
 
-    std::cout << "Usage: syncd [-N] [-U] [-d] [-p profile] [-i interval] [-t [cold|warm|fast]] [-h] [-u] [-S]" << std::endl;
+    std::cout << "Usage: syncd [-N] [-U] [-d] [-p profile] [-i interval] [-t [cold|warm|fast|fast-fast]] [-h] [-u] [-S]" << std::endl;
     std::cout << "    -N --nocounters" << std::endl;
     std::cout << "        Disable counter thread" << std::endl;
     std::cout << "    -d --diag" << std::endl;
@@ -2929,7 +2929,7 @@ void printUsage()
     std::cout << "    -i --countersInterval interval" << std::endl;
     std::cout << "        Provide counter thread interval" << std::endl;
     std::cout << "    -t --startType type" << std::endl;
-    std::cout << "        Specify cold|warm|fast start type" << std::endl;
+    std::cout << "        Specify cold|warm|fast|fast-fast start type" << std::endl;
     std::cout << "    -u --useTempView:" << std::endl;
     std::cout << "        Use temporary view between init and apply" << std::endl;
     std::cout << "    -S --disableExitSleep" << std::endl;
@@ -3028,6 +3028,10 @@ void handleCmdLine(int argc, char **argv)
                 {
                     options.startType = SAI_FAST_BOOT;
                 }
+                else if (std::string(optarg) == "fast-fast")
+                {
+                    options.startType = SAI_FAST_FAST_BOOT;
+                }
                 else
                 {
                     SWSS_LOG_ERROR("unknown start type %s", optarg);
@@ -3206,6 +3210,35 @@ syncd_restart_type_t handleRestartQuery(swss::NotificationConsumer &restartQuery
     return SYNCD_RESTART_TYPE_COLD;
 }
 
+void handleFfbEvent(swss::NotificationConsumer &ffb)
+{
+    SWSS_LOG_ENTER();
+
+    std::string op;
+    std::string data;
+    std::vector<swss::FieldValueTuple> values;
+
+    ffb.pop(op, data, values);
+
+    if ((op == "SET") && (data == "ISSU_END"))
+    {
+        sai_switch_api_t *sai_switch_api = NULL;
+        sai_api_query(SAI_API_SWITCH, (void**)&sai_switch_api);
+
+        sai_attribute_t attr;
+
+        attr.id = SAI_SWITCH_ATTR_FAST_API_ENABLE;
+        attr.value.booldata = false;
+
+        sai_status_t status = sai_switch_api->set_switch_attribute(gSwitchId, &attr);
+
+        if (status != SAI_STATUS_SUCCESS)
+        {
+            SWSS_LOG_ERROR("Failed to set SAI_SWITCH_ATTR_FAST_API_ENABLE=false: %s", sai_serialize_status(status).c_str());
+        }
+    }
+}
+
 bool isVeryFirstRun()
 {
     SWSS_LOG_ENTER();
@@ -3459,6 +3492,7 @@ int syncd_main(int argc, char **argv)
     std::shared_ptr<swss::NotificationConsumer> restartQuery = std::make_shared<swss::NotificationConsumer>(dbAsic.get(), "RESTARTQUERY");
     std::shared_ptr<swss::ConsumerTable> flexCounter = std::make_shared<swss::ConsumerTable>(dbFlexCounter.get(), FLEX_COUNTER_TABLE);
     std::shared_ptr<swss::ConsumerTable> flexCounterGroup = std::make_shared<swss::ConsumerTable>(dbFlexCounter.get(), FLEX_COUNTER_GROUP_TABLE);
+    std::shared_ptr<swss::NotificationConsumer> ffb = std::make_shared<swss::NotificationConsumer>(dbAsic.get(), "MLNX_FFB");
 
     /*
      * At the end we cant use producer consumer concept since if one process
@@ -3504,7 +3538,16 @@ int syncd_main(int argc, char **argv)
         options.startType = SAI_COLD_BOOT;
     }
 
-    gProfileMap[SAI_KEY_BOOT_TYPE] = std::to_string(options.startType);
+    if (options.startType == SAI_FAST_FAST_BOOT)
+    {
+        /*
+         * Mellanox SAI requires to pass SAI_WARM_BOOT as SAI_BOOT_KEY
+         * to start 'fast-fast'
+         */
+        gProfileMap[SAI_KEY_BOOT_TYPE] = std::to_string(SAI_WARM_BOOT);
+    } else {
+        gProfileMap[SAI_KEY_BOOT_TYPE] = std::to_string(options.startType);
+    }
 
     sai_status_t status = sai_api_initialize(0, (sai_service_method_table_t*)&test_services);
 
@@ -3555,6 +3598,7 @@ int syncd_main(int argc, char **argv)
         s.addSelectable(restartQuery.get());
         s.addSelectable(flexCounter.get());
         s.addSelectable(flexCounterGroup.get());
+        s.addSelectable(ffb.get());
 
         SWSS_LOG_NOTICE("starting main loop");
 
@@ -3577,6 +3621,10 @@ int syncd_main(int argc, char **argv)
                 shutdownType = handleRestartQuery(*restartQuery);
                 break;
             }
+            else if (sel == ffb.get())
+            {
+                handleFfbEvent(*ffb);
+            }
             else if (sel == flexCounter.get())
             {
                 processFlexCounterEvent(*(swss::ConsumerTable*)sel);

syncd/syncd.h

@@ -55,6 +55,11 @@ extern "C" {
 #define SAI_COLD_BOOT               0
 #define SAI_WARM_BOOT               1
 #define SAI_FAST_BOOT               2
+/**
+ * A special type of boot used by Mellanox platforms
+ * to start in 'fast-fast' boot mode
+ */
+#define SAI_FAST_FAST_BOOT          3
 
 #ifdef SAITHRIFT
 #define SWITCH_SAI_THRIFT_RPC_SERVER_PORT 9092

tests/aspell.en.pws

@@ -99,6 +99,7 @@ LOOPBACK
 lua
 MCAST
 md
+Mellanox
 metadata
 mlnx
 mpls