[phosphor-sel-logger] Watchdog Event Monitor Usage

[phosphor-sel-logger] Watchdog Event Monitor Usage

[Willy Tu]

[-- Attachment #1: Type: text/plain, Size: 764 bytes --]

Hi all,

I am writing this to see if anyone is using the Watchdog Event Monitor
option in phosphor-sel-logger.

Option: `log-watchdog`

I have made a few changes to `phosphor-watchdog` that sends out Timeout
signals during timeout events.

See https://gerrit.openbmc-project.xyz/c/openbmc/phosphor-watchdog/+/44749

I am thinking of replacing the existing watchdog event to check that signal
instead or checking for the `Enabled` property change.

See https://gerrit.openbmc-project.xyz/c/openbmc/phosphor-sel-logger/+/45622

I think the existing change is wrong, since changing the `Enabled` property
does not necessary mean the watchdog event got triggered.

So I am wondering if anyone needs the watchdog event monitor? or if I can
replace it.

Best,

Willy Tu

[-- Attachment #2: Type: text/html, Size: 1209 bytes --]

Re: [phosphor-sel-logger] Watchdog Event Monitor Usage

[Willy Tu]

[-- Attachment #1: Type: text/plain, Size: 1029 bytes --]

Just a friendly ping on this topic.

If no one has any concern, then I will replace it.

Adding `Charles.Hsu@quantatw.com` since he added this feature.

On Mon, Aug 9, 2021 at 1:38 PM Willy Tu <wltu@google.com> wrote:

> Hi all,
>
> I am writing this to see if anyone is using the Watchdog Event Monitor
> option in phosphor-sel-logger.
>
> Option: `log-watchdog`
>
> I have made a few changes to `phosphor-watchdog` that sends out Timeout
> signals during timeout events.
>
> See https://gerrit.openbmc-project.xyz/c/openbmc/phosphor-watchdog/+/44749
>
> I am thinking of replacing the existing watchdog event to check that
> signal instead or checking for the `Enabled` property change.
>
> See
> https://gerrit.openbmc-project.xyz/c/openbmc/phosphor-sel-logger/+/45622
>
> I think the existing change is wrong, since changing the `Enabled`
> property does not necessary mean the watchdog event got triggered.
>
> So I am wondering if anyone needs the watchdog event monitor? or if I can
> replace it.
>
> Best,
>
> Willy Tu
>

[-- Attachment #2: Type: text/html, Size: 1829 bytes --]