commit | 8454571297812c51ae120812d245bbe88d697191 | [log] [tgz] |
---|---|---|
author | Arun Lal K M <arun.lal@intel.com> | Fri Dec 31 13:29:56 2021 +0000 |
committer | Arun Lal K M <arun.lal@intel.com> | Tue Apr 19 06:49:51 2022 +0000 |
tree | 302a43650648835ff3e9027deb847fd5c9667c2b | |
parent | 31ec7dbb6ae668b4c7bb4f9d630f80253b23f0d8 [diff] |
Fix for cpusensor going into D state. When PECI driver returns the error code EAGAIN, async_read_until will go into a keep trying again loop. As async_read_until does not handle EAGAIN reliably. Same issue was identified and fix for PSUSensor: https://github.com/openbmc/dbus-sensors/commit/bcf76717697238e380be785bd28150963ecf0e9e Fix for this is to use async_wait instead of async_read_until. Tested: In Intel system verified cpusensor daemon is not getting into D state when there is a peci error EAGAIN. Verified sensors are getting populated as expected when there is no PECI error. Signed-off-by: Arun Lal K M <arun.lal@intel.com> Signed-off-by: Arun P. Mohanan <arun.p.m@linux.intel.com> Change-Id: I52e35075e522d0ae0d99e1c893db76156e299871
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