Fix logic bug in cache_nvidia_string_value_update() (#2022)

* Update nvidia.cc Made a suggested change in logic from issue #1177 * Update nvidia.cc Made a suggested change in logic from issue #1178 * Update nvidia.cc Made a suggested change in logic from issue #1178
brndnmtthws · Aug 29, 2024 · 03711ea · 03711ea
1 parent c428663
commit 03711ea
Showing 1 changed file with 5 additions and 4 deletions.
diff --git a/src/nvidia.cc b/src/nvidia.cc
@@ -829,11 +829,12 @@ void cache_nvidia_string_value_update(nvidia_c_string *ac_string, char *token,
              ac_string[gid].memTransferRatemax < 0) {
     ac_string[gid].memTransferRatemax = *value;
 
-  } else if (strcmp(token, (char *)"perf") == 0 &&
-             ac_string[gid].memTransferRatemax < 0) {
-    if (search == SEARCH_MIN) {
+  } else if (strcmp(token, (char *)"perf") == 0) {
+    if (search == SEARCH_MIN &&
+        ac_string[gid].perfmin < 0) {
       ac_string[gid].perfmin = *value;
-    } else if (search == SEARCH_MAX) {
+    } else if (search == SEARCH_MAX &&
+               ac_string[gid].perfmax < 0) {
       ac_string[gid].perfmax = *value;
     }
   }