SAI 1.3 changes

.gitmodules

@@ -2,4 +2,4 @@
 	path = SAI
 	url = https://github.com/opencomputeproject/SAI.git
 	ignore = dirty
-	branch = v1.2
+	branch = v1.3

lib/src/sai_redis_bridge.cpp

@@ -38,6 +38,34 @@ sai_status_t redis_get_bridge_port_stats(
     return SAI_STATUS_NOT_IMPLEMENTED;
 }
 
+sai_status_t redis_get_bridge_port_stats_ext(
+        _In_ sai_object_id_t bridge_port_id,
+        _In_ uint32_t number_of_counters,
+        _In_ const sai_bridge_port_stat_t *counter_ids,
+        _In_ sai_stats_mode_t mode,
+        _Out_ uint64_t *counters)
+{
+    MUTEX();
+
+    SWSS_LOG_ENTER();
+
+    return SAI_STATUS_NOT_IMPLEMENTED;
+}
+
+sai_status_t redis_get_bridge_stats_ext(
+        _In_ sai_object_id_t bridge_id,
+        _In_ uint32_t number_of_counters,
+        _In_ const sai_bridge_stat_t *counter_ids,
+        _In_ sai_stats_mode_t mode,
+        _Out_ uint64_t *counters)
+{
+    MUTEX();
+
+    SWSS_LOG_ENTER();
+
+    return SAI_STATUS_NOT_IMPLEMENTED;
+}
+
 sai_status_t redis_clear_bridge_port_stats(
         _In_ sai_object_id_t bridge_port_id,
         _In_ uint32_t number_of_counters,
@@ -58,10 +86,12 @@ const sai_bridge_api_t redis_bridge_api =
     REDIS_GENERIC_QUAD_API(bridge)
 
     redis_get_bridge_stats,
+    redis_get_bridge_stats_ext,
     redis_clear_bridge_stats,
 
     REDIS_GENERIC_QUAD_API(bridge_port)
 
     redis_get_bridge_port_stats,
+    redis_get_bridge_port_stats_ext,
     redis_clear_bridge_port_stats,
 };

lib/src/sai_redis_buffer.cpp

@@ -12,6 +12,20 @@ sai_status_t redis_clear_ingress_priority_group_stats(
     return SAI_STATUS_NOT_IMPLEMENTED;
 }
 
