Implementation for SWDEV-479728:[RDC] - Clock Speed/Power Cap Control

Change-Id: I767a71325527aa3c691e9607953ceafebacfb4d5
Signed-off-by: adapryor <Adam.pryor@amd.com>

include/rdc_lib/impl/RdcConfigSettingsImpl.h

@@ -0,0 +1,73 @@
+/*
+Copyright (c) 2024 - present Advanced Micro Devices, Inc. All rights reserved.
+
+Permission is hereby granted, free of charge, to any person obtaining a copy
+of this software and associated documentation files (the "Software"), to deal
+in the Software without restriction, including without limitation the rights
+to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+copies of the Software, and to permit persons to whom the Software is
+furnished to do so, subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in
+all copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL THE
+AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
+THE SOFTWARE.
+*/
+#ifndef INCLUDE_RDC_LIB_IMPL_RDCCONFIGSETTINGSIMPL_H_
+#define INCLUDE_RDC_LIB_IMPL_RDCCONFIGSETTINGSIMPL_H_
+
+#include <atomic>
+#include <condition_variable>
+#include <mutex>  // NOLINT
+#include <thread>
+#include <unordered_map>
+
+#include "rdc_lib/RdcConfigSettings.h"
+#include "rdc_lib/RdcGroupSettings.h"
+
+namespace amd {
+namespace rdc {
+
+class RdcConfigSettingsImpl : public RdcConfigSettings {
+ public:
+  // Set one configure
+  rdc_status_t rdc_config_set(rdc_gpu_group_t group_id, rdc_config_setting_t setting) override;
+
+  // Get the setting
+  rdc_status_t rdc_config_get(rdc_gpu_group_t group_id,
+                              rdc_config_setting_list_t* settings) override;
+
+  // clear the setting
+  rdc_status_t rdc_config_clear(rdc_gpu_group_t group_id) override;
+
+  explicit RdcConfigSettingsImpl(const RdcGroupSettingsPtr& group_settings);
+
+ private:
+  RdcGroupSettingsPtr group_settings_;
+  std::unordered_map<rdc_gpu_group_t, std::unordered_map<rdc_config_type_t, rdc_config_setting_t>>
+      cached_group_settings_;
+  std::thread monitor_thread_;
+  std::mutex mutex_;              // Mutex for cached_group_settings_
+  std::atomic<bool> is_running_;  // Bool for  if the thread should keep running
+  std::condition_variable cv_;
+
+  // monitorSettings() is kicked off from the RdcConfigSettingsImpl constructor as it's own thread
+  // Every minute, it will check if gpu settings from amdsmi are the same as inside
+  // cached_group_settings_ If not, it sets the mismatched values to the value in
+  // cached_group_settings_
+  void monitorSettings();
+  uint64_t wattsToMicrowatts(uint64_t watts) const;
+  uint64_t microwattsToWatts(int microwatts) const;
+  rdc_status_t get_group_info(rdc_gpu_group_t group_id, rdc_group_info_t* rdc_group_info);
+};
+
+}  // namespace rdc
+}  // namespace amd
+
+#endif  // INCLUDE_RDC_LIB_IMPL_RDCCONFIGSETTINGSIMPL_H_

include/rdc_lib/impl/RdcEmbeddedHandler.h

@@ -25,6 +25,7 @@ THE SOFTWARE.
 #include <future>  // NOLINT(build/c++11)
 
 #include "rdc_lib/RdcCacheManager.h"
+#include "rdc_lib/RdcConfigSettings.h"
 #include "rdc_lib/RdcGroupSettings.h"
 #include "rdc_lib/RdcHandler.h"
 #include "rdc_lib/RdcMetricFetcher.h"
@@ -86,17 +87,20 @@ class RdcEmbeddedHandler final : public RdcHandler {
   // Diagnostic API
   rdc_status_t rdc_diagnostic_run(rdc_gpu_group_t group_id, rdc_diag_level_t level,
                                   const char* config, size_t config_size,
-                                  rdc_diag_response_t* response, rdc_diag_callback_t* callback) override;
+                                  rdc_diag_response_t* response,
+                                  rdc_diag_callback_t* callback) override;
   rdc_status_t rdc_test_case_run(rdc_gpu_group_t group_id, rdc_diag_test_cases_t test_case,
                                  const char* config, size_t config_size,
-                                 rdc_diag_test_result_t* result, rdc_diag_callback_t* callback) override;
+                                 rdc_diag_test_result_t* result,
+                                 rdc_diag_callback_t* callback) override;
 
   // Control API
   rdc_status_t rdc_field_update_all(uint32_t wait_for_update) override;
 
   // It is just a client interface under the GRPC framework and is not used as an RDC API.
   // Pure virtual functions need to be overridden.
-  rdc_status_t get_mixed_component_version(mixed_component_t component, mixed_component_version_t* p_mixed_compv) override;
+  rdc_status_t get_mixed_component_version(mixed_component_t component,
+                                           mixed_component_version_t* p_mixed_compv) override;
   // Policy API
   rdc_status_t rdc_policy_set(rdc_gpu_group_t group_id, rdc_policy_t policy) override;
 
