| commit | 9eb0cc3d8272bcdc58ead70c9a39fe8cecd5b6b5 | [log] [tgz] |
|---|---|---|
| author | Willy Tu <wltu@google.com> | Wed Apr 06 11:03:32 2022 -0700 |
| committer | Willy Tu <wltu@google.com> | Wed May 11 19:46:37 2022 +0000 |
| tree | d0c996dcd249e5a8d49db7c85aadc632d965113f | |
| parent | 75872ef8011c22a419fbc0cff50aad8ae68faa4c [diff] |
hwmon-temp-sensor: Support sensor with missing labels
Erase sensorName from hwmonName even if it is not included in the
`Labels` property. This makes sure that all hwmonNames that it parsed
will be removed and allows us to support missing labels when we don't
want to expose all sensors.
Currently the hwmon temp sensors forces all sensor labels to be
required even when we don't want to expose all of them.
With
```
{
"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"]` for
this to work properly.
With the fix to the daemon it doesn't need it anymore.
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: I926b76dbb2777bc3e0f273b001139cd468617e63
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