James Feist | 6ef2040 | 2019-01-07 16:45:08 -0800 | [diff] [blame] | 1 | /* |
| 2 | // Copyright (c) 2019 Intel Corporation |
| 3 | // |
| 4 | // Licensed under the Apache License, Version 2.0 (the "License"); |
| 5 | // you may not use this file except in compliance with the License. |
| 6 | // You may obtain a copy of the License at |
| 7 | // |
| 8 | // http://www.apache.org/licenses/LICENSE-2.0 |
| 9 | // |
| 10 | // Unless required by applicable law or agreed to in writing, software |
| 11 | // distributed under the License is distributed on an "AS IS" BASIS, |
| 12 | // WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. |
| 13 | // See the License for the specific language governing permissions and |
| 14 | // limitations under the License. |
| 15 | */ |
| 16 | |
Andrew Jeffery | e73bd0a | 2023-01-25 10:39:57 +1030 | [diff] [blame] | 17 | #include "IpmbSensor.hpp" |
| 18 | |
| 19 | #include "IpmbSDRSensor.hpp" |
Ed Tanous | eacbfdd | 2024-04-04 12:00:24 -0700 | [diff] [blame] | 20 | #include "SensorPaths.hpp" |
| 21 | #include "Thresholds.hpp" |
Andrew Jeffery | e73bd0a | 2023-01-25 10:39:57 +1030 | [diff] [blame] | 22 | #include "Utils.hpp" |
| 23 | #include "VariantVisitors.hpp" |
Ed Tanous | eacbfdd | 2024-04-04 12:00:24 -0700 | [diff] [blame] | 24 | #include "sensor.hpp" |
Andrew Jeffery | e73bd0a | 2023-01-25 10:39:57 +1030 | [diff] [blame] | 25 | |
Ed Tanous | eacbfdd | 2024-04-04 12:00:24 -0700 | [diff] [blame] | 26 | #include <boost/asio/error.hpp> |
| 27 | #include <boost/asio/io_context.hpp> |
Ed Tanous | eacbfdd | 2024-04-04 12:00:24 -0700 | [diff] [blame] | 28 | #include <boost/asio/steady_timer.hpp> |
Patrick Venture | 96e97db | 2019-10-31 13:44:38 -0700 | [diff] [blame] | 29 | #include <boost/container/flat_map.hpp> |
James Feist | 38fb598 | 2020-05-28 10:09:54 -0700 | [diff] [blame] | 30 | #include <sdbusplus/asio/connection.hpp> |
| 31 | #include <sdbusplus/asio/object_server.hpp> |
Ed Tanous | eacbfdd | 2024-04-04 12:00:24 -0700 | [diff] [blame] | 32 | #include <sdbusplus/message.hpp> |
| 33 | #include <sdbusplus/message/native_types.hpp> |
James Feist | 38fb598 | 2020-05-28 10:09:54 -0700 | [diff] [blame] | 34 | |
Ed Tanous | eacbfdd | 2024-04-04 12:00:24 -0700 | [diff] [blame] | 35 | #include <algorithm> |
| 36 | #include <array> |
James Feist | 6ef2040 | 2019-01-07 16:45:08 -0800 | [diff] [blame] | 37 | #include <chrono> |
Ed Tanous | eacbfdd | 2024-04-04 12:00:24 -0700 | [diff] [blame] | 38 | #include <cstddef> |
| 39 | #include <cstdint> |
James Feist | 6ef2040 | 2019-01-07 16:45:08 -0800 | [diff] [blame] | 40 | #include <iostream> |
| 41 | #include <limits> |
Patrick Venture | 96e97db | 2019-10-31 13:44:38 -0700 | [diff] [blame] | 42 | #include <memory> |
Ed Tanous | eacbfdd | 2024-04-04 12:00:24 -0700 | [diff] [blame] | 43 | #include <stdexcept> |
Patrick Venture | 96e97db | 2019-10-31 13:44:38 -0700 | [diff] [blame] | 44 | #include <string> |
| 45 | #include <tuple> |
Ed Tanous | eacbfdd | 2024-04-04 12:00:24 -0700 | [diff] [blame] | 46 | #include <utility> |
Patrick Venture | 96e97db | 2019-10-31 13:44:38 -0700 | [diff] [blame] | 47 | #include <variant> |
James Feist | 6ef2040 | 2019-01-07 16:45:08 -0800 | [diff] [blame] | 48 | #include <vector> |
| 49 | |
| 50 | constexpr const bool debug = false; |
| 51 | |
James Feist | 6ef2040 | 2019-01-07 16:45:08 -0800 | [diff] [blame] | 52 | static constexpr double ipmbMaxReading = 0xFF; |
| 53 | static constexpr double ipmbMinReading = 0; |
| 54 | |
| 55 | static constexpr uint8_t meAddress = 1; |
| 56 | static constexpr uint8_t lun = 0; |
Anoop S | 832a2c6 | 2020-11-20 19:21:22 +0000 | [diff] [blame] | 57 | static constexpr uint8_t hostSMbusIndexDefault = 0x03; |
Jayashree Dhanapal | 6ee6294 | 2021-12-14 15:22:23 +0530 | [diff] [blame] | 58 | static constexpr uint8_t ipmbBusIndexDefault = 0; |
Jayashree-D | 9f6d4fd | 2021-04-13 18:27:22 +0530 | [diff] [blame] | 59 | static constexpr float pollRateDefault = 1; // in seconds |
James Feist | 6ef2040 | 2019-01-07 16:45:08 -0800 | [diff] [blame] | 60 | |
Vijay Khemka | 682a5cb | 2019-07-18 17:34:03 -0700 | [diff] [blame] | 61 | static constexpr const char* sensorPathPrefix = "/xyz/openbmc_project/sensors/"; |
| 62 | |
James Feist | 6ef2040 | 2019-01-07 16:45:08 -0800 | [diff] [blame] | 63 | IpmbSensor::IpmbSensor(std::shared_ptr<sdbusplus::asio::connection>& conn, |
Ed Tanous | 1f97863 | 2023-02-28 18:16:39 -0800 | [diff] [blame] | 64 | boost::asio::io_context& io, |
James Feist | 6ef2040 | 2019-01-07 16:45:08 -0800 | [diff] [blame] | 65 | const std::string& sensorName, |
| 66 | const std::string& sensorConfiguration, |
| 67 | sdbusplus::asio::object_server& objectServer, |
| 68 | std::vector<thresholds::Threshold>&& thresholdData, |
Anoop S | 832a2c6 | 2020-11-20 19:21:22 +0000 | [diff] [blame] | 69 | uint8_t deviceAddress, uint8_t hostSMbusIndex, |
Jayashree-D | 9f6d4fd | 2021-04-13 18:27:22 +0530 | [diff] [blame] | 70 | const float pollRate, std::string& sensorTypeName) : |
Zhikui Ren | da98f09 | 2021-11-01 09:41:08 -0700 | [diff] [blame] | 71 | Sensor(escapeName(sensorName), std::move(thresholdData), |
Zev Weiss | 054aad8 | 2022-08-18 01:37:34 -0700 | [diff] [blame] | 72 | sensorConfiguration, "IpmbSensor", false, false, ipmbMaxReading, |
| 73 | ipmbMinReading, conn, PowerState::on), |
Anoop S | 832a2c6 | 2020-11-20 19:21:22 +0000 | [diff] [blame] | 74 | deviceAddress(deviceAddress), hostSMbusIndex(hostSMbusIndex), |
Jayashree-D | 9f6d4fd | 2021-04-13 18:27:22 +0530 | [diff] [blame] | 75 | sensorPollMs(static_cast<int>(pollRate * 1000)), objectServer(objectServer), |
| 76 | waitTimer(io) |
James Feist | 6ef2040 | 2019-01-07 16:45:08 -0800 | [diff] [blame] | 77 | { |
Vijay Khemka | 682a5cb | 2019-07-18 17:34:03 -0700 | [diff] [blame] | 78 | std::string dbusPath = sensorPathPrefix + sensorTypeName + "/" + name; |
| 79 | |
James Feist | 6ef2040 | 2019-01-07 16:45:08 -0800 | [diff] [blame] | 80 | sensorInterface = objectServer.add_interface( |
Vijay Khemka | 682a5cb | 2019-07-18 17:34:03 -0700 | [diff] [blame] | 81 | dbusPath, "xyz.openbmc_project.Sensor.Value"); |
James Feist | 6ef2040 | 2019-01-07 16:45:08 -0800 | [diff] [blame] | 82 | |
Jayashree Dhanapal | 5667808 | 2022-01-04 17:27:20 +0530 | [diff] [blame] | 83 | for (const auto& threshold : thresholds) |
James Feist | 6ef2040 | 2019-01-07 16:45:08 -0800 | [diff] [blame] | 84 | { |
Jayashree Dhanapal | 5667808 | 2022-01-04 17:27:20 +0530 | [diff] [blame] | 85 | std::string interface = thresholds::getInterface(threshold.level); |
| 86 | thresholdInterfaces[static_cast<size_t>(threshold.level)] = |
| 87 | objectServer.add_interface(dbusPath, interface); |
James Feist | 6ef2040 | 2019-01-07 16:45:08 -0800 | [diff] [blame] | 88 | } |
James Feist | 2adc95c | 2019-09-30 14:55:28 -0700 | [diff] [blame] | 89 | association = objectServer.add_interface(dbusPath, association::interface); |
James Feist | 6ef2040 | 2019-01-07 16:45:08 -0800 | [diff] [blame] | 90 | } |
| 91 | |
| 92 | IpmbSensor::~IpmbSensor() |
| 93 | { |
| 94 | waitTimer.cancel(); |
Jayashree Dhanapal | 5667808 | 2022-01-04 17:27:20 +0530 | [diff] [blame] | 95 | for (const auto& iface : thresholdInterfaces) |
| 96 | { |
| 97 | objectServer.remove_interface(iface); |
| 98 | } |
James Feist | 6ef2040 | 2019-01-07 16:45:08 -0800 | [diff] [blame] | 99 | objectServer.remove_interface(sensorInterface); |
James Feist | 078f232 | 2019-03-08 11:09:05 -0800 | [diff] [blame] | 100 | objectServer.remove_interface(association); |
James Feist | 6ef2040 | 2019-01-07 16:45:08 -0800 | [diff] [blame] | 101 | } |
| 102 | |
Ed Tanous | 201a101 | 2024-04-03 18:07:28 -0700 | [diff] [blame] | 103 | std::string IpmbSensor::getSubTypeUnits() const |
Zev Weiss | 6b6891c | 2021-04-22 02:46:21 -0500 | [diff] [blame] | 104 | { |
| 105 | switch (subType) |
| 106 | { |
| 107 | case IpmbSubType::temp: |
| 108 | return sensor_paths::unitDegreesC; |
| 109 | case IpmbSubType::curr: |
| 110 | return sensor_paths::unitAmperes; |
| 111 | case IpmbSubType::power: |
| 112 | return sensor_paths::unitWatts; |
| 113 | case IpmbSubType::volt: |
| 114 | return sensor_paths::unitVolts; |
| 115 | case IpmbSubType::util: |
| 116 | return sensor_paths::unitPercent; |
| 117 | default: |
| 118 | throw std::runtime_error("Invalid sensor type"); |
| 119 | } |
| 120 | } |
| 121 | |
Ed Tanous | 201a101 | 2024-04-03 18:07:28 -0700 | [diff] [blame] | 122 | void IpmbSensor::init() |
James Feist | 6ef2040 | 2019-01-07 16:45:08 -0800 | [diff] [blame] | 123 | { |
James Feist | 6ef2040 | 2019-01-07 16:45:08 -0800 | [diff] [blame] | 124 | loadDefaults(); |
Andrei Kartashev | 3928741 | 2022-02-04 16:04:47 +0300 | [diff] [blame] | 125 | setInitialProperties(getSubTypeUnits()); |
James Feist | 6ef2040 | 2019-01-07 16:45:08 -0800 | [diff] [blame] | 126 | if (initCommand) |
| 127 | { |
James Feist | f7e2c5d | 2019-02-13 17:27:51 -0800 | [diff] [blame] | 128 | runInitCmd(); |
| 129 | } |
| 130 | read(); |
| 131 | } |
| 132 | |
Vikash Chandola | 1f84797 | 2022-09-28 09:47:32 +0000 | [diff] [blame] | 133 | static void initCmdCb(const std::weak_ptr<IpmbSensor>& weakRef, |
| 134 | const boost::system::error_code& ec, |
| 135 | const IpmbMethodType& response) |
| 136 | { |
| 137 | std::shared_ptr<IpmbSensor> self = weakRef.lock(); |
| 138 | if (!self) |
| 139 | { |
| 140 | return; |
| 141 | } |
| 142 | const int& status = std::get<0>(response); |
| 143 | if (ec || (status != 0)) |
| 144 | { |
| 145 | std::cerr << "Error setting init command for device: " << self->name |
| 146 | << "\n"; |
| 147 | } |
| 148 | } |
| 149 | |
James Feist | f7e2c5d | 2019-02-13 17:27:51 -0800 | [diff] [blame] | 150 | void IpmbSensor::runInitCmd() |
| 151 | { |
Vikash Chandola | 1f84797 | 2022-09-28 09:47:32 +0000 | [diff] [blame] | 152 | if (!initCommand) |
James Feist | f7e2c5d | 2019-02-13 17:27:51 -0800 | [diff] [blame] | 153 | { |
Vikash Chandola | 1f84797 | 2022-09-28 09:47:32 +0000 | [diff] [blame] | 154 | return; |
James Feist | 6ef2040 | 2019-01-07 16:45:08 -0800 | [diff] [blame] | 155 | } |
Vikash Chandola | 1f84797 | 2022-09-28 09:47:32 +0000 | [diff] [blame] | 156 | dbusConnection->async_method_call( |
| 157 | [weakRef{weak_from_this()}](const boost::system::error_code& ec, |
| 158 | const IpmbMethodType& response) { |
| 159 | initCmdCb(weakRef, ec, response); |
Patrick Williams | 597e842 | 2023-10-20 11:19:01 -0500 | [diff] [blame] | 160 | }, |
Vikash Chandola | 1f84797 | 2022-09-28 09:47:32 +0000 | [diff] [blame] | 161 | "xyz.openbmc_project.Ipmi.Channel.Ipmb", |
| 162 | "/xyz/openbmc_project/Ipmi/Channel/Ipmb", "org.openbmc.Ipmb", |
| 163 | "sendRequest", commandAddress, netfn, lun, *initCommand, initData); |
James Feist | 6ef2040 | 2019-01-07 16:45:08 -0800 | [diff] [blame] | 164 | } |
| 165 | |
| 166 | void IpmbSensor::loadDefaults() |
| 167 | { |
| 168 | if (type == IpmbType::meSensor) |
| 169 | { |
| 170 | commandAddress = meAddress; |
Adrian Ambrożewicz | 58e02ef | 2020-08-06 14:42:38 +0200 | [diff] [blame] | 171 | netfn = ipmi::sensor::netFn; |
| 172 | command = ipmi::sensor::getSensorReading; |
James Feist | 6ef2040 | 2019-01-07 16:45:08 -0800 | [diff] [blame] | 173 | commandData = {deviceAddress}; |
James Feist | d7ae29a | 2020-06-25 15:42:46 -0700 | [diff] [blame] | 174 | readingFormat = ReadingFormat::byte0; |
James Feist | 6ef2040 | 2019-01-07 16:45:08 -0800 | [diff] [blame] | 175 | } |
| 176 | else if (type == IpmbType::PXE1410CVR) |
| 177 | { |
| 178 | commandAddress = meAddress; |
Adrian Ambrożewicz | 58e02ef | 2020-08-06 14:42:38 +0200 | [diff] [blame] | 179 | netfn = ipmi::me_bridge::netFn; |
| 180 | command = ipmi::me_bridge::sendRawPmbus; |
| 181 | initCommand = ipmi::me_bridge::sendRawPmbus; |
James Feist | d7ae29a | 2020-06-25 15:42:46 -0700 | [diff] [blame] | 182 | // pmbus read temp |
Anoop S | 832a2c6 | 2020-11-20 19:21:22 +0000 | [diff] [blame] | 183 | commandData = {0x57, 0x01, 0x00, 0x16, hostSMbusIndex, |
| 184 | deviceAddress, 0x00, 0x00, 0x00, 0x00, |
| 185 | 0x01, 0x02, 0x8d}; |
James Feist | d7ae29a | 2020-06-25 15:42:46 -0700 | [diff] [blame] | 186 | // goto page 0 |
Anoop S | 832a2c6 | 2020-11-20 19:21:22 +0000 | [diff] [blame] | 187 | initData = {0x57, 0x01, 0x00, 0x14, hostSMbusIndex, |
| 188 | deviceAddress, 0x00, 0x00, 0x00, 0x00, |
| 189 | 0x02, 0x00, 0x00, 0x00}; |
Jayashree-D | 3732257 | 2021-03-19 17:40:56 +0530 | [diff] [blame] | 190 | readingFormat = ReadingFormat::linearElevenBit; |
James Feist | 6ef2040 | 2019-01-07 16:45:08 -0800 | [diff] [blame] | 191 | } |
| 192 | else if (type == IpmbType::IR38363VR) |
| 193 | { |
| 194 | commandAddress = meAddress; |
Adrian Ambrożewicz | 58e02ef | 2020-08-06 14:42:38 +0200 | [diff] [blame] | 195 | netfn = ipmi::me_bridge::netFn; |
| 196 | command = ipmi::me_bridge::sendRawPmbus; |
James Feist | d7ae29a | 2020-06-25 15:42:46 -0700 | [diff] [blame] | 197 | // pmbus read temp |
Anoop S | 832a2c6 | 2020-11-20 19:21:22 +0000 | [diff] [blame] | 198 | commandData = {0x57, 0x01, 0x00, 0x16, hostSMbusIndex, |
| 199 | deviceAddress, 00, 0x00, 0x00, 0x00, |
| 200 | 0x01, 0x02, 0x8D}; |
James Feist | d7ae29a | 2020-06-25 15:42:46 -0700 | [diff] [blame] | 201 | readingFormat = ReadingFormat::elevenBitShift; |
James Feist | 6ef2040 | 2019-01-07 16:45:08 -0800 | [diff] [blame] | 202 | } |
Vijay Khemka | 682a5cb | 2019-07-18 17:34:03 -0700 | [diff] [blame] | 203 | else if (type == IpmbType::ADM1278HSC) |
| 204 | { |
| 205 | commandAddress = meAddress; |
Ed Tanous | a771f6a | 2022-01-14 09:36:51 -0800 | [diff] [blame] | 206 | uint8_t snsNum = 0; |
Vijay Khemka | 682a5cb | 2019-07-18 17:34:03 -0700 | [diff] [blame] | 207 | switch (subType) |
| 208 | { |
| 209 | case IpmbSubType::temp: |
| 210 | case IpmbSubType::curr: |
Vijay Khemka | 682a5cb | 2019-07-18 17:34:03 -0700 | [diff] [blame] | 211 | if (subType == IpmbSubType::temp) |
Ed Tanous | 8a57ec0 | 2020-10-09 12:46:52 -0700 | [diff] [blame] | 212 | { |
Vijay Khemka | 682a5cb | 2019-07-18 17:34:03 -0700 | [diff] [blame] | 213 | snsNum = 0x8d; |
Ed Tanous | 8a57ec0 | 2020-10-09 12:46:52 -0700 | [diff] [blame] | 214 | } |
Vijay Khemka | 682a5cb | 2019-07-18 17:34:03 -0700 | [diff] [blame] | 215 | else |
Ed Tanous | 8a57ec0 | 2020-10-09 12:46:52 -0700 | [diff] [blame] | 216 | { |
Vijay Khemka | 682a5cb | 2019-07-18 17:34:03 -0700 | [diff] [blame] | 217 | snsNum = 0x8c; |
Ed Tanous | 8a57ec0 | 2020-10-09 12:46:52 -0700 | [diff] [blame] | 218 | } |
Adrian Ambrożewicz | 58e02ef | 2020-08-06 14:42:38 +0200 | [diff] [blame] | 219 | netfn = ipmi::me_bridge::netFn; |
| 220 | command = ipmi::me_bridge::sendRawPmbus; |
Vijay Khemka | 682a5cb | 2019-07-18 17:34:03 -0700 | [diff] [blame] | 221 | commandData = {0x57, 0x01, 0x00, 0x86, deviceAddress, |
| 222 | 0x00, 0x00, 0x01, 0x02, snsNum}; |
James Feist | d7ae29a | 2020-06-25 15:42:46 -0700 | [diff] [blame] | 223 | readingFormat = ReadingFormat::elevenBit; |
Vijay Khemka | 682a5cb | 2019-07-18 17:34:03 -0700 | [diff] [blame] | 224 | break; |
| 225 | case IpmbSubType::power: |
| 226 | case IpmbSubType::volt: |
Adrian Ambrożewicz | 58e02ef | 2020-08-06 14:42:38 +0200 | [diff] [blame] | 227 | netfn = ipmi::sensor::netFn; |
| 228 | command = ipmi::sensor::getSensorReading; |
Vijay Khemka | 682a5cb | 2019-07-18 17:34:03 -0700 | [diff] [blame] | 229 | commandData = {deviceAddress}; |
James Feist | d7ae29a | 2020-06-25 15:42:46 -0700 | [diff] [blame] | 230 | readingFormat = ReadingFormat::byte0; |
Vijay Khemka | 682a5cb | 2019-07-18 17:34:03 -0700 | [diff] [blame] | 231 | break; |
| 232 | default: |
| 233 | throw std::runtime_error("Invalid sensor type"); |
| 234 | } |
| 235 | } |
James Feist | 6ef2040 | 2019-01-07 16:45:08 -0800 | [diff] [blame] | 236 | else if (type == IpmbType::mpsVR) |
| 237 | { |
| 238 | commandAddress = meAddress; |
Adrian Ambrożewicz | 58e02ef | 2020-08-06 14:42:38 +0200 | [diff] [blame] | 239 | netfn = ipmi::me_bridge::netFn; |
| 240 | command = ipmi::me_bridge::sendRawPmbus; |
| 241 | initCommand = ipmi::me_bridge::sendRawPmbus; |
James Feist | d7ae29a | 2020-06-25 15:42:46 -0700 | [diff] [blame] | 242 | // pmbus read temp |
Anoop S | 832a2c6 | 2020-11-20 19:21:22 +0000 | [diff] [blame] | 243 | commandData = {0x57, 0x01, 0x00, 0x16, hostSMbusIndex, |
| 244 | deviceAddress, 0x00, 0x00, 0x00, 0x00, |
| 245 | 0x01, 0x02, 0x8d}; |
James Feist | d7ae29a | 2020-06-25 15:42:46 -0700 | [diff] [blame] | 246 | // goto page 0 |
Anoop S | 832a2c6 | 2020-11-20 19:21:22 +0000 | [diff] [blame] | 247 | initData = {0x57, 0x01, 0x00, 0x14, hostSMbusIndex, |
| 248 | deviceAddress, 0x00, 0x00, 0x00, 0x00, |
| 249 | 0x02, 0x00, 0x00, 0x00}; |
James Feist | d7ae29a | 2020-06-25 15:42:46 -0700 | [diff] [blame] | 250 | readingFormat = ReadingFormat::byte3; |
James Feist | 6ef2040 | 2019-01-07 16:45:08 -0800 | [diff] [blame] | 251 | } |
Rebecca Cran | 394f0c5 | 2023-12-17 20:08:55 -0700 | [diff] [blame] | 252 | else if (type == IpmbType::SMPro) |
| 253 | { |
| 254 | // This is an Ampere SMPro reachable via a BMC. For example, |
| 255 | // this architecture is used on ADLINK Ampere Altra systems. |
| 256 | // See the Ampere Family SoC BMC Interface Specification at |
| 257 | // https://amperecomputing.com/customer-connect/products/altra-family-software---firmware |
| 258 | // for details of the sensors. |
| 259 | commandAddress = 0; |
| 260 | netfn = 0x30; |
| 261 | command = 0x31; |
| 262 | commandData = {0x9e, deviceAddress}; |
| 263 | switch (subType) |
| 264 | { |
| 265 | case IpmbSubType::temp: |
| 266 | readingFormat = ReadingFormat::nineBit; |
| 267 | break; |
| 268 | case IpmbSubType::power: |
| 269 | readingFormat = ReadingFormat::tenBit; |
| 270 | break; |
| 271 | case IpmbSubType::curr: |
| 272 | case IpmbSubType::volt: |
| 273 | readingFormat = ReadingFormat::fifteenBit; |
| 274 | break; |
| 275 | default: |
| 276 | throw std::runtime_error("Invalid sensor type"); |
| 277 | } |
| 278 | } |
James Feist | 6ef2040 | 2019-01-07 16:45:08 -0800 | [diff] [blame] | 279 | else |
| 280 | { |
| 281 | throw std::runtime_error("Invalid sensor type"); |
| 282 | } |
Adrian Ambrożewicz | 45e9277 | 2020-06-04 13:59:55 +0200 | [diff] [blame] | 283 | |
| 284 | if (subType == IpmbSubType::util) |
| 285 | { |
| 286 | // Utilization need to be scaled to percent |
| 287 | maxValue = 100; |
| 288 | minValue = 0; |
| 289 | } |
James Feist | 6ef2040 | 2019-01-07 16:45:08 -0800 | [diff] [blame] | 290 | } |
| 291 | |
Ed Tanous | 201a101 | 2024-04-03 18:07:28 -0700 | [diff] [blame] | 292 | void IpmbSensor::checkThresholds() |
James Feist | 6ef2040 | 2019-01-07 16:45:08 -0800 | [diff] [blame] | 293 | { |
James Feist | 6ef2040 | 2019-01-07 16:45:08 -0800 | [diff] [blame] | 294 | thresholds::checkThresholds(this); |
| 295 | } |
| 296 | |
Ed Tanous | 828c5a6 | 2024-02-09 16:59:22 -0800 | [diff] [blame] | 297 | bool IpmbSensor::processReading(ReadingFormat readingFormat, uint8_t command, |
| 298 | const std::vector<uint8_t>& data, double& resp, |
| 299 | size_t errCount) |
James Feist | d7ae29a | 2020-06-25 15:42:46 -0700 | [diff] [blame] | 300 | { |
James Feist | d7ae29a | 2020-06-25 15:42:46 -0700 | [diff] [blame] | 301 | switch (readingFormat) |
| 302 | { |
| 303 | case (ReadingFormat::byte0): |
James Feist | e4a970d | 2020-08-19 11:21:58 -0700 | [diff] [blame] | 304 | { |
Adrian Ambrożewicz | 58e02ef | 2020-08-06 14:42:38 +0200 | [diff] [blame] | 305 | if (command == ipmi::sensor::getSensorReading && |
| 306 | !ipmi::sensor::isValid(data)) |
James Feist | cf4238e | 2020-07-28 16:40:03 -0700 | [diff] [blame] | 307 | { |
| 308 | return false; |
| 309 | } |
James Feist | 961bf09 | 2020-07-01 16:38:12 -0700 | [diff] [blame] | 310 | resp = data[0]; |
| 311 | return true; |
James Feist | e4a970d | 2020-08-19 11:21:58 -0700 | [diff] [blame] | 312 | } |
James Feist | d7ae29a | 2020-06-25 15:42:46 -0700 | [diff] [blame] | 313 | case (ReadingFormat::byte3): |
James Feist | e4a970d | 2020-08-19 11:21:58 -0700 | [diff] [blame] | 314 | { |
James Feist | d7ae29a | 2020-06-25 15:42:46 -0700 | [diff] [blame] | 315 | if (data.size() < 4) |
| 316 | { |
Ed Tanous | 2049bd2 | 2022-07-09 07:20:26 -0700 | [diff] [blame] | 317 | if (errCount == 0U) |
James Feist | 961bf09 | 2020-07-01 16:38:12 -0700 | [diff] [blame] | 318 | { |
Ed Tanous | 828c5a6 | 2024-02-09 16:59:22 -0800 | [diff] [blame] | 319 | std::cerr << "Invalid data length returned\n"; |
James Feist | 961bf09 | 2020-07-01 16:38:12 -0700 | [diff] [blame] | 320 | } |
| 321 | return false; |
James Feist | d7ae29a | 2020-06-25 15:42:46 -0700 | [diff] [blame] | 322 | } |
James Feist | 961bf09 | 2020-07-01 16:38:12 -0700 | [diff] [blame] | 323 | resp = data[3]; |
| 324 | return true; |
James Feist | e4a970d | 2020-08-19 11:21:58 -0700 | [diff] [blame] | 325 | } |
Rebecca Cran | 394f0c5 | 2023-12-17 20:08:55 -0700 | [diff] [blame] | 326 | case (ReadingFormat::nineBit): |
| 327 | case (ReadingFormat::tenBit): |
| 328 | case (ReadingFormat::fifteenBit): |
| 329 | { |
| 330 | if (data.size() != 2) |
| 331 | { |
| 332 | if (errCount == 0U) |
| 333 | { |
| 334 | std::cerr << "Invalid data length returned\n"; |
| 335 | } |
| 336 | return false; |
| 337 | } |
| 338 | |
| 339 | // From the Altra Family SoC BMC Interface Specification: |
| 340 | // 0xFFFF – This sensor data is either missing or is not supported |
| 341 | // by the device. |
| 342 | if ((data[0] == 0xff) && (data[1] == 0xff)) |
| 343 | { |
| 344 | return false; |
| 345 | } |
| 346 | |
| 347 | if (readingFormat == ReadingFormat::nineBit) |
| 348 | { |
| 349 | int16_t value = data[0]; |
| 350 | if ((data[1] & 0x1) != 0) |
| 351 | { |
| 352 | // Sign extend to 16 bits |
| 353 | value |= 0xFF00; |
| 354 | } |
| 355 | resp = value; |
| 356 | } |
| 357 | else if (readingFormat == ReadingFormat::tenBit) |
| 358 | { |
| 359 | uint16_t value = ((data[1] & 0x3) << 8) + data[0]; |
| 360 | resp = value; |
| 361 | } |
| 362 | else if (readingFormat == ReadingFormat::fifteenBit) |
| 363 | { |
| 364 | uint16_t value = ((data[1] & 0x7F) << 8) + data[0]; |
| 365 | // Convert mV to V |
| 366 | resp = value / 1000.0; |
| 367 | } |
| 368 | |
| 369 | return true; |
| 370 | } |
James Feist | d7ae29a | 2020-06-25 15:42:46 -0700 | [diff] [blame] | 371 | case (ReadingFormat::elevenBit): |
James Feist | e4a970d | 2020-08-19 11:21:58 -0700 | [diff] [blame] | 372 | { |
James Feist | d7ae29a | 2020-06-25 15:42:46 -0700 | [diff] [blame] | 373 | if (data.size() < 5) |
| 374 | { |
Ed Tanous | 2049bd2 | 2022-07-09 07:20:26 -0700 | [diff] [blame] | 375 | if (errCount == 0U) |
James Feist | 961bf09 | 2020-07-01 16:38:12 -0700 | [diff] [blame] | 376 | { |
Ed Tanous | 828c5a6 | 2024-02-09 16:59:22 -0800 | [diff] [blame] | 377 | std::cerr << "Invalid data length returned\n"; |
James Feist | 961bf09 | 2020-07-01 16:38:12 -0700 | [diff] [blame] | 378 | } |
| 379 | return false; |
James Feist | d7ae29a | 2020-06-25 15:42:46 -0700 | [diff] [blame] | 380 | } |
| 381 | |
James Feist | e4a970d | 2020-08-19 11:21:58 -0700 | [diff] [blame] | 382 | int16_t value = ((data[4] << 8) | data[3]); |
James Feist | e4a970d | 2020-08-19 11:21:58 -0700 | [diff] [blame] | 383 | resp = value; |
James Feist | 961bf09 | 2020-07-01 16:38:12 -0700 | [diff] [blame] | 384 | return true; |
James Feist | e4a970d | 2020-08-19 11:21:58 -0700 | [diff] [blame] | 385 | } |
James Feist | d7ae29a | 2020-06-25 15:42:46 -0700 | [diff] [blame] | 386 | case (ReadingFormat::elevenBitShift): |
James Feist | e4a970d | 2020-08-19 11:21:58 -0700 | [diff] [blame] | 387 | { |
James Feist | d7ae29a | 2020-06-25 15:42:46 -0700 | [diff] [blame] | 388 | if (data.size() < 5) |
| 389 | { |
Ed Tanous | 2049bd2 | 2022-07-09 07:20:26 -0700 | [diff] [blame] | 390 | if (errCount == 0U) |
James Feist | 961bf09 | 2020-07-01 16:38:12 -0700 | [diff] [blame] | 391 | { |
Ed Tanous | 828c5a6 | 2024-02-09 16:59:22 -0800 | [diff] [blame] | 392 | std::cerr << "Invalid data length returned\n"; |
James Feist | 961bf09 | 2020-07-01 16:38:12 -0700 | [diff] [blame] | 393 | } |
| 394 | return false; |
James Feist | d7ae29a | 2020-06-25 15:42:46 -0700 | [diff] [blame] | 395 | } |
| 396 | |
James Feist | 961bf09 | 2020-07-01 16:38:12 -0700 | [diff] [blame] | 397 | resp = ((data[4] << 8) | data[3]) >> 3; |
| 398 | return true; |
James Feist | e4a970d | 2020-08-19 11:21:58 -0700 | [diff] [blame] | 399 | } |
Jayashree-D | 3732257 | 2021-03-19 17:40:56 +0530 | [diff] [blame] | 400 | case (ReadingFormat::linearElevenBit): |
| 401 | { |
| 402 | if (data.size() < 5) |
| 403 | { |
Ed Tanous | 2049bd2 | 2022-07-09 07:20:26 -0700 | [diff] [blame] | 404 | if (errCount == 0U) |
Jayashree-D | 3732257 | 2021-03-19 17:40:56 +0530 | [diff] [blame] | 405 | { |
Ed Tanous | 828c5a6 | 2024-02-09 16:59:22 -0800 | [diff] [blame] | 406 | std::cerr << "Invalid data length returned\n"; |
Jayashree-D | 3732257 | 2021-03-19 17:40:56 +0530 | [diff] [blame] | 407 | } |
| 408 | return false; |
| 409 | } |
| 410 | |
| 411 | int16_t value = ((data[4] << 8) | data[3]); |
| 412 | constexpr const size_t shift = 16 - 11; // 11bit into 16bit |
| 413 | value <<= shift; |
| 414 | value >>= shift; |
| 415 | resp = value; |
| 416 | return true; |
| 417 | } |
James Feist | d7ae29a | 2020-06-25 15:42:46 -0700 | [diff] [blame] | 418 | default: |
| 419 | throw std::runtime_error("Invalid reading type"); |
| 420 | } |
| 421 | } |
| 422 | |
Vikash Chandola | 1f84797 | 2022-09-28 09:47:32 +0000 | [diff] [blame] | 423 | void IpmbSensor::ipmbRequestCompletionCb(const boost::system::error_code& ec, |
| 424 | const IpmbMethodType& response) |
| 425 | { |
| 426 | const int& status = std::get<0>(response); |
| 427 | if (ec || (status != 0)) |
| 428 | { |
| 429 | incrementError(); |
| 430 | read(); |
| 431 | return; |
| 432 | } |
| 433 | const std::vector<uint8_t>& data = std::get<5>(response); |
| 434 | if constexpr (debug) |
| 435 | { |
| 436 | std::cout << name << ": "; |
| 437 | for (size_t d : data) |
| 438 | { |
| 439 | std::cout << d << " "; |
| 440 | } |
| 441 | std::cout << "\n"; |
| 442 | } |
| 443 | if (data.empty()) |
| 444 | { |
| 445 | incrementError(); |
| 446 | read(); |
| 447 | return; |
| 448 | } |
| 449 | |
| 450 | double value = 0; |
| 451 | |
Ed Tanous | 828c5a6 | 2024-02-09 16:59:22 -0800 | [diff] [blame] | 452 | if (!processReading(readingFormat, command, data, value, errCount)) |
Vikash Chandola | 1f84797 | 2022-09-28 09:47:32 +0000 | [diff] [blame] | 453 | { |
| 454 | incrementError(); |
| 455 | read(); |
| 456 | return; |
| 457 | } |
| 458 | |
| 459 | // rawValue only used in debug logging |
| 460 | // up to 5th byte in data are used to derive value |
| 461 | size_t end = std::min(sizeof(uint64_t), data.size()); |
| 462 | uint64_t rawData = 0; |
| 463 | for (size_t i = 0; i < end; i++) |
| 464 | { |
| 465 | // NOLINTNEXTLINE(cppcoreguidelines-pro-type-reinterpret-cast) |
| 466 | reinterpret_cast<uint8_t*>(&rawData)[i] = data[i]; |
| 467 | } |
| 468 | rawValue = static_cast<double>(rawData); |
| 469 | |
| 470 | /* Adjust value as per scale and offset */ |
| 471 | value = (value * scaleVal) + offsetVal; |
| 472 | updateValue(value); |
| 473 | read(); |
| 474 | } |
| 475 | |
Ed Tanous | 201a101 | 2024-04-03 18:07:28 -0700 | [diff] [blame] | 476 | void IpmbSensor::read() |
James Feist | 6ef2040 | 2019-01-07 16:45:08 -0800 | [diff] [blame] | 477 | { |
Ed Tanous | 83db50c | 2023-03-01 10:20:24 -0800 | [diff] [blame] | 478 | waitTimer.expires_after(std::chrono::milliseconds(sensorPollMs)); |
Vikash Chandola | 1f84797 | 2022-09-28 09:47:32 +0000 | [diff] [blame] | 479 | waitTimer.async_wait( |
| 480 | [weakRef{weak_from_this()}](const boost::system::error_code& ec) { |
James Feist | 6ef2040 | 2019-01-07 16:45:08 -0800 | [diff] [blame] | 481 | if (ec == boost::asio::error::operation_aborted) |
| 482 | { |
| 483 | return; // we're being canceled |
| 484 | } |
Vikash Chandola | 1f84797 | 2022-09-28 09:47:32 +0000 | [diff] [blame] | 485 | std::shared_ptr<IpmbSensor> self = weakRef.lock(); |
| 486 | if (!self) |
James Feist | 6ef2040 | 2019-01-07 16:45:08 -0800 | [diff] [blame] | 487 | { |
James Feist | 6ef2040 | 2019-01-07 16:45:08 -0800 | [diff] [blame] | 488 | return; |
| 489 | } |
Vikash Chandola | 1f84797 | 2022-09-28 09:47:32 +0000 | [diff] [blame] | 490 | self->sendIpmbRequest(); |
James Feist | 6ef2040 | 2019-01-07 16:45:08 -0800 | [diff] [blame] | 491 | }); |
| 492 | } |
Jayashree Dhanapal | 8418975 | 2022-03-07 12:51:54 +0530 | [diff] [blame] | 493 | |
Vikash Chandola | 1f84797 | 2022-09-28 09:47:32 +0000 | [diff] [blame] | 494 | void IpmbSensor::sendIpmbRequest() |
| 495 | { |
| 496 | if (!readingStateGood()) |
| 497 | { |
| 498 | updateValue(std::numeric_limits<double>::quiet_NaN()); |
| 499 | read(); |
| 500 | return; |
| 501 | } |
| 502 | dbusConnection->async_method_call( |
| 503 | [weakRef{weak_from_this()}](boost::system::error_code ec, |
| 504 | const IpmbMethodType& response) { |
| 505 | std::shared_ptr<IpmbSensor> self = weakRef.lock(); |
| 506 | if (!self) |
| 507 | { |
| 508 | return; |
| 509 | } |
| 510 | self->ipmbRequestCompletionCb(ec, response); |
Patrick Williams | 597e842 | 2023-10-20 11:19:01 -0500 | [diff] [blame] | 511 | }, |
Vikash Chandola | 1f84797 | 2022-09-28 09:47:32 +0000 | [diff] [blame] | 512 | "xyz.openbmc_project.Ipmi.Channel.Ipmb", |
| 513 | "/xyz/openbmc_project/Ipmi/Channel/Ipmb", "org.openbmc.Ipmb", |
| 514 | "sendRequest", commandAddress, netfn, lun, command, commandData); |
| 515 | } |
| 516 | |
Jayashree Dhanapal | 8418975 | 2022-03-07 12:51:54 +0530 | [diff] [blame] | 517 | bool IpmbSensor::sensorClassType(const std::string& sensorClass) |
| 518 | { |
| 519 | if (sensorClass == "PxeBridgeTemp") |
| 520 | { |
| 521 | type = IpmbType::PXE1410CVR; |
| 522 | } |
| 523 | else if (sensorClass == "IRBridgeTemp") |
| 524 | { |
| 525 | type = IpmbType::IR38363VR; |
| 526 | } |
| 527 | else if (sensorClass == "HSCBridge") |
| 528 | { |
| 529 | type = IpmbType::ADM1278HSC; |
| 530 | } |
| 531 | else if (sensorClass == "MpsBridgeTemp") |
| 532 | { |
| 533 | type = IpmbType::mpsVR; |
| 534 | } |
| 535 | else if (sensorClass == "METemp" || sensorClass == "MESensor") |
| 536 | { |
| 537 | type = IpmbType::meSensor; |
| 538 | } |
Rebecca Cran | 394f0c5 | 2023-12-17 20:08:55 -0700 | [diff] [blame] | 539 | else if (sensorClass == "SMPro") |
| 540 | { |
| 541 | type = IpmbType::SMPro; |
| 542 | } |
Jayashree Dhanapal | 8418975 | 2022-03-07 12:51:54 +0530 | [diff] [blame] | 543 | else |
| 544 | { |
| 545 | std::cerr << "Invalid class " << sensorClass << "\n"; |
| 546 | return false; |
| 547 | } |
| 548 | return true; |
| 549 | } |
| 550 | |
| 551 | void IpmbSensor::sensorSubType(const std::string& sensorTypeName) |
| 552 | { |
| 553 | if (sensorTypeName == "voltage") |
| 554 | { |
| 555 | subType = IpmbSubType::volt; |
| 556 | } |
| 557 | else if (sensorTypeName == "power") |
| 558 | { |
| 559 | subType = IpmbSubType::power; |
| 560 | } |
| 561 | else if (sensorTypeName == "current") |
| 562 | { |
| 563 | subType = IpmbSubType::curr; |
| 564 | } |
| 565 | else if (sensorTypeName == "utilization") |
| 566 | { |
| 567 | subType = IpmbSubType::util; |
| 568 | } |
| 569 | else |
| 570 | { |
| 571 | subType = IpmbSubType::temp; |
| 572 | } |
| 573 | } |
| 574 | |
| 575 | void IpmbSensor::parseConfigValues(const SensorBaseConfigMap& entry) |
| 576 | { |
| 577 | auto findScaleVal = entry.find("ScaleValue"); |
| 578 | if (findScaleVal != entry.end()) |
| 579 | { |
| 580 | scaleVal = std::visit(VariantToDoubleVisitor(), findScaleVal->second); |
| 581 | } |
| 582 | |
| 583 | auto findOffsetVal = entry.find("OffsetValue"); |
| 584 | if (findOffsetVal != entry.end()) |
| 585 | { |
| 586 | offsetVal = std::visit(VariantToDoubleVisitor(), findOffsetVal->second); |
| 587 | } |
| 588 | |
Zev Weiss | a4d2768 | 2022-07-19 15:30:36 -0700 | [diff] [blame] | 589 | readState = getPowerState(entry); |
Jayashree Dhanapal | 8418975 | 2022-03-07 12:51:54 +0530 | [diff] [blame] | 590 | } |
| 591 | |
James Feist | 6ef2040 | 2019-01-07 16:45:08 -0800 | [diff] [blame] | 592 | void createSensors( |
Ed Tanous | 1f97863 | 2023-02-28 18:16:39 -0800 | [diff] [blame] | 593 | boost::asio::io_context& io, sdbusplus::asio::object_server& objectServer, |
Vikash Chandola | 1f84797 | 2022-09-28 09:47:32 +0000 | [diff] [blame] | 594 | boost::container::flat_map<std::string, std::shared_ptr<IpmbSensor>>& |
James Feist | 6ef2040 | 2019-01-07 16:45:08 -0800 | [diff] [blame] | 595 | sensors, |
| 596 | std::shared_ptr<sdbusplus::asio::connection>& dbusConnection) |
| 597 | { |
| 598 | if (!dbusConnection) |
| 599 | { |
| 600 | std::cerr << "Connection not created\n"; |
| 601 | return; |
| 602 | } |
| 603 | dbusConnection->async_method_call( |
| 604 | [&](boost::system::error_code ec, const ManagedObjectType& resp) { |
Ed Tanous | bb67932 | 2022-05-16 16:10:00 -0700 | [diff] [blame] | 605 | if (ec) |
| 606 | { |
| 607 | std::cerr << "Error contacting entity manager\n"; |
| 608 | return; |
| 609 | } |
Zev Weiss | 8ba551b | 2022-08-12 18:21:02 -0700 | [diff] [blame] | 610 | for (const auto& [path, interfaces] : resp) |
Ed Tanous | bb67932 | 2022-05-16 16:10:00 -0700 | [diff] [blame] | 611 | { |
Zev Weiss | 8ba551b | 2022-08-12 18:21:02 -0700 | [diff] [blame] | 612 | for (const auto& [intf, cfg] : interfaces) |
James Feist | 6ef2040 | 2019-01-07 16:45:08 -0800 | [diff] [blame] | 613 | { |
Zev Weiss | 054aad8 | 2022-08-18 01:37:34 -0700 | [diff] [blame] | 614 | if (intf != configInterfaceName(sensorType)) |
James Feist | 6ef2040 | 2019-01-07 16:45:08 -0800 | [diff] [blame] | 615 | { |
Ed Tanous | bb67932 | 2022-05-16 16:10:00 -0700 | [diff] [blame] | 616 | continue; |
James Feist | 6ef2040 | 2019-01-07 16:45:08 -0800 | [diff] [blame] | 617 | } |
Zev Weiss | 8ba551b | 2022-08-12 18:21:02 -0700 | [diff] [blame] | 618 | std::string name = loadVariant<std::string>(cfg, "Name"); |
Ed Tanous | bb67932 | 2022-05-16 16:10:00 -0700 | [diff] [blame] | 619 | |
| 620 | std::vector<thresholds::Threshold> sensorThresholds; |
Zev Weiss | 8ba551b | 2022-08-12 18:21:02 -0700 | [diff] [blame] | 621 | if (!parseThresholdsFromConfig(interfaces, sensorThresholds)) |
Ed Tanous | bb67932 | 2022-05-16 16:10:00 -0700 | [diff] [blame] | 622 | { |
Ed Tanous | 828c5a6 | 2024-02-09 16:59:22 -0800 | [diff] [blame] | 623 | std::cerr << "error populating thresholds " << name << "\n"; |
Ed Tanous | bb67932 | 2022-05-16 16:10:00 -0700 | [diff] [blame] | 624 | } |
Zev Weiss | 8ba551b | 2022-08-12 18:21:02 -0700 | [diff] [blame] | 625 | uint8_t deviceAddress = loadVariant<uint8_t>(cfg, "Address"); |
Ed Tanous | bb67932 | 2022-05-16 16:10:00 -0700 | [diff] [blame] | 626 | |
Patrick Williams | 779c96a | 2023-05-10 07:50:42 -0500 | [diff] [blame] | 627 | std::string sensorClass = loadVariant<std::string>(cfg, |
| 628 | "Class"); |
Ed Tanous | bb67932 | 2022-05-16 16:10:00 -0700 | [diff] [blame] | 629 | |
| 630 | uint8_t hostSMbusIndex = hostSMbusIndexDefault; |
Zev Weiss | 8ba551b | 2022-08-12 18:21:02 -0700 | [diff] [blame] | 631 | auto findSmType = cfg.find("HostSMbusIndex"); |
| 632 | if (findSmType != cfg.end()) |
Ed Tanous | bb67932 | 2022-05-16 16:10:00 -0700 | [diff] [blame] | 633 | { |
| 634 | hostSMbusIndex = std::visit(VariantToUnsignedIntVisitor(), |
| 635 | findSmType->second); |
| 636 | } |
| 637 | |
Zev Weiss | 8569bf2 | 2022-10-11 15:37:44 -0700 | [diff] [blame] | 638 | float pollRate = getPollRate(cfg, pollRateDefault); |
Ed Tanous | bb67932 | 2022-05-16 16:10:00 -0700 | [diff] [blame] | 639 | |
Jayashree Dhanapal | 6ee6294 | 2021-12-14 15:22:23 +0530 | [diff] [blame] | 640 | uint8_t ipmbBusIndex = ipmbBusIndexDefault; |
| 641 | auto findBusType = cfg.find("Bus"); |
| 642 | if (findBusType != cfg.end()) |
| 643 | { |
| 644 | ipmbBusIndex = std::visit(VariantToUnsignedIntVisitor(), |
| 645 | findBusType->second); |
| 646 | std::cerr << "Ipmb Bus Index for " << name << " is " |
| 647 | << static_cast<int>(ipmbBusIndex) << "\n"; |
| 648 | } |
| 649 | |
Ed Tanous | bb67932 | 2022-05-16 16:10:00 -0700 | [diff] [blame] | 650 | /* Default sensor type is "temperature" */ |
| 651 | std::string sensorTypeName = "temperature"; |
Zev Weiss | 8ba551b | 2022-08-12 18:21:02 -0700 | [diff] [blame] | 652 | auto findType = cfg.find("SensorType"); |
| 653 | if (findType != cfg.end()) |
Ed Tanous | bb67932 | 2022-05-16 16:10:00 -0700 | [diff] [blame] | 654 | { |
Patrick Williams | 779c96a | 2023-05-10 07:50:42 -0500 | [diff] [blame] | 655 | sensorTypeName = std::visit(VariantToStringVisitor(), |
| 656 | findType->second); |
Ed Tanous | bb67932 | 2022-05-16 16:10:00 -0700 | [diff] [blame] | 657 | } |
| 658 | |
| 659 | auto& sensor = sensors[name]; |
Vikash Chandola | 1f84797 | 2022-09-28 09:47:32 +0000 | [diff] [blame] | 660 | sensor = nullptr; |
| 661 | sensor = std::make_shared<IpmbSensor>( |
Zev Weiss | 8ba551b | 2022-08-12 18:21:02 -0700 | [diff] [blame] | 662 | dbusConnection, io, name, path, objectServer, |
Ed Tanous | bb67932 | 2022-05-16 16:10:00 -0700 | [diff] [blame] | 663 | std::move(sensorThresholds), deviceAddress, hostSMbusIndex, |
| 664 | pollRate, sensorTypeName); |
| 665 | |
Zev Weiss | 8ba551b | 2022-08-12 18:21:02 -0700 | [diff] [blame] | 666 | sensor->parseConfigValues(cfg); |
Ed Tanous | bb67932 | 2022-05-16 16:10:00 -0700 | [diff] [blame] | 667 | if (!(sensor->sensorClassType(sensorClass))) |
| 668 | { |
| 669 | continue; |
| 670 | } |
| 671 | sensor->sensorSubType(sensorTypeName); |
| 672 | sensor->init(); |
James Feist | 6ef2040 | 2019-01-07 16:45:08 -0800 | [diff] [blame] | 673 | } |
Ed Tanous | bb67932 | 2022-05-16 16:10:00 -0700 | [diff] [blame] | 674 | } |
Patrick Williams | 597e842 | 2023-10-20 11:19:01 -0500 | [diff] [blame] | 675 | }, |
JeffLin | 2c5a1f2 | 2022-10-05 15:19:09 +0800 | [diff] [blame] | 676 | entityManagerName, "/xyz/openbmc_project/inventory", |
| 677 | "org.freedesktop.DBus.ObjectManager", "GetManagedObjects"); |
James Feist | 6ef2040 | 2019-01-07 16:45:08 -0800 | [diff] [blame] | 678 | } |
| 679 | |
Kumar Thangavel | 5d032bc | 2022-11-30 20:24:47 +0530 | [diff] [blame] | 680 | void interfaceRemoved( |
| 681 | sdbusplus::message_t& message, |
| 682 | boost::container::flat_map<std::string, std::shared_ptr<IpmbSensor>>& |
| 683 | sensors) |
| 684 | { |
| 685 | if (message.is_method_error()) |
| 686 | { |
| 687 | std::cerr << "interfacesRemoved callback method error\n"; |
| 688 | return; |
| 689 | } |
| 690 | |
| 691 | sdbusplus::message::object_path removedPath; |
| 692 | std::vector<std::string> interfaces; |
| 693 | |
| 694 | message.read(removedPath, interfaces); |
| 695 | |
| 696 | // If the xyz.openbmc_project.Confguration.X interface was removed |
| 697 | // for one or more sensors, delete those sensor objects. |
| 698 | auto sensorIt = sensors.begin(); |
| 699 | while (sensorIt != sensors.end()) |
| 700 | { |
| 701 | if ((sensorIt->second->configurationPath == removedPath) && |
| 702 | (std::find(interfaces.begin(), interfaces.end(), |
| 703 | configInterfaceName(sdrInterface)) != interfaces.end())) |
| 704 | { |
| 705 | sensorIt = sensors.erase(sensorIt); |
| 706 | } |
| 707 | else |
| 708 | { |
| 709 | sensorIt++; |
| 710 | } |
| 711 | } |
| 712 | } |