@@ -106,19 +110,30 @@ class RdcEmbeddedHandler final : public RdcHandler {
   rdc_status_t rdc_policy_delete(rdc_gpu_group_t group_id,
                                  rdc_policy_condition_type_t condition_type) override;
 
-  rdc_status_t rdc_policy_register(rdc_gpu_group_t group_id, rdc_policy_register_callback callback) override;
+  rdc_status_t rdc_policy_register(rdc_gpu_group_t group_id,
+                                   rdc_policy_register_callback callback) override;
 
   rdc_status_t rdc_policy_unregister(rdc_gpu_group_t group_id) override;
 
   // Health API
   rdc_status_t rdc_health_set(rdc_gpu_group_t group_id, unsigned int components) override;
   rdc_status_t rdc_health_get(rdc_gpu_group_t group_id, unsigned int* components) override;
-  rdc_status_t rdc_health_check(rdc_gpu_group_t group_id, rdc_health_response_t *response) override;
+  rdc_status_t rdc_health_check(rdc_gpu_group_t group_id, rdc_health_response_t* response) override;
   rdc_status_t rdc_health_clear(rdc_gpu_group_t group_id) override;
   rdc_status_t rdc_device_topology_get(uint32_t gpu_index, rdc_device_topology_t* results) override;
   
   rdc_status_t rdc_link_status_get(rdc_link_status_t* results) override;
 
+  // Set one configure
+  rdc_status_t rdc_config_set(rdc_gpu_group_t group_id, rdc_config_setting_t setting) override;
+
+  // Get the setting
+  rdc_status_t rdc_config_get(rdc_gpu_group_t group_id,
+                              rdc_config_setting_list_t* settings) override;
+
+  // Clear the setting
+  rdc_status_t rdc_config_clear(rdc_gpu_group_t group_id) override;
+
   explicit RdcEmbeddedHandler(rdc_operation_mode_t op_mode);
   ~RdcEmbeddedHandler() final;
 
@@ -132,6 +147,7 @@ class RdcEmbeddedHandler final : public RdcHandler {
   RdcWatchTablePtr watch_table_;
   RdcMetricsUpdaterPtr metrics_updater_;
   RdcPolicyPtr policy_;
+  RdcConfigSettingsPtr config_handler_;
   std::future<void> updater_;
   RdcTopologyLinkPtr topologylink_;
 };

include/rdc_lib/impl/RdcStandaloneHandler.h

@@ -23,8 +23,8 @@ THE SOFTWARE.
 #define INCLUDE_RDC_LIB_IMPL_RDCSTANDALONEHANDLER_H_
 #include <grpcpp/grpcpp.h>
 
-#include <memory>
 #include <future>
+#include <memory>
 #include <thread>
 
 #include "rdc.grpc.pb.h"  // NOLINT
@@ -82,21 +82,35 @@ class RdcStandaloneHandler : public RdcHandler {
   // Diagnostic API
   rdc_status_t rdc_diagnostic_run(rdc_gpu_group_t group_id, rdc_diag_level_t level,
                                   const char* config, size_t config_size,
-                                  rdc_diag_response_t* response, rdc_diag_callback_t* callback) override;
+                                  rdc_diag_response_t* response,
+                                  rdc_diag_callback_t* callback) override;
   rdc_status_t rdc_test_case_run(rdc_gpu_group_t group_id, rdc_diag_test_cases_t test_case,
                                  const char* config, size_t config_size,
-                                 rdc_diag_test_result_t* result, rdc_diag_callback_t* callback) override;
+                                 rdc_diag_test_result_t* result,
+                                 rdc_diag_callback_t* callback) override;
 
   // Control RdcAPI
   rdc_status_t rdc_field_update_all(uint32_t wait_for_update) override;
 
+  // Set one configure
+  rdc_status_t rdc_config_set(rdc_gpu_group_t group_id, rdc_config_setting_t setting) override;
+
+  // Get the setting
+  rdc_status_t rdc_config_get(rdc_gpu_group_t group_id,
+                              rdc_config_setting_list_t* settings) override;
+
+  // Clear the setting
+  rdc_status_t rdc_config_clear(rdc_gpu_group_t group_id) override;
+
   // It is just a client interface under the GRPC framework and is not used as an RDC API.
   // Pure virtual functions need to be overridden
-  rdc_status_t get_mixed_component_version(mixed_component_t component, mixed_component_version_t* p_mixed_compv) override;
+  rdc_status_t get_mixed_component_version(mixed_component_t component,
+                                           mixed_component_version_t* p_mixed_compv) override;
   // Policy API
   rdc_status_t rdc_policy_set(rdc_gpu_group_t group_id, rdc_policy_t policy) override;
 
-  rdc_status_t rdc_policy_get(rdc_gpu_group_t group_id, uint32_t* count, rdc_policy_t policies[RDC_MAX_POLICY_SETTINGS]) override;
+  rdc_status_t rdc_policy_get(rdc_gpu_group_t group_id, uint32_t* count,
+                              rdc_policy_t policies[RDC_MAX_POLICY_SETTINGS]) override;
 
   rdc_status_t rdc_policy_delete(rdc_gpu_group_t group_id,
                                  rdc_policy_condition_type_t condition_type) override;
@@ -109,7 +123,7 @@ class RdcStandaloneHandler : public RdcHandler {
   // Health API
   rdc_status_t rdc_health_set(rdc_gpu_group_t group_id, unsigned int components) override;
   rdc_status_t rdc_health_get(rdc_gpu_group_t group_id, unsigned int* components) override;
-  rdc_status_t rdc_health_check(rdc_gpu_group_t group_id, rdc_health_response_t *response) override;
+  rdc_status_t rdc_health_check(rdc_gpu_group_t group_id, rdc_health_response_t* response) override;
   rdc_status_t rdc_health_clear(rdc_gpu_group_t group_id) override;
   rdc_status_t rdc_device_topology_get(uint32_t gpu_index, rdc_device_topology_t* results) override;
   
@@ -129,11 +143,10 @@ class RdcStandaloneHandler : public RdcHandler {
 
   struct policy_thread_context {
     bool start;
-    std::thread *t;
+    std::thread* t;
   };
 
   std::map<uint32_t, struct policy_thread_context> policy_threads_;
-
 };
 
 }  // namespace rdc