+sai_status_t redis_get_ingress_priority_group_stats_ext(
+        _In_ sai_object_id_t ingress_priority_group_id,
+        _In_ uint32_t number_of_counters,
+        _In_ const sai_ingress_priority_group_stat_t *counter_ids,
+        _In_ sai_stats_mode_t mode,
+        _Out_ uint64_t *counters)
+{
+    MUTEX();
+
+    SWSS_LOG_ENTER();
+
+    return SAI_STATUS_NOT_IMPLEMENTED;
+}
+
 sai_status_t redis_get_buffer_pool_stats(
         _In_ sai_object_id_t pool_id,
         _In_ uint32_t number_of_counters,
@@ -25,6 +39,20 @@ sai_status_t redis_get_buffer_pool_stats(
     return SAI_STATUS_NOT_IMPLEMENTED;
 }
 
+sai_status_t redis_get_buffer_pool_stats_ext(
+        _In_ sai_object_id_t buffer_pool_id,
+        _In_ uint32_t number_of_counters,
+        _In_ const sai_buffer_pool_stat_t *counter_ids,
+        _In_ sai_stats_mode_t mode,
+        _Out_ uint64_t *counters)
+{
+    MUTEX();
+
+    SWSS_LOG_ENTER();
+
+    return SAI_STATUS_NOT_IMPLEMENTED;
+}
+
 sai_status_t redis_clear_buffer_pool_stats(
         _In_ sai_object_id_t pool_id,
         _In_ uint32_t number_of_counters,
@@ -48,11 +76,13 @@ const sai_buffer_api_t redis_buffer_api = {
     REDIS_GENERIC_QUAD_API(buffer_pool)
 
     redis_get_buffer_pool_stats,
+    redis_get_buffer_pool_stats_ext,
     redis_clear_buffer_pool_stats,
 
     REDIS_GENERIC_QUAD_API(ingress_priority_group)
 
     redis_get_ingress_priority_group_stats,
+    redis_get_ingress_priority_group_stats_ext,
     redis_clear_ingress_priority_group_stats,
 
     REDIS_GENERIC_QUAD_API(buffer_profile)

lib/src/sai_redis_hostintf.cpp

@@ -1,9 +1,9 @@
 #include "sai_redis.h"
 
 sai_status_t redis_recv_hostif_packet(
-        _In_ sai_object_id_t hif_id,
-        _Out_ void *buffer,
+        _In_ sai_object_id_t hostif_id,
         _Inout_ sai_size_t *buffer_size,
+        _Out_ void *buffer,
         _Inout_ uint32_t *attr_count,
         _Out_ sai_attribute_t *attr_list)
 {
@@ -15,11 +15,11 @@ sai_status_t redis_recv_hostif_packet(
 }
 
 sai_status_t redis_send_hostif_packet(
-        _In_ sai_object_id_t hif_id,
-        _In_ void *buffer,
+        _In_ sai_object_id_t hostif_id,
         _In_ sai_size_t buffer_size,
+        _In_ const void *buffer,
         _In_ uint32_t attr_count,
-        _In_ sai_attribute_t *attr_list)
+        _In_ const sai_attribute_t *attr_list)
 {
     MUTEX();
 

lib/src/sai_redis_policer.cpp

@@ -13,6 +13,20 @@ sai_status_t redis_get_policer_stats(
     return SAI_STATUS_NOT_IMPLEMENTED;
 }
 
+sai_status_t redis_sai_get_policer_stats_ext(
+        _In_ sai_object_id_t policer_id,
+        _In_ uint32_t number_of_counters,
+        _In_ const sai_policer_stat_t *counter_ids,
+        _In_ sai_stats_mode_t mode,
+        _Out_ uint64_t *counters)
+{
+    MUTEX();
+
+    SWSS_LOG_ENTER();
+
+    return SAI_STATUS_NOT_IMPLEMENTED;
+}
+
 sai_status_t redis_clear_policer_stats(
         _In_ sai_object_id_t policer_id,
         _In_ uint32_t number_of_counters,
@@ -32,5 +46,6 @@ const sai_policer_api_t redis_policer_api = {
     REDIS_GENERIC_QUAD_API(policer)
 
     redis_get_policer_stats,
+    redis_sai_get_policer_stats_ext,
     redis_clear_policer_stats,
 };

lib/src/sai_redis_port.cpp

@@ -12,6 +12,20 @@ sai_status_t redis_clear_port_stats(
     return SAI_STATUS_NOT_IMPLEMENTED;
 }
 
+sai_status_t redis_get_port_stats_ext(
+        _In_ sai_object_id_t port_id,
+        _In_ uint32_t number_of_counters,
+        _In_ const sai_port_stat_t *counter_ids,
+        _In_ sai_stats_mode_t mode,
+        _Out_ uint64_t *counters)
+{
+    MUTEX();
+
+    SWSS_LOG_ENTER();
+
+    return SAI_STATUS_NOT_IMPLEMENTED;
+}
+
 sai_status_t redis_clear_port_all_stats(
         _In_ sai_object_id_t port_id)
 {
@@ -22,6 +36,20 @@ sai_status_t redis_clear_port_all_stats(
     return SAI_STATUS_NOT_IMPLEMENTED;
 }
 
+sai_status_t redis_get_port_pool_stats_ext(
+        _In_ sai_object_id_t port_pool_id,
+        _In_ uint32_t number_of_counters,
+        _In_ const sai_port_pool_stat_t *counter_ids,
+        _In_ sai_stats_mode_t mode,
+        _Out_ uint64_t *counters)
+{
+    MUTEX();
+
+    SWSS_LOG_ENTER();
+
+    return SAI_STATUS_NOT_IMPLEMENTED;
+}
+
 sai_status_t redis_clear_port_pool_stats(
         _In_ sai_object_id_t port_pool_id,
         _In_ uint32_t number_of_counters,
@@ -46,11 +74,13 @@ const sai_port_api_t redis_port_api = {
     REDIS_GENERIC_QUAD_API(port)
 
     redis_get_port_stats,
+    redis_get_port_stats_ext,
     redis_clear_port_stats,
     redis_clear_port_all_stats,
 
     REDIS_GENERIC_QUAD_API(port_pool)
 
     redis_get_port_pool_stats,
+    redis_get_port_pool_stats_ext,
     redis_clear_port_pool_stats
 };

lib/src/sai_redis_queue.cpp

@@ -13,6 +13,20 @@ sai_status_t redis_clear_queue_stats(
     return SAI_STATUS_NOT_IMPLEMENTED;
 }
 
+sai_status_t redis_get_queue_stats_ext(
+        _In_ sai_object_id_t queue_id,
+        _In_ uint32_t number_of_counters,
+        _In_ const sai_queue_stat_t *counter_ids,
+        _In_ sai_stats_mode_t mode,
+        _Out_ uint64_t *counters)
+{
+    MUTEX();
+
+    SWSS_LOG_ENTER();
+
+    return SAI_STATUS_NOT_IMPLEMENTED;
+}
+
 REDIS_GENERIC_QUAD(QUEUE,queue);
 
 REDIS_GENERIC_GET_STATS(QUEUE, queue);
@@ -22,5 +36,6 @@ const sai_queue_api_t redis_queue_api = {
     REDIS_GENERIC_QUAD_API(queue)
 
     redis_get_queue_stats,
+    redis_get_queue_stats_ext,
     redis_clear_queue_stats,
 };

lib/src/sai_redis_tunnel.cpp

@@ -12,6 +12,19 @@ sai_status_t redis_clear_tunnel_stats(
     return SAI_STATUS_NOT_IMPLEMENTED;
 }
 
+sai_status_t redis_get_tunnel_stats_ext(
+        _In_ sai_object_id_t tunnel_id,
+        _In_ uint32_t number_of_counters,
+        _In_ const sai_tunnel_stat_t *counter_ids,
+        _In_ sai_stats_mode_t mode,
+        _Out_ uint64_t *counters)
+{
+    MUTEX();
+
+    SWSS_LOG_ENTER();
+
+    return SAI_STATUS_NOT_IMPLEMENTED;
+}
 
 REDIS_GENERIC_QUAD(TUNNEL_MAP,tunnel_map);
 REDIS_GENERIC_QUAD(TUNNEL,tunnel);
@@ -25,6 +38,7 @@ const sai_tunnel_api_t redis_tunnel_api = {
     REDIS_GENERIC_QUAD_API(tunnel)
 
     redis_get_tunnel_stats,
+    redis_get_tunnel_stats_ext,
     redis_clear_tunnel_stats,
 
     REDIS_GENERIC_QUAD_API(tunnel_term_table_entry)

lib/src/sai_redis_vlan.cpp

@@ -42,6 +42,20 @@ sai_status_t redis_get_vlan_stats(
     return SAI_STATUS_NOT_IMPLEMENTED;
 }
 
+sai_status_t redis_get_vlan_stats_ext(
+        _In_ sai_object_id_t vlan_id,
+        _In_ uint32_t number_of_counters,
+        _In_ const sai_vlan_stat_t *counter_ids,
+        _In_ sai_stats_mode_t mode,
+        _Out_ uint64_t *counters)
+{
+    MUTEX();
+
+    SWSS_LOG_ENTER();
+
+    return SAI_STATUS_NOT_IMPLEMENTED;
+}
+
 sai_status_t redis_clear_vlan_stats(
         _In_ sai_object_id_t vlan_id,
         _In_ uint32_t number_of_counters,
@@ -65,5 +79,6 @@ const sai_vlan_api_t redis_vlan_api = {
     redis_create_vlan_members,
     redis_remove_vlan_members,
     redis_get_vlan_stats,
+    redis_get_vlan_stats_ext,
     redis_clear_vlan_stats,
 };

syncd/syncd_notifications.cpp

@@ -460,7 +460,7 @@ void on_switch_state_change(
 
 void on_fdb_event(
         _In_ uint32_t count,
-        _In_ sai_fdb_event_notification_data_t *data)
+        _In_ const sai_fdb_event_notification_data_t *data)
 {
     SWSS_LOG_ENTER();
 
@@ -471,7 +471,7 @@ void on_fdb_event(
 
 void on_port_state_change(
         _In_ uint32_t count,
-        _In_ sai_port_oper_status_notification_t *data)
+        _In_ const sai_port_oper_status_notification_t *data)
 {
     SWSS_LOG_ENTER();
 
@@ -492,8 +492,8 @@ void on_switch_shutdown_request(
 
 void on_packet_event(
         _In_ sai_object_id_t switch_id,
-        _In_ const void *buffer,
         _In_ sai_size_t buffer_size,
+        _In_ const void *buffer,
         _In_ uint32_t attr_count,
         _In_ const sai_attribute_t *attr_list)
 {
@@ -579,7 +579,7 @@ void stopNotificationsProcessingThread()
 
 void on_queue_deadlock(
         _In_ uint32_t count,
-        _In_ sai_queue_deadlock_notification_data_t *data)
+        _In_ const sai_queue_deadlock_notification_data_t *data)
 {
     std::lock_guard<std::mutex> lock(g_mutex);
 
@@ -589,7 +589,7 @@ void on_queue_deadlock(
 
     for (uint32_t i = 0; i < count; i++)
     {
-        sai_queue_deadlock_notification_data_t *deadlock_data = &data[i];
+        //sai_queue_deadlock_notification_data_t *deadlock_data = &data[i];
 
         /*
          * We are using switch_rid as null, since queue should be already
@@ -600,7 +600,7 @@ void on_queue_deadlock(
          * switch vid.
          */
 
-        deadlock_data->queue_id = translate_rid_to_vid(deadlock_data->queue_id, SAI_NULL_OBJECT_ID);
+        //deadlock_data->queue_id = translate_rid_to_vid(deadlock_data->queue_id, SAI_NULL_OBJECT_ID);
     }
 
     std::string s = sai_serialize_queue_deadlock_ntf(count, data);