fix wrong API type , add cmdline parse option , remove unuseful api c…

…all (#499) fix wrong API type in internal_api_wait_for_response() add cmdline parse option "s" , otherwise syncd cannot be executed via -s option, system cannot be up with syncd
sonic-net · Aug 1, 2019 · 70490b4 · 70490b4
1 parent 54725e7
commit 70490b4
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 6 deletions.
diff --git a/lib/src/sai_redis_generic_remove.cpp b/lib/src/sai_redis_generic_remove.cpp
@@ -134,7 +134,7 @@ sai_status_t internal_redis_bulk_generic_remove(
         g_asicState->set(key, entries, "bulkremove");
     }
 
-    return internal_api_wait_for_response(SAI_COMMON_API_CREATE);
+    return internal_api_wait_for_response(SAI_COMMON_API_REMOVE);
 }
 
 

diff --git a/lib/src/sai_redis_generic_set.cpp b/lib/src/sai_redis_generic_set.cpp
@@ -187,7 +187,7 @@ sai_status_t internal_redis_bulk_generic_set(
         g_asicState->set(key, entries, "bulkset");
     }
 
-    return internal_api_wait_for_response(SAI_COMMON_API_CREATE);
+    return internal_api_wait_for_response(SAI_COMMON_API_SET);
 }
 
 

diff --git a/lib/src/sai_redis_interfacequery.cpp b/lib/src/sai_redis_interfacequery.cpp
@@ -124,8 +124,6 @@ sai_status_t sai_api_initialize(
     g_redisNotifications = std::make_shared<swss::NotificationConsumer>(g_dbNtf.get(), "NOTIFICATIONS");
     g_redisClient        = std::make_shared<swss::RedisClient>(g_db.get());
 
-    g_asicState->setBuffered(false); // in sync mode, always false
-
     clear_local_state();
 
     g_asicInitViewMode = false;

diff --git a/syncd/syncd.cpp b/syncd/syncd.cpp
@@ -3302,9 +3302,9 @@ void handleCmdLine(int argc, char **argv)
 
 #ifdef SAITHRIFT
     options.run_rpc_server = false;
-    const char* const optstring = "dNUCt:p:i:rm:huS";
+    const char* const optstring = "dNUCt:p:i:rm:huSs";
 #else
-    const char* const optstring = "dNUCt:p:i:huS";
+    const char* const optstring = "dNUCt:p:i:huSs";
 #endif // SAITHRIFT
 
     while(true)