commit e3b96a119fb58da9e5fd78976ddd5a5dea84c9eb
author Jason M. Bills <jason.m.bills@linux.intel.com> Mon Nov 04 06:54:31 2024 -0800
committer Jason Bills <jason.m.bills@linux.intel.com> Mon Jan 27 20:06:45 2025 +0000
tree bc442e7730e6a5e67e19a425d4c428cabe334b1e
parent f955029d7fa7b18c050e9265f833cb9337f2856d

Change to zero-based CPU numbering

Preferences have moved toward zero-based CPU numbering, so update error
logs to remove the increment.

Signed-off-by: Jason M. Bills <jason.m.bills@linux.intel.com>
Change-Id: If6c92476a636f2966d6dbed1eac5daab20101e2c

include/error_monitors/err_pin_monitor.hpp

diff --git a/include/error_monitors/err_pin_monitor.hpp b/include/error_monitors/err_pin_monitor.hpp
index e1b8954..802da91 100644
--- a/include/error_monitors/err_pin_monitor.hpp
+++ b/include/error_monitors/err_pin_monitor.hpp
@@ -62,7 +62,7 @@
     void errPinLog(const int cpuNum)
     {
         std::string msg = "ERR" + std::to_string(errPin) + " on CPU " +
-                          std::to_string(cpuNum + 1);
+                          std::to_string(cpuNum);
 
         log_message(LOG_INFO, msg, "OpenBMC.0.1.CPUError", msg);
     }

include/error_monitors/err_pin_timeout_monitor.hpp

diff --git a/include/error_monitors/err_pin_timeout_monitor.hpp b/include/error_monitors/err_pin_timeout_monitor.hpp
index 852c491..14f2895 100644
--- a/include/error_monitors/err_pin_timeout_monitor.hpp
+++ b/include/error_monitors/err_pin_timeout_monitor.hpp
@@ -63,7 +63,7 @@
     void errPinTimeoutLog(const int cpuNum)
     {
         std::string msg = "ERR" + std::to_string(errPin) + " Timeout on CPU " +
-                          std::to_string(cpuNum + 1);
+                          std::to_string(cpuNum);
 
         log_message(LOG_INFO, msg, "OpenBMC.0.1.CPUError", msg);
     }

include/error_monitors/ierr_monitor.hpp

diff --git a/include/error_monitors/ierr_monitor.hpp b/include/error_monitors/ierr_monitor.hpp
index b16ce17..2a2dc78 100644
--- a/include/error_monitors/ierr_monitor.hpp
+++ b/include/error_monitors/ierr_monitor.hpp
@@ -59,14 +59,14 @@
 
     void cpuIERRLog(const int cpuNum)
     {
-        std::string msg = "IERR on CPU " + std::to_string(cpuNum + 1);
+        std::string msg = "IERR on CPU " + std::to_string(cpuNum);
 
         log_message(LOG_INFO, msg, "OpenBMC.0.1.CPUError", msg);
     }
 
     void cpuIERRLog(const int cpuNum, const std::string& type)
     {
-        std::string msg = type + " IERR on CPU " + std::to_string(cpuNum + 1);
+        std::string msg = type + " IERR on CPU " + std::to_string(cpuNum);
 
         log_message(LOG_INFO, msg, "OpenBMC.0.1.CPUError", msg);
     }