Optimize bulk counter to make the polling interval accurate

Signed-off-by: Stephen Sun <[email protected]>
sonic-net · Jan 8, 2025 · 44a3c65 · 44a3c65
1 parent 1684aec
commit 44a3c65
Show file tree

Hide file tree

Showing 5 changed files with 665 additions and 48 deletions.
diff --git a/lib/RedisRemoteSaiInterface.cpp b/lib/RedisRemoteSaiInterface.cpp
@@ -568,6 +568,8 @@ sai_status_t RedisRemoteSaiInterface::notifyCounterGroupOperations(
     std::string key((const char*)flexCounterGroupParam->counter_group_name.list, flexCounterGroupParam->counter_group_name.count);
 
     emplaceStrings(POLL_INTERVAL_FIELD, flexCounterGroupParam->poll_interval, entries);
+    emplaceStrings(BULK_CHUNK_SIZE_FIELD, flexCounterGroupParam->bulk_chunk_size, entries);
+    emplaceStrings(BULK_CHUNK_SIZE_PER_PREFIX_FIELD, flexCounterGroupParam->bulk_chunk_size_per_prefix, entries);
     emplaceStrings(STATS_MODE_FIELD, flexCounterGroupParam->stats_mode, entries);
     emplaceStrings(flexCounterGroupParam->plugin_name, flexCounterGroupParam->plugins, entries);
     emplaceStrings(FLEX_COUNTER_STATUS_FIELD, flexCounterGroupParam->operation, entries);

diff --git a/lib/sairedis.h b/lib/sairedis.h
@@ -140,6 +140,20 @@ typedef struct _sai_redis_flex_counter_group_parameter_t
      */
     sai_s8_list_t plugins;
 
+    /**
+     * @brief The bulk chunk size of the counter group
+     *
+     * It should be a number representing the bulk chunk size.
+     */
+    sai_s8_list_t bulk_chunk_size;
+
+    /**
+     * @brief The bulk counter prefix map the counter group
+     *
+     * It should be a string representing bulk chunk size of each sub counter group.
+     */
+    sai_s8_list_t bulk_chunk_size_per_prefix;
+
 } sai_redis_flex_counter_group_parameter_t;
 
 typedef struct _sai_redis_flex_counter_parameter_t