nvidia-gpu: add thresholds support to TLimit
This patch introduces support for retrieving GPU TLimit thresholds
directly from the GPU device. TLimit Temperature represents the
difference in degrees Celsius between the current GPU temperature and
the initial throttle threshold. The patch also enables the extraction of
three critical throttle thresholds — Warning Low, Critical Low, and Hard
Shutdown Low — from the GPU hardware.
Tested: Build an image for gb200nvl-obmc machine with the following
patches cherry picked. This patches are needed to enable the mctp stack.
https://gerrit.openbmc.org/c/openbmc/openbmc/+/79422
```
$ curl -s -k -u 'root:0penBmc' https://10.137.203.137/redfish/v1/Chassis/NVIDIA_GB200_1/Sensors/temperature_NVIDIA_GB200_GPU_0_TEMP_1
{
"@odata.id": "/redfish/v1/Chassis/NVIDIA_GB200_1/Sensors/temperature_NVIDIA_GB200_GPU_0_TEMP_1",
"@odata.type": "#Sensor.v1_2_0.Sensor",
"Id": "temperature_NVIDIA_GB200_GPU_0_TEMP_1",
"Name": "NVIDIA GB200 GPU 0 TEMP 1",
"Reading": 57.3984375,
"ReadingRangeMax": 127.0,
"ReadingRangeMin": -128.0,
"ReadingType": "Temperature",
"ReadingUnits": "Cel",
"Status": {
"Health": "OK",
"State": "Enabled"
},
"Thresholds": {
"LowerCaution": {
"Reading": 0.0
},
"LowerCritical": {
"Reading": 0.0
},
"LowerFatal": {
"Reading": 0.0
}
}
}%
```
Change-Id: I6f2ff2652ce9246287f9bd63c4297d9ad3229963
Signed-off-by: Harshit Aghera <haghera@nvidia.com>
diff --git a/src/nvidia-gpu/NvidiaGpuDevice.cpp b/src/nvidia-gpu/NvidiaGpuDevice.cpp
index ab476da..509a353 100644
--- a/src/nvidia-gpu/NvidiaGpuDevice.cpp
+++ b/src/nvidia-gpu/NvidiaGpuDevice.cpp
@@ -14,6 +14,7 @@
#include <bits/basic_string.h>
#include <MctpRequester.hpp>
+#include <NvidiaGpuThresholds.hpp>
#include <boost/asio/io_context.hpp>
#include <phosphor-logging/lg2.hpp>
#include <sdbusplus/asio/connection.hpp>
@@ -21,8 +22,10 @@
#include <chrono>
#include <cstdint>
+#include <functional>
#include <memory>
#include <string>
+#include <utility>
#include <vector>
GpuDevice::GpuDevice(const SensorConfigs& configs, const std::string& name,
@@ -45,9 +48,13 @@
conn, mctpRequester, name + "_TEMP_0", path, eid, gpuTempSensorId,
objectServer, std::vector<thresholds::Threshold>{});
- tLimitSensor = std::make_shared<NvidiaGpuTempSensor>(
- conn, mctpRequester, name + "_TEMP_1", path, eid, gpuTLimitSensorId,
- objectServer, std::vector<thresholds::Threshold>{});
+ readThermalParameters(
+ eid,
+ std::vector<gpuThresholdId>{gpuTLimitWarnringThresholdId,
+ gpuTLimitCriticalThresholdId,
+ gpuTLimitHardshutDownThresholdId},
+ mctpRequester,
+ std::bind_front(&GpuDevice::processTLimitThresholds, this));
lg2::info("Added GPU {NAME} Sensors with chassis path: {PATH}.", "NAME",
name, "PATH", path);
@@ -55,10 +62,36 @@
read();
}
+void GpuDevice::processTLimitThresholds(uint8_t rc,
+ const std::vector<int32_t>& thresholds)
+{
+ std::vector<thresholds::Threshold> tLimitThresholds{};
+ if (rc == 0)
+ {
+ tLimitThresholds = {
+ thresholds::Threshold{thresholds::Level::WARNING,
+ thresholds::Direction::LOW,
+ static_cast<double>(thresholds[0])},
+ thresholds::Threshold{thresholds::Level::CRITICAL,
+ thresholds::Direction::LOW,
+ static_cast<double>(thresholds[1])},
+ thresholds::Threshold{thresholds::Level::HARDSHUTDOWN,
+ thresholds::Direction::LOW,
+ static_cast<double>(thresholds[2])}};
+ }
+
+ tLimitSensor = std::make_shared<NvidiaGpuTempSensor>(
+ conn, mctpRequester, name + "_TEMP_1", path, eid, gpuTLimitSensorId,
+ objectServer, std::move(tLimitThresholds));
+}
+
void GpuDevice::read()
{
tempSensor->update();
- tLimitSensor->update();
+ if (tLimitSensor)
+ {
+ tLimitSensor->update();
+ }
waitTimer.expires_after(std::chrono::milliseconds(sensorPollMs));
waitTimer.async_wait([this](const boost::system::error_code& ec) {