SAI submodule update to bring saithrift changes for syncd bookworm (#…

…1370) * SAI submodule update to bring saithrift changes for bookworm syncd Signed-off-by: Keshav Gupta <keshavg@marvell.com> * Fix build issues Signed-off-by: Keshav Gupta <keshavg@marvell.com> * Add tests for code coverage Signed-off-by: Keshav Gupta <keshavg@marvell.com> --------- Signed-off-by: Keshav Gupta <keshavg@marvell.com> Co-authored-by: Saikrishna Arcot <sarcot@microsoft.com>
sonic-net · Apr 15, 2024 · c41a0cb · c41a0cb
1 parent be47489
commit c41a0cb
Show file tree

Hide file tree

Showing 5 changed files with 11 additions and 1 deletion.
diff --git a/SAI b/SAI
diff --git a/lib/sai_redis_dash_eni.cpp b/lib/sai_redis_dash_eni.cpp
@@ -5,6 +5,7 @@ REDIS_BULK_CREATE_ENTRY_EX(ENI_ETHER_ADDRESS_MAP_ENTRY, eni_ether_address_map_en
 REDIS_BULK_REMOVE_ENTRY_EX(ENI_ETHER_ADDRESS_MAP_ENTRY, eni_ether_address_map_entry, eni_ether_address_map_entries);
 
 REDIS_GENERIC_QUAD(ENI, eni);
+REDIS_GENERIC_STATS(ENI, eni);
 REDIS_BULK_CREATE(ENI, enis);
 REDIS_BULK_REMOVE(ENI, enis);
 
@@ -14,6 +15,7 @@ const sai_dash_eni_api_t redis_dash_eni_api = {
     redis_bulk_remove_eni_ether_address_map_entries,
 
     REDIS_GENERIC_QUAD_API(eni)
+    REDIS_GENERIC_STATS_API(eni)
     redis_bulk_create_enis,
     redis_bulk_remove_enis,
 };
diff --git a/unittest/lib/test_sai_redis_dash_eni.cpp b/unittest/lib/test_sai_redis_dash_eni.cpp
@@ -28,6 +28,9 @@ TEST(libsairedis, dash_eni)
     EXPECT_NE(SAI_STATUS_SUCCESS, api->remove_eni(0));
     EXPECT_NE(SAI_STATUS_SUCCESS, api->set_eni_attribute(0,0));
     EXPECT_NE(SAI_STATUS_SUCCESS, api->get_eni_attribute(0,0,0));
+    EXPECT_NE(SAI_STATUS_SUCCESS, api->get_eni_stats(0,0,0,0));
+    EXPECT_NE(SAI_STATUS_SUCCESS, api->get_eni_stats_ext(0,0,0,SAI_STATS_MODE_READ,0));
+    EXPECT_NE(SAI_STATUS_SUCCESS, api->clear_eni_stats(0,0,0));
     EXPECT_NE(SAI_STATUS_SUCCESS, api->create_enis(0,0,0,0,SAI_BULK_OP_ERROR_MODE_IGNORE_ERROR,0,0));
     EXPECT_NE(SAI_STATUS_SUCCESS, api->remove_enis(0,0,SAI_BULK_OP_ERROR_MODE_IGNORE_ERROR,0));
 }
diff --git a/unittest/vslib/test_sai_vs_dash_eni.cpp b/unittest/vslib/test_sai_vs_dash_eni.cpp
@@ -28,6 +28,9 @@ TEST(libsaivs, dash_eni)
     EXPECT_NE(SAI_STATUS_SUCCESS, api->remove_eni(0));
     EXPECT_NE(SAI_STATUS_SUCCESS, api->set_eni_attribute(0,0));
     EXPECT_NE(SAI_STATUS_SUCCESS, api->get_eni_attribute(0,0,0));
+    EXPECT_NE(SAI_STATUS_SUCCESS, api->get_eni_stats(0,0,0,0));
+    EXPECT_NE(SAI_STATUS_SUCCESS, api->get_eni_stats_ext(0,0,0,SAI_STATS_MODE_READ,0));
+    EXPECT_NE(SAI_STATUS_SUCCESS, api->clear_eni_stats(0,0,0));
     EXPECT_NE(SAI_STATUS_SUCCESS, api->create_enis(0,0,0,0,SAI_BULK_OP_ERROR_MODE_IGNORE_ERROR,0,0));
     EXPECT_NE(SAI_STATUS_SUCCESS, api->remove_enis(0,0,SAI_BULK_OP_ERROR_MODE_IGNORE_ERROR,0));
 }
diff --git a/vslib/sai_vs_dash_eni.cpp b/vslib/sai_vs_dash_eni.cpp
@@ -5,6 +5,7 @@ VS_BULK_CREATE_ENTRY_EX(ENI_ETHER_ADDRESS_MAP_ENTRY, eni_ether_address_map_entry
 VS_BULK_REMOVE_ENTRY_EX(ENI_ETHER_ADDRESS_MAP_ENTRY, eni_ether_address_map_entry, eni_ether_address_map_entries);
 
 VS_GENERIC_QUAD(ENI, eni);
+VS_GENERIC_STATS(ENI, eni);
 VS_BULK_CREATE(ENI, enis);
 VS_BULK_REMOVE(ENI, enis);
 
@@ -14,6 +15,7 @@ const sai_dash_eni_api_t vs_dash_eni_api = {
     vs_bulk_remove_eni_ether_address_map_entries,
 
     VS_GENERIC_QUAD_API(eni)
+    VS_GENERIC_STATS_API(eni)
     vs_bulk_create_enis,
     vs_bulk_remove_enis,
 };