commit | 0f580944e22534c8814634d76bd752a5d4236f60 | [log] [tgz] |
---|---|---|
author | Willy Tu <wltu@google.com> | Wed Apr 06 11:03:32 2022 -0700 |
committer | Willy Tu <wltu@google.com> | Wed Apr 06 19:43:39 2022 +0000 |
tree | 1558afef1d118ba057554f70503b31490da82527 | |
parent | a7afacc4fa5ee4da5b10b2fd47e29a6342460a99 [diff] |
hwmon-temp-sensor: Support sensor with missing labels Currently the hwmon temp sensors forces all sensor labels to be required even when we don't want to expose all of them. For example, ``` { "Address": "0x4e", "Bus": "$bus", "Name": "test0", "Name1": "test1", "Name2": "test2", "Name3": "test3", "Labels": ["temp4"], "Type": "MAX31730" }, ``` It will have ``` $ /tmp/hwmontempsensor & error getting SpecialMode status No route to host malloc(): unsorted double linked list corrupted -sh: line 438: 20466 Aborted (core dumped) /tmp/hwmontempsensor ``` It needs the `Labels` to be `["temp1, "temp2", "temp3", "temp4"]` to work properly. With the fix to the daemon it doesn't need it anymore and we can expose only the sensor that we want. Tested: Working and didn't crash ``` busctl tree xyz.openbmc_project.HwmonTempSensor `-/xyz `-/xyz/openbmc_project `-/xyz/openbmc_project/sensors `-/xyz/openbmc_project/sensors/temperature `-/xyz/openbmc_project/sensors/temperature/test3 ``` Change-Id: I73d236cf36f22ec0e64a1603b83b14e29e56d36b Signed-off-by: Willy Tu <wltu@google.com>
dbus-sensors is a collection of sensor applications that provide the xyz.openbmc_project.Sensor collection of interfaces. They read sensor values from hwmon, d-bus, or direct driver access to provide readings. Some advance non-sensor features such as fan presence, pwm control, and automatic cpu detection (x86) are also supported.
runtime re-configurable from d-bus (entity-manager or the like)
isolated: each sensor type is isolated into its own daemon, so a bug in one sensor is unlikely to affect another, and single sensor modifications are possible
async single-threaded: uses sdbusplus/asio bindings
multiple data inputs: hwmon, d-bus, direct driver access