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 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 mailing list