Patrick Venture | 863b924 | 2018-03-08 08:29:23 -0800 | [diff] [blame] | 1 | /** |
| 2 | * Copyright 2017 Google Inc. |
| 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 | */ |
Patrick Venture | da4a5dd | 2018-08-31 09:42:48 -0700 | [diff] [blame] | 16 | #include "dbuspassive.hpp" |
| 17 | |
Patrick Venture | aadb30d | 2020-08-10 09:17:11 -0700 | [diff] [blame] | 18 | #include "dbushelper_interface.hpp" |
James Feist | 98b704e | 2019-06-03 16:24:53 -0700 | [diff] [blame] | 19 | #include "dbuspassiveredundancy.hpp" |
Patrick Venture | aadb30d | 2020-08-10 09:17:11 -0700 | [diff] [blame] | 20 | #include "dbusutil.hpp" |
James Feist | 0c8223b | 2019-05-08 15:33:33 -0700 | [diff] [blame] | 21 | #include "util.hpp" |
Patrick Venture | da4a5dd | 2018-08-31 09:42:48 -0700 | [diff] [blame] | 22 | |
Patrick Venture | a83a3ec | 2020-08-04 09:52:05 -0700 | [diff] [blame] | 23 | #include <sdbusplus/bus.hpp> |
| 24 | |
Patrick Venture | 863b924 | 2018-03-08 08:29:23 -0800 | [diff] [blame] | 25 | #include <chrono> |
| 26 | #include <cmath> |
Patrick Venture | 0ef1faf | 2018-06-13 12:50:53 -0700 | [diff] [blame] | 27 | #include <memory> |
Patrick Venture | 863b924 | 2018-03-08 08:29:23 -0800 | [diff] [blame] | 28 | #include <mutex> |
Patrick Venture | 0ef1faf | 2018-06-13 12:50:53 -0700 | [diff] [blame] | 29 | #include <string> |
James Feist | 1f802f5 | 2019-02-08 13:51:43 -0800 | [diff] [blame] | 30 | #include <variant> |
Patrick Venture | 863b924 | 2018-03-08 08:29:23 -0800 | [diff] [blame] | 31 | |
Patrick Venture | a076487 | 2020-08-08 07:48:43 -0700 | [diff] [blame] | 32 | namespace pid_control |
| 33 | { |
| 34 | |
Patrick Venture | 563a356 | 2018-10-30 09:31:26 -0700 | [diff] [blame] | 35 | std::unique_ptr<ReadInterface> DbusPassive::createDbusPassive( |
Patrick Venture | da4a5dd | 2018-08-31 09:42:48 -0700 | [diff] [blame] | 36 | sdbusplus::bus::bus& bus, const std::string& type, const std::string& id, |
Patrick Venture | 8729eb9 | 2020-08-10 10:38:44 -0700 | [diff] [blame] | 37 | std::unique_ptr<DbusHelperInterface> helper, const conf::SensorConfig* info, |
James Feist | 98b704e | 2019-06-03 16:24:53 -0700 | [diff] [blame] | 38 | const std::shared_ptr<DbusPassiveRedundancy>& redundancy) |
Patrick Venture | 0ef1faf | 2018-06-13 12:50:53 -0700 | [diff] [blame] | 39 | { |
| 40 | if (helper == nullptr) |
| 41 | { |
| 42 | return nullptr; |
| 43 | } |
Patrick Venture | 7af157b | 2018-10-30 11:24:40 -0700 | [diff] [blame] | 44 | if (!validType(type)) |
Patrick Venture | 0ef1faf | 2018-06-13 12:50:53 -0700 | [diff] [blame] | 45 | { |
| 46 | return nullptr; |
| 47 | } |
| 48 | |
Patrick Venture | 863b924 | 2018-03-08 08:29:23 -0800 | [diff] [blame] | 49 | /* Need to get the scale and initial value */ |
Patrick Venture | 863b924 | 2018-03-08 08:29:23 -0800 | [diff] [blame] | 50 | /* service == busname */ |
Patrick Venture | 7af157b | 2018-10-30 11:24:40 -0700 | [diff] [blame] | 51 | std::string path = getSensorPath(type, id); |
Patrick Venture | 34ddc90 | 2018-10-30 11:05:17 -0700 | [diff] [blame] | 52 | |
Patrick Venture | 1df9e87 | 2020-10-08 15:35:01 -0700 | [diff] [blame] | 53 | SensorProperties settings; |
Patrick Venture | f8cb464 | 2018-10-30 12:02:53 -0700 | [diff] [blame] | 54 | bool failed; |
Patrick Venture | 863b924 | 2018-03-08 08:29:23 -0800 | [diff] [blame] | 55 | |
Patrick Venture | f8cb464 | 2018-10-30 12:02:53 -0700 | [diff] [blame] | 56 | try |
| 57 | { |
Patrick Venture | 9b93692 | 2020-08-10 11:28:39 -0700 | [diff] [blame] | 58 | std::string service = helper->getService(sensorintf, path); |
Patrick Venture | f8cb464 | 2018-10-30 12:02:53 -0700 | [diff] [blame] | 59 | |
Patrick Venture | 9b93692 | 2020-08-10 11:28:39 -0700 | [diff] [blame] | 60 | helper->getProperties(service, path, &settings); |
| 61 | failed = helper->thresholdsAsserted(service, path); |
Patrick Venture | f8cb464 | 2018-10-30 12:02:53 -0700 | [diff] [blame] | 62 | } |
| 63 | catch (const std::exception& e) |
| 64 | { |
| 65 | return nullptr; |
| 66 | } |
| 67 | |
Patrick Venture | 6b9f599 | 2019-09-10 09:18:28 -0700 | [diff] [blame] | 68 | /* if these values are zero, they're ignored. */ |
| 69 | if (info->ignoreDbusMinMax) |
| 70 | { |
| 71 | settings.min = 0; |
| 72 | settings.max = 0; |
| 73 | } |
| 74 | |
Alex.Song | 8f73ad7 | 2021-10-07 00:18:27 +0800 | [diff] [blame] | 75 | settings.unavailableAsFailed = info->unavailableAsFailed; |
| 76 | |
Patrick Venture | 8729eb9 | 2020-08-10 10:38:44 -0700 | [diff] [blame] | 77 | return std::make_unique<DbusPassive>(bus, type, id, std::move(helper), |
| 78 | settings, failed, path, redundancy); |
Patrick Venture | f8cb464 | 2018-10-30 12:02:53 -0700 | [diff] [blame] | 79 | } |
| 80 | |
James Feist | 98b704e | 2019-06-03 16:24:53 -0700 | [diff] [blame] | 81 | DbusPassive::DbusPassive( |
| 82 | sdbusplus::bus::bus& bus, const std::string& type, const std::string& id, |
Patrick Venture | 8729eb9 | 2020-08-10 10:38:44 -0700 | [diff] [blame] | 83 | std::unique_ptr<DbusHelperInterface> helper, |
Patrick Venture | 1df9e87 | 2020-10-08 15:35:01 -0700 | [diff] [blame] | 84 | const SensorProperties& settings, bool failed, const std::string& path, |
James Feist | 98b704e | 2019-06-03 16:24:53 -0700 | [diff] [blame] | 85 | const std::shared_ptr<DbusPassiveRedundancy>& redundancy) : |
Patrick Venture | f8cb464 | 2018-10-30 12:02:53 -0700 | [diff] [blame] | 86 | ReadInterface(), |
Patrick Venture | d057130 | 2020-08-10 12:42:23 -0700 | [diff] [blame] | 87 | _signal(bus, getMatch(type, id).c_str(), dbusHandleSignal, this), _id(id), |
| 88 | _helper(std::move(helper)), _failed(failed), path(path), |
James Feist | 98b704e | 2019-06-03 16:24:53 -0700 | [diff] [blame] | 89 | redundancy(redundancy) |
| 90 | |
Patrick Venture | f8cb464 | 2018-10-30 12:02:53 -0700 | [diff] [blame] | 91 | { |
Patrick Venture | 863b924 | 2018-03-08 08:29:23 -0800 | [diff] [blame] | 92 | _scale = settings.scale; |
Josh Lehan | 3e2f758 | 2020-09-20 22:06:03 -0700 | [diff] [blame] | 93 | _min = settings.min * std::pow(10.0, _scale); |
| 94 | _max = settings.max * std::pow(10.0, _scale); |
Alex.Song | 8f73ad7 | 2021-10-07 00:18:27 +0800 | [diff] [blame] | 95 | _available = settings.available; |
| 96 | _unavailableAsFailed = settings.unavailableAsFailed; |
Josh Lehan | 3e2f758 | 2020-09-20 22:06:03 -0700 | [diff] [blame] | 97 | |
| 98 | // Cache this type knowledge, to avoid repeated string comparison |
| 99 | _typeMargin = (type == "margin"); |
Alex.Song | 8f73ad7 | 2021-10-07 00:18:27 +0800 | [diff] [blame] | 100 | _typeFan = (type == "fan"); |
Josh Lehan | 3e2f758 | 2020-09-20 22:06:03 -0700 | [diff] [blame] | 101 | |
| 102 | // Force value to be stored, otherwise member would be uninitialized |
| 103 | updateValue(settings.value, true); |
Patrick Venture | 863b924 | 2018-03-08 08:29:23 -0800 | [diff] [blame] | 104 | } |
| 105 | |
| 106 | ReadReturn DbusPassive::read(void) |
| 107 | { |
| 108 | std::lock_guard<std::mutex> guard(_lock); |
| 109 | |
Patrick Venture | 1df9e87 | 2020-10-08 15:35:01 -0700 | [diff] [blame] | 110 | ReadReturn r = {_value, _updated}; |
Patrick Venture | 863b924 | 2018-03-08 08:29:23 -0800 | [diff] [blame] | 111 | |
| 112 | return r; |
| 113 | } |
| 114 | |
| 115 | void DbusPassive::setValue(double value) |
| 116 | { |
| 117 | std::lock_guard<std::mutex> guard(_lock); |
| 118 | |
| 119 | _value = value; |
| 120 | _updated = std::chrono::high_resolution_clock::now(); |
| 121 | } |
| 122 | |
James Feist | 36b7d8e | 2018-10-05 15:39:01 -0700 | [diff] [blame] | 123 | bool DbusPassive::getFailed(void) const |
| 124 | { |
James Feist | 98b704e | 2019-06-03 16:24:53 -0700 | [diff] [blame] | 125 | if (redundancy) |
| 126 | { |
| 127 | const std::set<std::string>& failures = redundancy->getFailed(); |
| 128 | if (failures.find(path) != failures.end()) |
| 129 | { |
| 130 | return true; |
| 131 | } |
| 132 | } |
James Feist | 4b36f26 | 2020-07-07 16:56:41 -0700 | [diff] [blame] | 133 | |
Alex.Song | 8f73ad7 | 2021-10-07 00:18:27 +0800 | [diff] [blame] | 134 | /* |
| 135 | * Unavailable thermal sensors, who are not present or |
| 136 | * power-state-not-matching, should not trigger the failSafe mode. For |
| 137 | * example, when a system stays at a powered-off state, its CPU Temp |
| 138 | * sensors will be unavailable, these unavailable sensors should not be |
| 139 | * treated as failed and trigger failSafe. |
| 140 | * This is important for systems whose Fans are always on. |
| 141 | */ |
| 142 | if (!_typeFan && !_available && !_unavailableAsFailed) |
| 143 | { |
| 144 | return false; |
| 145 | } |
| 146 | |
Josh Lehan | 3e2f758 | 2020-09-20 22:06:03 -0700 | [diff] [blame] | 147 | // If a reading has came in, |
| 148 | // but its value bad in some way (determined by sensor type), |
| 149 | // indicate this sensor has failed, |
| 150 | // until another value comes in that is no longer bad. |
| 151 | // This is different from the overall _failed flag, |
| 152 | // which is set and cleared by other causes. |
| 153 | if (_badReading) |
| 154 | { |
| 155 | return true; |
| 156 | } |
| 157 | |
| 158 | // If a reading has came in, and it is not a bad reading, |
| 159 | // but it indicates there is no more thermal margin left, |
| 160 | // that is bad, something is wrong with the PID loops, |
| 161 | // they are not cooling the system, enable failsafe mode also. |
| 162 | if (_marginHot) |
| 163 | { |
| 164 | return true; |
| 165 | } |
| 166 | |
Alex.Song | 8f73ad7 | 2021-10-07 00:18:27 +0800 | [diff] [blame] | 167 | return _failed || !_available || !_functional; |
James Feist | 36b7d8e | 2018-10-05 15:39:01 -0700 | [diff] [blame] | 168 | } |
| 169 | |
| 170 | void DbusPassive::setFailed(bool value) |
| 171 | { |
| 172 | _failed = value; |
| 173 | } |
| 174 | |
James Feist | 4b36f26 | 2020-07-07 16:56:41 -0700 | [diff] [blame] | 175 | void DbusPassive::setFunctional(bool value) |
| 176 | { |
| 177 | _functional = value; |
| 178 | } |
| 179 | |
Alex.Song | 8f73ad7 | 2021-10-07 00:18:27 +0800 | [diff] [blame] | 180 | void DbusPassive::setAvailable(bool value) |
| 181 | { |
| 182 | _available = value; |
| 183 | } |
| 184 | |
Patrick Venture | 863b924 | 2018-03-08 08:29:23 -0800 | [diff] [blame] | 185 | int64_t DbusPassive::getScale(void) |
| 186 | { |
| 187 | return _scale; |
| 188 | } |
| 189 | |
Patrick Venture | 563a356 | 2018-10-30 09:31:26 -0700 | [diff] [blame] | 190 | std::string DbusPassive::getID(void) |
Patrick Venture | 863b924 | 2018-03-08 08:29:23 -0800 | [diff] [blame] | 191 | { |
| 192 | return _id; |
| 193 | } |
| 194 | |
James Feist | 75eb769 | 2019-02-25 12:50:02 -0800 | [diff] [blame] | 195 | double DbusPassive::getMax(void) |
| 196 | { |
| 197 | return _max; |
| 198 | } |
| 199 | |
| 200 | double DbusPassive::getMin(void) |
| 201 | { |
| 202 | return _min; |
| 203 | } |
| 204 | |
Josh Lehan | 3e2f758 | 2020-09-20 22:06:03 -0700 | [diff] [blame] | 205 | void DbusPassive::updateValue(double value, bool force) |
| 206 | { |
| 207 | _badReading = false; |
| 208 | |
| 209 | // Do not let a NAN, or other floating-point oddity, be used to update |
| 210 | // the value, as that indicates the sensor has no valid reading. |
| 211 | if (!(std::isfinite(value))) |
| 212 | { |
| 213 | _badReading = true; |
| 214 | |
| 215 | // Do not continue with a bad reading, unless caller forcing |
| 216 | if (!force) |
| 217 | { |
| 218 | return; |
| 219 | } |
| 220 | } |
| 221 | |
| 222 | value *= std::pow(10.0, _scale); |
| 223 | |
| 224 | auto unscaled = value; |
| 225 | scaleSensorReading(_min, _max, value); |
| 226 | |
| 227 | if (_typeMargin) |
| 228 | { |
| 229 | _marginHot = false; |
| 230 | |
| 231 | // Unlike an absolute temperature sensor, |
| 232 | // where 0 degrees C is a good reading, |
| 233 | // a value received of 0 (or negative) margin is worrisome, |
| 234 | // and should be flagged. |
| 235 | // Either it indicates margin not calculated properly, |
| 236 | // or somebody forgot to set the margin-zero setpoint, |
| 237 | // or the system is really overheating that much. |
| 238 | // This is a different condition from _failed |
| 239 | // and _badReading, so it merits its own flag. |
| 240 | // The sensor has not failed, the reading is good, but the zone |
| 241 | // still needs to know that it should go to failsafe mode. |
| 242 | if (unscaled <= 0.0) |
| 243 | { |
| 244 | _marginHot = true; |
| 245 | } |
| 246 | } |
| 247 | |
| 248 | setValue(value); |
| 249 | } |
| 250 | |
Patrick Venture | 7af157b | 2018-10-30 11:24:40 -0700 | [diff] [blame] | 251 | int handleSensorValue(sdbusplus::message::message& msg, DbusPassive* owner) |
Patrick Venture | 863b924 | 2018-03-08 08:29:23 -0800 | [diff] [blame] | 252 | { |
Patrick Venture | 863b924 | 2018-03-08 08:29:23 -0800 | [diff] [blame] | 253 | std::string msgSensor; |
James Feist | 1f802f5 | 2019-02-08 13:51:43 -0800 | [diff] [blame] | 254 | std::map<std::string, std::variant<int64_t, double, bool>> msgData; |
Patrick Venture | d0c7566 | 2018-06-12 19:03:21 -0700 | [diff] [blame] | 255 | |
| 256 | msg.read(msgSensor, msgData); |
Patrick Venture | 863b924 | 2018-03-08 08:29:23 -0800 | [diff] [blame] | 257 | |
| 258 | if (msgSensor == "xyz.openbmc_project.Sensor.Value") |
| 259 | { |
| 260 | auto valPropMap = msgData.find("Value"); |
| 261 | if (valPropMap != msgData.end()) |
| 262 | { |
James Feist | 1f802f5 | 2019-02-08 13:51:43 -0800 | [diff] [blame] | 263 | double value = |
| 264 | std::visit(VariantToDoubleVisitor(), valPropMap->second); |
Patrick Venture | 863b924 | 2018-03-08 08:29:23 -0800 | [diff] [blame] | 265 | |
Josh Lehan | 3e2f758 | 2020-09-20 22:06:03 -0700 | [diff] [blame] | 266 | owner->updateValue(value, false); |
Patrick Venture | 863b924 | 2018-03-08 08:29:23 -0800 | [diff] [blame] | 267 | } |
| 268 | } |
James Feist | 36b7d8e | 2018-10-05 15:39:01 -0700 | [diff] [blame] | 269 | else if (msgSensor == "xyz.openbmc_project.Sensor.Threshold.Critical") |
| 270 | { |
| 271 | auto criticalAlarmLow = msgData.find("CriticalAlarmLow"); |
| 272 | auto criticalAlarmHigh = msgData.find("CriticalAlarmHigh"); |
| 273 | if (criticalAlarmHigh == msgData.end() && |
| 274 | criticalAlarmLow == msgData.end()) |
| 275 | { |
| 276 | return 0; |
| 277 | } |
| 278 | |
| 279 | bool asserted = false; |
| 280 | if (criticalAlarmLow != msgData.end()) |
| 281 | { |
James Feist | 1f802f5 | 2019-02-08 13:51:43 -0800 | [diff] [blame] | 282 | asserted = std::get<bool>(criticalAlarmLow->second); |
James Feist | 36b7d8e | 2018-10-05 15:39:01 -0700 | [diff] [blame] | 283 | } |
| 284 | |
| 285 | // checking both as in theory you could de-assert one threshold and |
| 286 | // assert the other at the same moment |
| 287 | if (!asserted && criticalAlarmHigh != msgData.end()) |
| 288 | { |
James Feist | 1f802f5 | 2019-02-08 13:51:43 -0800 | [diff] [blame] | 289 | asserted = std::get<bool>(criticalAlarmHigh->second); |
James Feist | 36b7d8e | 2018-10-05 15:39:01 -0700 | [diff] [blame] | 290 | } |
| 291 | owner->setFailed(asserted); |
| 292 | } |
Alex.Song | 8f73ad7 | 2021-10-07 00:18:27 +0800 | [diff] [blame] | 293 | else if (msgSensor == "xyz.openbmc_project.State.Decorator.Availability") |
| 294 | { |
| 295 | auto available = msgData.find("Available"); |
| 296 | if (available == msgData.end()) |
| 297 | { |
| 298 | return 0; |
| 299 | } |
| 300 | bool asserted = std::get<bool>(available->second); |
| 301 | owner->setAvailable(asserted); |
| 302 | if (!asserted) |
| 303 | { |
| 304 | // A thermal controller will continue its PID calculation and not |
| 305 | // trigger a 'failsafe' when some inputs are unavailable. |
| 306 | // So, forced to clear the value here to prevent a historical |
| 307 | // value to participate in a latter PID calculation. |
| 308 | owner->updateValue(std::numeric_limits<double>::quiet_NaN(), true); |
| 309 | } |
| 310 | } |
James Feist | 4b36f26 | 2020-07-07 16:56:41 -0700 | [diff] [blame] | 311 | else if (msgSensor == |
| 312 | "xyz.openbmc_project.State.Decorator.OperationalStatus") |
| 313 | { |
| 314 | auto functional = msgData.find("Functional"); |
| 315 | if (functional == msgData.end()) |
| 316 | { |
| 317 | return 0; |
| 318 | } |
| 319 | bool asserted = std::get<bool>(functional->second); |
| 320 | owner->setFunctional(asserted); |
| 321 | } |
Patrick Venture | 863b924 | 2018-03-08 08:29:23 -0800 | [diff] [blame] | 322 | |
| 323 | return 0; |
| 324 | } |
Patrick Venture | d0c7566 | 2018-06-12 19:03:21 -0700 | [diff] [blame] | 325 | |
Patrick Venture | 7af157b | 2018-10-30 11:24:40 -0700 | [diff] [blame] | 326 | int dbusHandleSignal(sd_bus_message* msg, void* usrData, sd_bus_error* err) |
Patrick Venture | d0c7566 | 2018-06-12 19:03:21 -0700 | [diff] [blame] | 327 | { |
| 328 | auto sdbpMsg = sdbusplus::message::message(msg); |
| 329 | DbusPassive* obj = static_cast<DbusPassive*>(usrData); |
| 330 | |
Patrick Venture | 7af157b | 2018-10-30 11:24:40 -0700 | [diff] [blame] | 331 | return handleSensorValue(sdbpMsg, obj); |
Patrick Venture | d0c7566 | 2018-06-12 19:03:21 -0700 | [diff] [blame] | 332 | } |
Patrick Venture | a076487 | 2020-08-08 07:48:43 -0700 | [diff] [blame] | 333 | |
| 334 | } // namespace pid_control |