dbus-sensor: setting the upper non-critical(unc) threshold value smaller than reading value would get 3 logs ( assert log, de-assert log and then assert log)
james.feist at linux.intel.com
Tue Feb 25 07:37:59 AEDT 2020
On 2/24/20 3:31 AM, Max Lai/WYHQ/Wiwynn wrote:
> Hi James,
> We had tried your fix solution (Deleting lines 117 and 118). Deleting
> the lines 117 and 118 would stop sending the PropertiesChanged signal
> and even stop updating threshold value on Dbus. The result we want is we
> can change threshold value on Dbus and get assert sel log when we
> trigger the threshold mechanism. And we also tried the latest source
> revision on upstream dbus-sensor repository. We found that latest source
> revision in IpmbSensor.cpp, struct sensor's "objectType" member which
> was set "xyz.openbmc_project.Configuration.ExitAirTemp" was different
> than our "xyz.openbmc_project.EntityManager". So this issue doesn’t happen.
> What's the purpose of this changing?
Can you send a diff? I'm not exactly sure the line you're mentioning.
More information about the openbmc