Josh Lehan | 2a40e93 | 2020-09-02 11:48:14 -0700 | [diff] [blame] | 1 | #include "ExternalSensor.hpp" |
Ed Tanous | eacbfdd | 2024-04-04 12:00:24 -0700 | [diff] [blame] | 2 | #include "Thresholds.hpp" |
Josh Lehan | 2a40e93 | 2020-09-02 11:48:14 -0700 | [diff] [blame] | 3 | #include "Utils.hpp" |
| 4 | #include "VariantVisitors.hpp" |
| 5 | |
Ed Tanous | eacbfdd | 2024-04-04 12:00:24 -0700 | [diff] [blame] | 6 | #include <boost/asio/error.hpp> |
| 7 | #include <boost/asio/io_context.hpp> |
| 8 | #include <boost/asio/post.hpp> |
| 9 | #include <boost/asio/steady_timer.hpp> |
Josh Lehan | 2a40e93 | 2020-09-02 11:48:14 -0700 | [diff] [blame] | 10 | #include <boost/container/flat_map.hpp> |
| 11 | #include <boost/container/flat_set.hpp> |
| 12 | #include <sdbusplus/asio/connection.hpp> |
| 13 | #include <sdbusplus/asio/object_server.hpp> |
| 14 | #include <sdbusplus/bus/match.hpp> |
Ed Tanous | eacbfdd | 2024-04-04 12:00:24 -0700 | [diff] [blame] | 15 | #include <sdbusplus/message.hpp> |
| 16 | #include <sdbusplus/message/native_types.hpp> |
Josh Lehan | 2a40e93 | 2020-09-02 11:48:14 -0700 | [diff] [blame] | 17 | |
Ed Tanous | eacbfdd | 2024-04-04 12:00:24 -0700 | [diff] [blame] | 18 | #include <algorithm> |
Josh Lehan | 2a40e93 | 2020-09-02 11:48:14 -0700 | [diff] [blame] | 19 | #include <array> |
Ed Tanous | eacbfdd | 2024-04-04 12:00:24 -0700 | [diff] [blame] | 20 | #include <chrono> |
| 21 | #include <cmath> |
Josh Lehan | 2a40e93 | 2020-09-02 11:48:14 -0700 | [diff] [blame] | 22 | #include <functional> |
Ed Tanous | eacbfdd | 2024-04-04 12:00:24 -0700 | [diff] [blame] | 23 | #include <iostream> |
Josh Lehan | 2a40e93 | 2020-09-02 11:48:14 -0700 | [diff] [blame] | 24 | #include <memory> |
Josh Lehan | 2a40e93 | 2020-09-02 11:48:14 -0700 | [diff] [blame] | 25 | #include <string> |
| 26 | #include <utility> |
| 27 | #include <variant> |
| 28 | #include <vector> |
| 29 | |
| 30 | // Copied from HwmonTempSensor and inspired by |
| 31 | // https://gerrit.openbmc-project.xyz/c/openbmc/dbus-sensors/+/35476 |
| 32 | |
| 33 | // The ExternalSensor is a sensor whose value is intended to be writable |
| 34 | // by something external to the BMC, so that the host (or something else) |
Josh Lehan | 7243217 | 2021-03-17 13:35:43 -0700 | [diff] [blame] | 35 | // can write to it, perhaps by using an IPMI or Redfish connection. |
Josh Lehan | 2a40e93 | 2020-09-02 11:48:14 -0700 | [diff] [blame] | 36 | |
| 37 | // Unlike most other sensors, an external sensor does not correspond |
Josh Lehan | 7243217 | 2021-03-17 13:35:43 -0700 | [diff] [blame] | 38 | // to a hwmon file or any other kernel/hardware interface, |
Josh Lehan | 2a40e93 | 2020-09-02 11:48:14 -0700 | [diff] [blame] | 39 | // so, after initialization, this module does not have much to do, |
| 40 | // but it handles reinitialization and thresholds, similar to the others. |
Josh Lehan | 7243217 | 2021-03-17 13:35:43 -0700 | [diff] [blame] | 41 | // The main work of this module is to provide backing storage for a |
| 42 | // sensor that exists only virtually, and to provide an optional |
| 43 | // timeout service for detecting loss of timely updates. |
Josh Lehan | 2a40e93 | 2020-09-02 11:48:14 -0700 | [diff] [blame] | 44 | |
| 45 | // As there is no corresponding driver or hardware to support, |
| 46 | // all configuration of this sensor comes from the JSON parameters: |
Josh Lehan | 7243217 | 2021-03-17 13:35:43 -0700 | [diff] [blame] | 47 | // MinValue, MaxValue, Timeout, PowerState, Units, Name |
Josh Lehan | 2a40e93 | 2020-09-02 11:48:14 -0700 | [diff] [blame] | 48 | |
Josh Lehan | 7243217 | 2021-03-17 13:35:43 -0700 | [diff] [blame] | 49 | // The purpose of "Units" is to specify the physical characteristic |
Josh Lehan | 2a40e93 | 2020-09-02 11:48:14 -0700 | [diff] [blame] | 50 | // the external sensor is measuring, because with an external sensor |
| 51 | // there is no other way to tell, and it will be used for the object path |
Josh Lehan | 7243217 | 2021-03-17 13:35:43 -0700 | [diff] [blame] | 52 | // here: /xyz/openbmc_project/sensors/<Units>/<Name> |
| 53 | |
| 54 | // For more information, see external-sensor.md design document: |
| 55 | // https://gerrit.openbmc-project.xyz/c/openbmc/docs/+/41452 |
| 56 | // https://github.com/openbmc/docs/tree/master/designs/ |
Josh Lehan | 2a40e93 | 2020-09-02 11:48:14 -0700 | [diff] [blame] | 57 | |
Ed Tanous | 8a57ec0 | 2020-10-09 12:46:52 -0700 | [diff] [blame] | 58 | static constexpr bool debug = false; |
Josh Lehan | 2a40e93 | 2020-09-02 11:48:14 -0700 | [diff] [blame] | 59 | |
Zev Weiss | 054aad8 | 2022-08-18 01:37:34 -0700 | [diff] [blame] | 60 | static const char* sensorType = "ExternalSensor"; |
Josh Lehan | 2a40e93 | 2020-09-02 11:48:14 -0700 | [diff] [blame] | 61 | |
Patrick Williams | 2aaf717 | 2024-08-16 15:20:40 -0400 | [diff] [blame] | 62 | void updateReaper( |
| 63 | boost::container::flat_map<std::string, std::shared_ptr<ExternalSensor>>& |
| 64 | sensors, |
| 65 | boost::asio::steady_timer& timer, |
| 66 | const std::chrono::steady_clock::time_point& now) |
Josh Lehan | 7243217 | 2021-03-17 13:35:43 -0700 | [diff] [blame] | 67 | { |
| 68 | // First pass, reap all stale sensors |
Zev Weiss | 08cb50c | 2022-08-12 18:21:01 -0700 | [diff] [blame] | 69 | for (const auto& [name, sensor] : sensors) |
Josh Lehan | 7243217 | 2021-03-17 13:35:43 -0700 | [diff] [blame] | 70 | { |
Zev Weiss | 08cb50c | 2022-08-12 18:21:01 -0700 | [diff] [blame] | 71 | if (!sensor) |
Josh Lehan | 7243217 | 2021-03-17 13:35:43 -0700 | [diff] [blame] | 72 | { |
| 73 | continue; |
| 74 | } |
| 75 | |
Zev Weiss | 08cb50c | 2022-08-12 18:21:01 -0700 | [diff] [blame] | 76 | if (!sensor->isAliveAndPerishable()) |
Josh Lehan | 7243217 | 2021-03-17 13:35:43 -0700 | [diff] [blame] | 77 | { |
| 78 | continue; |
| 79 | } |
| 80 | |
Zev Weiss | 08cb50c | 2022-08-12 18:21:01 -0700 | [diff] [blame] | 81 | if (!sensor->isAliveAndFresh(now)) |
Josh Lehan | 7243217 | 2021-03-17 13:35:43 -0700 | [diff] [blame] | 82 | { |
| 83 | // Mark sensor as dead, no longer alive |
Zev Weiss | 08cb50c | 2022-08-12 18:21:01 -0700 | [diff] [blame] | 84 | sensor->writeInvalidate(); |
Josh Lehan | 7243217 | 2021-03-17 13:35:43 -0700 | [diff] [blame] | 85 | } |
| 86 | } |
| 87 | |
| 88 | std::chrono::steady_clock::duration nextCheck; |
| 89 | bool needCheck = false; |
| 90 | |
| 91 | // Second pass, determine timer interval to next check |
Zev Weiss | 08cb50c | 2022-08-12 18:21:01 -0700 | [diff] [blame] | 92 | for (const auto& [name, sensor] : sensors) |
Josh Lehan | 7243217 | 2021-03-17 13:35:43 -0700 | [diff] [blame] | 93 | { |
Zev Weiss | 08cb50c | 2022-08-12 18:21:01 -0700 | [diff] [blame] | 94 | if (!sensor) |
Josh Lehan | 7243217 | 2021-03-17 13:35:43 -0700 | [diff] [blame] | 95 | { |
| 96 | continue; |
| 97 | } |
| 98 | |
Zev Weiss | 08cb50c | 2022-08-12 18:21:01 -0700 | [diff] [blame] | 99 | if (!sensor->isAliveAndPerishable()) |
Josh Lehan | 7243217 | 2021-03-17 13:35:43 -0700 | [diff] [blame] | 100 | { |
| 101 | continue; |
| 102 | } |
| 103 | |
Zev Weiss | 08cb50c | 2022-08-12 18:21:01 -0700 | [diff] [blame] | 104 | auto expiration = sensor->ageRemaining(now); |
Josh Lehan | 7243217 | 2021-03-17 13:35:43 -0700 | [diff] [blame] | 105 | |
| 106 | if (needCheck) |
| 107 | { |
| 108 | nextCheck = std::min(nextCheck, expiration); |
| 109 | } |
| 110 | else |
| 111 | { |
| 112 | // Initialization |
| 113 | nextCheck = expiration; |
| 114 | needCheck = true; |
| 115 | } |
| 116 | } |
| 117 | |
| 118 | if (!needCheck) |
| 119 | { |
| 120 | if constexpr (debug) |
| 121 | { |
| 122 | std::cerr << "Next ExternalSensor timer idle\n"; |
| 123 | } |
| 124 | |
| 125 | return; |
| 126 | } |
| 127 | |
| 128 | timer.expires_at(now + nextCheck); |
| 129 | |
| 130 | timer.async_wait([&sensors, &timer](const boost::system::error_code& err) { |
| 131 | if (err != boost::system::errc::success) |
| 132 | { |
| 133 | // Cancellation is normal, as timer is dynamically rescheduled |
Josh Lehan | 0362738 | 2021-03-17 13:35:43 -0700 | [diff] [blame] | 134 | if (err != boost::asio::error::operation_aborted) |
Josh Lehan | 7243217 | 2021-03-17 13:35:43 -0700 | [diff] [blame] | 135 | { |
| 136 | std::cerr << "ExternalSensor timer scheduling problem: " |
| 137 | << err.message() << "\n"; |
| 138 | } |
| 139 | return; |
| 140 | } |
Josh Lehan | 0362738 | 2021-03-17 13:35:43 -0700 | [diff] [blame] | 141 | |
Josh Lehan | 7243217 | 2021-03-17 13:35:43 -0700 | [diff] [blame] | 142 | updateReaper(sensors, timer, std::chrono::steady_clock::now()); |
| 143 | }); |
| 144 | |
| 145 | if constexpr (debug) |
| 146 | { |
| 147 | std::cerr << "Next ExternalSensor timer " |
| 148 | << std::chrono::duration_cast<std::chrono::microseconds>( |
| 149 | nextCheck) |
| 150 | .count() |
| 151 | << " us\n"; |
| 152 | } |
| 153 | } |
| 154 | |
Josh Lehan | 2a40e93 | 2020-09-02 11:48:14 -0700 | [diff] [blame] | 155 | void createSensors( |
Ed Tanous | 8a17c30 | 2021-09-02 15:07:11 -0700 | [diff] [blame] | 156 | sdbusplus::asio::object_server& objectServer, |
Josh Lehan | 2a40e93 | 2020-09-02 11:48:14 -0700 | [diff] [blame] | 157 | boost::container::flat_map<std::string, std::shared_ptr<ExternalSensor>>& |
| 158 | sensors, |
| 159 | std::shared_ptr<sdbusplus::asio::connection>& dbusConnection, |
| 160 | const std::shared_ptr<boost::container::flat_set<std::string>>& |
Josh Lehan | 7243217 | 2021-03-17 13:35:43 -0700 | [diff] [blame] | 161 | sensorsChanged, |
| 162 | boost::asio::steady_timer& reaperTimer) |
Josh Lehan | 2a40e93 | 2020-09-02 11:48:14 -0700 | [diff] [blame] | 163 | { |
Josh Lehan | 0362738 | 2021-03-17 13:35:43 -0700 | [diff] [blame] | 164 | if constexpr (debug) |
| 165 | { |
| 166 | std::cerr << "ExternalSensor considering creating sensors\n"; |
| 167 | } |
| 168 | |
Josh Lehan | 2a40e93 | 2020-09-02 11:48:14 -0700 | [diff] [blame] | 169 | auto getter = std::make_shared<GetSensorConfiguration>( |
| 170 | dbusConnection, |
Ed Tanous | 8a17c30 | 2021-09-02 15:07:11 -0700 | [diff] [blame] | 171 | [&objectServer, &sensors, &dbusConnection, sensorsChanged, |
Josh Lehan | 7243217 | 2021-03-17 13:35:43 -0700 | [diff] [blame] | 172 | &reaperTimer](const ManagedObjectType& sensorConfigurations) { |
Patrick Williams | 2aaf717 | 2024-08-16 15:20:40 -0400 | [diff] [blame] | 173 | bool firstScan = (sensorsChanged == nullptr); |
Josh Lehan | 2a40e93 | 2020-09-02 11:48:14 -0700 | [diff] [blame] | 174 | |
Patrick Williams | 2aaf717 | 2024-08-16 15:20:40 -0400 | [diff] [blame] | 175 | for (const std::pair<sdbusplus::message::object_path, SensorData>& |
| 176 | sensor : sensorConfigurations) |
| 177 | { |
| 178 | const std::string& interfacePath = sensor.first.str; |
| 179 | const SensorData& sensorData = sensor.second; |
Ed Tanous | bb67932 | 2022-05-16 16:10:00 -0700 | [diff] [blame] | 180 | |
Patrick Williams | 2aaf717 | 2024-08-16 15:20:40 -0400 | [diff] [blame] | 181 | auto sensorBase = |
| 182 | sensorData.find(configInterfaceName(sensorType)); |
| 183 | if (sensorBase == sensorData.end()) |
Josh Lehan | 2a40e93 | 2020-09-02 11:48:14 -0700 | [diff] [blame] | 184 | { |
Patrick Williams | 2aaf717 | 2024-08-16 15:20:40 -0400 | [diff] [blame] | 185 | std::cerr << "Base configuration not found for " |
| 186 | << interfacePath << "\n"; |
Ed Tanous | bb67932 | 2022-05-16 16:10:00 -0700 | [diff] [blame] | 187 | continue; |
Josh Lehan | 7243217 | 2021-03-17 13:35:43 -0700 | [diff] [blame] | 188 | } |
Patrick Williams | 2aaf717 | 2024-08-16 15:20:40 -0400 | [diff] [blame] | 189 | |
| 190 | const SensorBaseConfiguration& baseConfiguration = *sensorBase; |
| 191 | const SensorBaseConfigMap& baseConfigMap = |
| 192 | baseConfiguration.second; |
| 193 | |
| 194 | // MinValue and MinValue are mandatory numeric parameters |
| 195 | auto minFound = baseConfigMap.find("MinValue"); |
| 196 | if (minFound == baseConfigMap.end()) |
| 197 | { |
| 198 | std::cerr << "MinValue parameter not found for " |
| 199 | << interfacePath << "\n"; |
| 200 | continue; |
| 201 | } |
| 202 | double minValue = |
| 203 | std::visit(VariantToDoubleVisitor(), minFound->second); |
| 204 | if (!std::isfinite(minValue)) |
| 205 | { |
| 206 | std::cerr << "MinValue parameter not parsed for " |
| 207 | << interfacePath << "\n"; |
| 208 | continue; |
| 209 | } |
| 210 | |
| 211 | auto maxFound = baseConfigMap.find("MaxValue"); |
| 212 | if (maxFound == baseConfigMap.end()) |
| 213 | { |
| 214 | std::cerr << "MaxValue parameter not found for " |
| 215 | << interfacePath << "\n"; |
| 216 | continue; |
| 217 | } |
| 218 | double maxValue = |
| 219 | std::visit(VariantToDoubleVisitor(), maxFound->second); |
| 220 | if (!std::isfinite(maxValue)) |
| 221 | { |
| 222 | std::cerr << "MaxValue parameter not parsed for " |
| 223 | << interfacePath << "\n"; |
| 224 | continue; |
| 225 | } |
| 226 | |
| 227 | double timeoutSecs = 0.0; |
| 228 | |
| 229 | // Timeout is an optional numeric parameter |
| 230 | auto timeoutFound = baseConfigMap.find("Timeout"); |
| 231 | if (timeoutFound != baseConfigMap.end()) |
| 232 | { |
| 233 | timeoutSecs = std::visit(VariantToDoubleVisitor(), |
| 234 | timeoutFound->second); |
| 235 | } |
| 236 | if (!std::isfinite(timeoutSecs) || (timeoutSecs < 0.0)) |
| 237 | { |
| 238 | std::cerr << "Timeout parameter not parsed for " |
| 239 | << interfacePath << "\n"; |
| 240 | continue; |
| 241 | } |
| 242 | |
| 243 | std::string sensorName; |
| 244 | std::string sensorUnits; |
| 245 | |
| 246 | // Name and Units are mandatory string parameters |
| 247 | auto nameFound = baseConfigMap.find("Name"); |
| 248 | if (nameFound == baseConfigMap.end()) |
| 249 | { |
| 250 | std::cerr << "Name parameter not found for " |
| 251 | << interfacePath << "\n"; |
| 252 | continue; |
| 253 | } |
| 254 | sensorName = |
| 255 | std::visit(VariantToStringVisitor(), nameFound->second); |
| 256 | if (sensorName.empty()) |
| 257 | { |
| 258 | std::cerr << "Name parameter not parsed for " |
| 259 | << interfacePath << "\n"; |
| 260 | continue; |
| 261 | } |
| 262 | |
| 263 | auto unitsFound = baseConfigMap.find("Units"); |
| 264 | if (unitsFound == baseConfigMap.end()) |
| 265 | { |
| 266 | std::cerr << "Units parameter not found for " |
| 267 | << interfacePath << "\n"; |
| 268 | continue; |
| 269 | } |
| 270 | sensorUnits = |
| 271 | std::visit(VariantToStringVisitor(), unitsFound->second); |
| 272 | if (sensorUnits.empty()) |
| 273 | { |
| 274 | std::cerr << "Units parameter not parsed for " |
| 275 | << interfacePath << "\n"; |
| 276 | continue; |
| 277 | } |
| 278 | |
| 279 | // on rescans, only update sensors we were signaled by |
| 280 | auto findSensor = sensors.find(sensorName); |
| 281 | if (!firstScan && (findSensor != sensors.end())) |
| 282 | { |
| 283 | std::string suffixName = "/"; |
| 284 | suffixName += findSensor->second->name; |
| 285 | bool found = false; |
| 286 | for (auto it = sensorsChanged->begin(); |
| 287 | it != sensorsChanged->end(); it++) |
| 288 | { |
| 289 | std::string suffixIt = "/"; |
| 290 | suffixIt += *it; |
| 291 | if (suffixIt.ends_with(suffixName)) |
| 292 | { |
| 293 | sensorsChanged->erase(it); |
| 294 | findSensor->second = nullptr; |
| 295 | found = true; |
| 296 | if constexpr (debug) |
| 297 | { |
| 298 | std::cerr << "ExternalSensor " << sensorName |
| 299 | << " change found\n"; |
| 300 | } |
| 301 | break; |
| 302 | } |
| 303 | } |
| 304 | if (!found) |
| 305 | { |
| 306 | continue; |
| 307 | } |
| 308 | } |
| 309 | |
| 310 | std::vector<thresholds::Threshold> sensorThresholds; |
| 311 | if (!parseThresholdsFromConfig(sensorData, sensorThresholds)) |
| 312 | { |
| 313 | std::cerr << "error populating thresholds for " |
| 314 | << sensorName << "\n"; |
| 315 | } |
| 316 | |
| 317 | PowerState readState = getPowerState(baseConfigMap); |
| 318 | |
| 319 | auto& sensorEntry = sensors[sensorName]; |
| 320 | sensorEntry = nullptr; |
| 321 | |
| 322 | sensorEntry = std::make_shared<ExternalSensor>( |
| 323 | sensorType, objectServer, dbusConnection, sensorName, |
| 324 | sensorUnits, std::move(sensorThresholds), interfacePath, |
| 325 | maxValue, minValue, timeoutSecs, readState); |
| 326 | sensorEntry->initWriteHook( |
| 327 | [&sensors, &reaperTimer]( |
| 328 | const std::chrono::steady_clock::time_point& now) { |
| 329 | updateReaper(sensors, reaperTimer, now); |
| 330 | }); |
| 331 | |
| 332 | if constexpr (debug) |
| 333 | { |
| 334 | std::cerr |
| 335 | << "ExternalSensor " << sensorName << " created\n"; |
| 336 | } |
Josh Lehan | 2a40e93 | 2020-09-02 11:48:14 -0700 | [diff] [blame] | 337 | } |
Patrick Williams | 2aaf717 | 2024-08-16 15:20:40 -0400 | [diff] [blame] | 338 | }); |
Josh Lehan | 2a40e93 | 2020-09-02 11:48:14 -0700 | [diff] [blame] | 339 | |
| 340 | getter->getConfiguration(std::vector<std::string>{sensorType}); |
| 341 | } |
| 342 | |
| 343 | int main() |
| 344 | { |
Josh Lehan | 7243217 | 2021-03-17 13:35:43 -0700 | [diff] [blame] | 345 | if constexpr (debug) |
| 346 | { |
| 347 | std::cerr << "ExternalSensor service starting up\n"; |
| 348 | } |
| 349 | |
Ed Tanous | 1f97863 | 2023-02-28 18:16:39 -0800 | [diff] [blame] | 350 | boost::asio::io_context io; |
Josh Lehan | 2a40e93 | 2020-09-02 11:48:14 -0700 | [diff] [blame] | 351 | auto systemBus = std::make_shared<sdbusplus::asio::connection>(io); |
Ed Tanous | 14ed5e9 | 2022-07-12 15:50:23 -0700 | [diff] [blame] | 352 | sdbusplus::asio::object_server objectServer(systemBus, true); |
| 353 | |
| 354 | objectServer.add_manager("/xyz/openbmc_project/sensors"); |
Josh Lehan | 2a40e93 | 2020-09-02 11:48:14 -0700 | [diff] [blame] | 355 | systemBus->request_name("xyz.openbmc_project.ExternalSensor"); |
Ed Tanous | 14ed5e9 | 2022-07-12 15:50:23 -0700 | [diff] [blame] | 356 | |
Josh Lehan | 2a40e93 | 2020-09-02 11:48:14 -0700 | [diff] [blame] | 357 | boost::container::flat_map<std::string, std::shared_ptr<ExternalSensor>> |
| 358 | sensors; |
Josh Lehan | 2a40e93 | 2020-09-02 11:48:14 -0700 | [diff] [blame] | 359 | auto sensorsChanged = |
| 360 | std::make_shared<boost::container::flat_set<std::string>>(); |
Josh Lehan | 7243217 | 2021-03-17 13:35:43 -0700 | [diff] [blame] | 361 | boost::asio::steady_timer reaperTimer(io); |
Josh Lehan | 2a40e93 | 2020-09-02 11:48:14 -0700 | [diff] [blame] | 362 | |
Patrick Williams | 2aaf717 | 2024-08-16 15:20:40 -0400 | [diff] [blame] | 363 | boost::asio::post(io, [&objectServer, &sensors, &systemBus, |
| 364 | &reaperTimer]() { |
Ed Tanous | 8a17c30 | 2021-09-02 15:07:11 -0700 | [diff] [blame] | 365 | createSensors(objectServer, sensors, systemBus, nullptr, reaperTimer); |
Josh Lehan | 2a40e93 | 2020-09-02 11:48:14 -0700 | [diff] [blame] | 366 | }); |
| 367 | |
Ed Tanous | 9b4a20e | 2022-09-06 08:47:11 -0700 | [diff] [blame] | 368 | boost::asio::steady_timer filterTimer(io); |
Patrick Williams | 92f8f51 | 2022-07-22 19:26:55 -0500 | [diff] [blame] | 369 | std::function<void(sdbusplus::message_t&)> eventHandler = |
Ed Tanous | 8a17c30 | 2021-09-02 15:07:11 -0700 | [diff] [blame] | 370 | [&objectServer, &sensors, &systemBus, &sensorsChanged, &filterTimer, |
Patrick Williams | 92f8f51 | 2022-07-22 19:26:55 -0500 | [diff] [blame] | 371 | &reaperTimer](sdbusplus::message_t& message) mutable { |
Patrick Williams | 2aaf717 | 2024-08-16 15:20:40 -0400 | [diff] [blame] | 372 | if (message.is_method_error()) |
Josh Lehan | 2a40e93 | 2020-09-02 11:48:14 -0700 | [diff] [blame] | 373 | { |
Patrick Williams | 2aaf717 | 2024-08-16 15:20:40 -0400 | [diff] [blame] | 374 | std::cerr << "callback method error\n"; |
Josh Lehan | 2a40e93 | 2020-09-02 11:48:14 -0700 | [diff] [blame] | 375 | return; |
| 376 | } |
Josh Lehan | 0362738 | 2021-03-17 13:35:43 -0700 | [diff] [blame] | 377 | |
Patrick Williams | 2aaf717 | 2024-08-16 15:20:40 -0400 | [diff] [blame] | 378 | const auto* messagePath = message.get_path(); |
| 379 | sensorsChanged->insert(messagePath); |
| 380 | if constexpr (debug) |
| 381 | { |
| 382 | std::cerr << "ExternalSensor change event received: " |
| 383 | << messagePath << "\n"; |
| 384 | } |
| 385 | |
| 386 | // this implicitly cancels the timer |
| 387 | filterTimer.expires_after(std::chrono::seconds(1)); |
| 388 | |
| 389 | filterTimer.async_wait( |
| 390 | [&objectServer, &sensors, &systemBus, &sensorsChanged, |
| 391 | &reaperTimer](const boost::system::error_code& ec) mutable { |
| 392 | if (ec != boost::system::errc::success) |
| 393 | { |
| 394 | if (ec != boost::asio::error::operation_aborted) |
| 395 | { |
| 396 | std::cerr |
| 397 | << "callback error: " << ec.message() << "\n"; |
| 398 | } |
| 399 | return; |
| 400 | } |
| 401 | |
| 402 | createSensors(objectServer, sensors, systemBus, |
| 403 | sensorsChanged, reaperTimer); |
| 404 | }); |
| 405 | }; |
Josh Lehan | 2a40e93 | 2020-09-02 11:48:14 -0700 | [diff] [blame] | 406 | |
Zev Weiss | 214d971 | 2022-08-12 12:54:31 -0700 | [diff] [blame] | 407 | std::vector<std::unique_ptr<sdbusplus::bus::match_t>> matches = |
| 408 | setupPropertiesChangedMatches( |
| 409 | *systemBus, std::to_array<const char*>({sensorType}), eventHandler); |
Josh Lehan | 2a40e93 | 2020-09-02 11:48:14 -0700 | [diff] [blame] | 410 | |
Josh Lehan | 7243217 | 2021-03-17 13:35:43 -0700 | [diff] [blame] | 411 | if constexpr (debug) |
| 412 | { |
| 413 | std::cerr << "ExternalSensor service entering main loop\n"; |
| 414 | } |
| 415 | |
Josh Lehan | 2a40e93 | 2020-09-02 11:48:14 -0700 | [diff] [blame] | 416 | io.run(); |
| 417 | } |