James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 1 | /* |
| 2 | // Copyright (c) 2017 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 | |
| 17 | #include <unistd.h> |
| 18 | |
| 19 | #include <HwmonTempSensor.hpp> |
| 20 | #include <boost/algorithm/string/predicate.hpp> |
| 21 | #include <boost/algorithm/string/replace.hpp> |
| 22 | #include <boost/date_time/posix_time/posix_time.hpp> |
| 23 | #include <iostream> |
| 24 | #include <limits> |
| 25 | #include <sdbusplus/asio/connection.hpp> |
| 26 | #include <sdbusplus/asio/object_server.hpp> |
| 27 | #include <string> |
| 28 | |
Jae Hyun Yoo | 9ced0a3 | 2018-10-25 10:42:39 -0700 | [diff] [blame] | 29 | static constexpr unsigned int sensorPollMs = 500; |
| 30 | static constexpr unsigned int sensorScaleFactor = 1000; |
| 31 | static constexpr size_t warnAfterErrorCount = 10; |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 32 | |
| 33 | HwmonTempSensor::HwmonTempSensor( |
| 34 | const std::string &path, const std::string &objectType, |
| 35 | sdbusplus::asio::object_server &objectServer, |
| 36 | std::shared_ptr<sdbusplus::asio::connection> &conn, |
Jae Hyun Yoo | 9ced0a3 | 2018-10-25 10:42:39 -0700 | [diff] [blame] | 37 | boost::asio::io_service &io, const std::string &sensorName, |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 38 | std::vector<thresholds::Threshold> &&_thresholds, |
| 39 | const std::string &sensorConfiguration) : |
James Feist | dc6c55f | 2018-10-31 12:53:20 -0700 | [diff] [blame^] | 40 | Sensor(boost::replace_all_copy(sensorName, " ", "_"), path, |
| 41 | std::move(_thresholds)), |
| 42 | objectType(objectType), configuration(sensorConfiguration), |
| 43 | objServer(objectServer), inputDev(io, open(path.c_str(), O_RDONLY)), |
| 44 | waitTimer(io), errCount(0), |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 45 | // todo, get these from config |
Jae Hyun Yoo | 9ced0a3 | 2018-10-25 10:42:39 -0700 | [diff] [blame] | 46 | maxValue(127), minValue(-128) |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 47 | { |
James Feist | 251c782 | 2018-09-12 12:54:15 -0700 | [diff] [blame] | 48 | sensorInterface = objectServer.add_interface( |
| 49 | "/xyz/openbmc_project/sensors/temperature/" + name, |
| 50 | "xyz.openbmc_project.Sensor.Value"); |
| 51 | |
Jae Hyun Yoo | 9ced0a3 | 2018-10-25 10:42:39 -0700 | [diff] [blame] | 52 | if (thresholds::hasWarningInterface(thresholds)) |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 53 | { |
James Feist | 251c782 | 2018-09-12 12:54:15 -0700 | [diff] [blame] | 54 | thresholdInterfaceWarning = objectServer.add_interface( |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 55 | "/xyz/openbmc_project/sensors/temperature/" + name, |
| 56 | "xyz.openbmc_project.Sensor.Threshold.Warning"); |
| 57 | } |
Jae Hyun Yoo | 9ced0a3 | 2018-10-25 10:42:39 -0700 | [diff] [blame] | 58 | if (thresholds::hasCriticalInterface(thresholds)) |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 59 | { |
James Feist | 251c782 | 2018-09-12 12:54:15 -0700 | [diff] [blame] | 60 | thresholdInterfaceCritical = objectServer.add_interface( |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 61 | "/xyz/openbmc_project/sensors/temperature/" + name, |
| 62 | "xyz.openbmc_project.Sensor.Threshold.Critical"); |
| 63 | } |
Jae Hyun Yoo | 9ced0a3 | 2018-10-25 10:42:39 -0700 | [diff] [blame] | 64 | setInitialProperties(conn); |
| 65 | setupRead(); |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 66 | } |
| 67 | |
| 68 | HwmonTempSensor::~HwmonTempSensor() |
| 69 | { |
| 70 | // close the input dev to cancel async operations |
Jae Hyun Yoo | 9ced0a3 | 2018-10-25 10:42:39 -0700 | [diff] [blame] | 71 | inputDev.close(); |
| 72 | waitTimer.cancel(); |
James Feist | 251c782 | 2018-09-12 12:54:15 -0700 | [diff] [blame] | 73 | objServer.remove_interface(thresholdInterfaceWarning); |
| 74 | objServer.remove_interface(thresholdInterfaceCritical); |
| 75 | objServer.remove_interface(sensorInterface); |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 76 | } |
| 77 | |
Jae Hyun Yoo | 9ced0a3 | 2018-10-25 10:42:39 -0700 | [diff] [blame] | 78 | void HwmonTempSensor::setupRead(void) |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 79 | { |
| 80 | boost::asio::async_read_until( |
Jae Hyun Yoo | 9ced0a3 | 2018-10-25 10:42:39 -0700 | [diff] [blame] | 81 | inputDev, readBuf, '\n', |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 82 | [&](const boost::system::error_code &ec, |
Jae Hyun Yoo | 9ced0a3 | 2018-10-25 10:42:39 -0700 | [diff] [blame] | 83 | std::size_t /*bytes_transfered*/) { handleResponse(ec); }); |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 84 | } |
| 85 | |
Jae Hyun Yoo | 9ced0a3 | 2018-10-25 10:42:39 -0700 | [diff] [blame] | 86 | void HwmonTempSensor::handleResponse(const boost::system::error_code &err) |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 87 | { |
| 88 | if (err == boost::system::errc::bad_file_descriptor) |
| 89 | { |
| 90 | return; // we're being destroyed |
| 91 | } |
Jae Hyun Yoo | 9ced0a3 | 2018-10-25 10:42:39 -0700 | [diff] [blame] | 92 | std::istream responseStream(&readBuf); |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 93 | if (!err) |
| 94 | { |
| 95 | std::string response; |
Jae Hyun Yoo | 9ced0a3 | 2018-10-25 10:42:39 -0700 | [diff] [blame] | 96 | std::getline(responseStream, response); |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 97 | try |
| 98 | { |
| 99 | float nvalue = std::stof(response); |
| 100 | |
Jae Hyun Yoo | 9ced0a3 | 2018-10-25 10:42:39 -0700 | [diff] [blame] | 101 | nvalue /= sensorScaleFactor; |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 102 | if (nvalue != value) |
| 103 | { |
Jae Hyun Yoo | 9ced0a3 | 2018-10-25 10:42:39 -0700 | [diff] [blame] | 104 | updateValue(nvalue); |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 105 | } |
Jae Hyun Yoo | 9ced0a3 | 2018-10-25 10:42:39 -0700 | [diff] [blame] | 106 | errCount = 0; |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 107 | } |
| 108 | catch (const std::invalid_argument &) |
| 109 | { |
Jae Hyun Yoo | 9ced0a3 | 2018-10-25 10:42:39 -0700 | [diff] [blame] | 110 | errCount++; |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 111 | } |
| 112 | } |
| 113 | else |
| 114 | { |
Jae Hyun Yoo | 9ced0a3 | 2018-10-25 10:42:39 -0700 | [diff] [blame] | 115 | errCount++; |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 116 | } |
James Feist | dc6c55f | 2018-10-31 12:53:20 -0700 | [diff] [blame^] | 117 | |
| 118 | // only print once |
Jae Hyun Yoo | 9ced0a3 | 2018-10-25 10:42:39 -0700 | [diff] [blame] | 119 | if (errCount == warnAfterErrorCount) |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 120 | { |
James Feist | dc6c55f | 2018-10-31 12:53:20 -0700 | [diff] [blame^] | 121 | std::cerr << "Failure to read sensor " << name << " at " << path |
| 122 | << " ec:" << err << "\n"; |
| 123 | } |
| 124 | |
| 125 | if (errCount >= warnAfterErrorCount) |
| 126 | { |
Jae Hyun Yoo | 9ced0a3 | 2018-10-25 10:42:39 -0700 | [diff] [blame] | 127 | updateValue(0); |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 128 | } |
Jae Hyun Yoo | 9ced0a3 | 2018-10-25 10:42:39 -0700 | [diff] [blame] | 129 | responseStream.clear(); |
| 130 | inputDev.close(); |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 131 | int fd = open(path.c_str(), O_RDONLY); |
| 132 | if (fd <= 0) |
| 133 | { |
| 134 | return; // we're no longer valid |
| 135 | } |
Jae Hyun Yoo | 9ced0a3 | 2018-10-25 10:42:39 -0700 | [diff] [blame] | 136 | inputDev.assign(fd); |
| 137 | waitTimer.expires_from_now(boost::posix_time::milliseconds(sensorPollMs)); |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 138 | ; |
Jae Hyun Yoo | 9ced0a3 | 2018-10-25 10:42:39 -0700 | [diff] [blame] | 139 | waitTimer.async_wait([&](const boost::system::error_code &ec) { |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 140 | if (ec == boost::asio::error::operation_aborted) |
| 141 | { |
| 142 | return; // we're being canceled |
| 143 | } |
Jae Hyun Yoo | 9ced0a3 | 2018-10-25 10:42:39 -0700 | [diff] [blame] | 144 | setupRead(); |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 145 | }); |
| 146 | } |
| 147 | |
Jae Hyun Yoo | 9ced0a3 | 2018-10-25 10:42:39 -0700 | [diff] [blame] | 148 | void HwmonTempSensor::checkThresholds(void) |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 149 | { |
James Feist | 251c782 | 2018-09-12 12:54:15 -0700 | [diff] [blame] | 150 | thresholds::checkThresholds(this); |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 151 | } |
| 152 | |
Jae Hyun Yoo | 9ced0a3 | 2018-10-25 10:42:39 -0700 | [diff] [blame] | 153 | void HwmonTempSensor::updateValue(const double &newValue) |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 154 | { |
Jae Hyun Yoo | 9ced0a3 | 2018-10-25 10:42:39 -0700 | [diff] [blame] | 155 | sensorInterface->set_property("Value", newValue); |
| 156 | value = newValue; |
| 157 | checkThresholds(); |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 158 | } |
| 159 | |
Jae Hyun Yoo | 9ced0a3 | 2018-10-25 10:42:39 -0700 | [diff] [blame] | 160 | void HwmonTempSensor::setInitialProperties( |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 161 | std::shared_ptr<sdbusplus::asio::connection> &conn) |
| 162 | { |
| 163 | // todo, get max and min from configuration |
Jae Hyun Yoo | 9ced0a3 | 2018-10-25 10:42:39 -0700 | [diff] [blame] | 164 | sensorInterface->register_property("MaxValue", maxValue); |
| 165 | sensorInterface->register_property("MinValue", minValue); |
James Feist | 251c782 | 2018-09-12 12:54:15 -0700 | [diff] [blame] | 166 | sensorInterface->register_property("Value", value); |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 167 | |
| 168 | for (auto &threshold : thresholds) |
| 169 | { |
| 170 | std::shared_ptr<sdbusplus::asio::dbus_interface> iface; |
| 171 | std::string level; |
| 172 | std::string alarm; |
| 173 | if (threshold.level == thresholds::Level::CRITICAL) |
| 174 | { |
James Feist | 251c782 | 2018-09-12 12:54:15 -0700 | [diff] [blame] | 175 | iface = thresholdInterfaceCritical; |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 176 | if (threshold.direction == thresholds::Direction::HIGH) |
| 177 | { |
| 178 | level = "CriticalHigh"; |
| 179 | alarm = "CriticalAlarmHigh"; |
| 180 | } |
| 181 | else |
| 182 | { |
| 183 | level = "CriticalLow"; |
| 184 | alarm = "CriticalAlarmLow"; |
| 185 | } |
| 186 | } |
| 187 | else if (threshold.level == thresholds::Level::WARNING) |
| 188 | { |
James Feist | 251c782 | 2018-09-12 12:54:15 -0700 | [diff] [blame] | 189 | iface = thresholdInterfaceWarning; |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 190 | if (threshold.direction == thresholds::Direction::HIGH) |
| 191 | { |
| 192 | level = "WarningHigh"; |
| 193 | alarm = "WarningAlarmHigh"; |
| 194 | } |
| 195 | else |
| 196 | { |
| 197 | level = "WarningLow"; |
| 198 | alarm = "WarningAlarmLow"; |
| 199 | } |
| 200 | } |
| 201 | else |
| 202 | { |
| 203 | std::cerr << "Unknown threshold level" << threshold.level << "\n"; |
| 204 | continue; |
| 205 | } |
| 206 | if (!iface) |
| 207 | { |
| 208 | std::cout << "trying to set uninitialized interface\n"; |
| 209 | continue; |
| 210 | } |
| 211 | iface->register_property( |
| 212 | level, threshold.value, |
| 213 | [&](const double &request, double &oldValue) { |
| 214 | oldValue = request; // todo, just let the config do this? |
| 215 | threshold.value = request; |
| 216 | thresholds::persistThreshold(configuration, objectType, |
| 217 | threshold, conn); |
| 218 | return 1; |
| 219 | }); |
| 220 | iface->register_property(alarm, false); |
| 221 | } |
James Feist | 251c782 | 2018-09-12 12:54:15 -0700 | [diff] [blame] | 222 | if (!sensorInterface->initialize()) |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 223 | { |
| 224 | std::cerr << "error initializing value interface\n"; |
| 225 | } |
James Feist | 251c782 | 2018-09-12 12:54:15 -0700 | [diff] [blame] | 226 | if (thresholdInterfaceWarning && !thresholdInterfaceWarning->initialize()) |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 227 | { |
| 228 | std::cerr << "error initializing warning threshold interface\n"; |
| 229 | } |
| 230 | |
James Feist | 251c782 | 2018-09-12 12:54:15 -0700 | [diff] [blame] | 231 | if (thresholdInterfaceCritical && !thresholdInterfaceCritical->initialize()) |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 232 | { |
| 233 | std::cerr << "error initializing critical threshold interface\n"; |
| 234 | } |
| 235 | } |