[External] Power-on monitor for phosphor-hwmon question

[James Feist]

On 11/5/19 11:25 PM, Ivan Li11 wrote:
>> -----Original Message-----
>> From: James Feist <james.feist at linux.intel.com>
>> Sent: Tuesday, October 22, 2019 11:51 PM
>> To: Matt Spinler <mspinler at linux.ibm.com>; Ivan Li11 <rli11 at lenovo.com>;
>> openbmc at lists.ozlabs.org
>> Subject: Re: [External] Power-on monitor for phosphor-hwmon question
>>
>> On 10/22/19 6:23 AM, Matt Spinler wrote:
>>>
>>>
>>> On 10/21/2019 6:02 AM, Ivan Li11 wrote:
>>>>
>>>> Hi Team,
>>>>
>>>> Does anyone have suggestion for this ?
>>>>
>>> Hi, you're right, it doesn't support that yet.  We've had a few
>>> internal discussions about how to handle that gracefully, though our
>>> platforms don't use sel-logger, but nothing solid came out of them and
>>> we don't have anything scheduled yet.
>>>
>>> It would be easy enough to have a watch on the PGOOD property, and
>>> then do you what you want, like maybe turning off the thresholds.
>>>
>>> I heard that the d-bus sensors repository has some sort of support for
>>> this, but since we don't use it I'm not sure what all that entails. I
>>> think it may also force you to replace phosphor-inventor-manager with
>>> entity-manager.
>>
>> It should force you to switch 100%, but you would at least have to run both.
>>
>> Here's what dbus-sensors does:
>>
>> https://github.com/openbmc/dbus-sensors/blob/432d1edf7ac86f69558273307
>> a59e4b1cf86b8a6/src/Utils.cpp#L141
>>
>> Basically just a dbus-match that sets a "powerStatusOn" bool that can be
>> queried. Depending on the sensor type thresholds don't get crossed if that bool
>> is set. We also have a similar bool for bios post, as some sensors come up later
>> after power on.
>>
> 
> I try to use dbus-sensors and set "powerStatusOn" bool, but still have incorrect threshold logs during power on(S0) to power off(S5) state.
> Could you help to comment on it ?
> 

This has only been tested with x86-power-control, are you using that 
project for power control or something else?

>>>
>>>> Thanks your great support in advance.
>>>>
>>>> *From:*openbmc <openbmc-bounces+rli11=lenovo.com at lists.ozlabs.org>
>> *On
>>>> Behalf Of *Ivan Li11
>>>> *Sent:* Friday, October 18, 2019 5:59 PM
>>>> *To:* openbmc at lists.ozlabs.org
>>>> *Subject:* [External] Power-on monitor for phosphor-hwmon question
>>>>
>>>> Hi Team,
>>>>
>>>> We found that there's no power-on monitor mechanism in
>> phosphor-hwmon.
>>>>
>>>> https://github.com/openbmc/phosphor-hwmon
>>>>
>>>> And it will cause incorrect threshold logs to be added to journal log
>>>> by 'sel-logger' when system is in power off(S5) state.
>>>>
>>>> Is there any plan to implement it? Or other repository we can refer to ?
>>>>
>>>> Thanks,
>>>>
>>>> Ivan
>>>>
>>>