* [phosphor-sel-logger] Watchdog Event Monitor Usage
@ 2021-08-09 20:38 Willy Tu
2021-08-11 4:55 ` Willy Tu
0 siblings, 1 reply; 2+ messages in thread
From: Willy Tu @ 2021-08-09 20:38 UTC (permalink / raw)
To: OpenBMC Maillist; +Cc: Bills, Jason M, William Kennington
[-- 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 --]
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [phosphor-sel-logger] Watchdog Event Monitor Usage
2021-08-09 20:38 [phosphor-sel-logger] Watchdog Event Monitor Usage Willy Tu
@ 2021-08-11 4:55 ` Willy Tu
0 siblings, 0 replies; 2+ messages in thread
From: Willy Tu @ 2021-08-11 4:55 UTC (permalink / raw)
To: OpenBMC Maillist; +Cc: Bills, Jason M, Charles.Hsu, William Kennington
[-- 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 --]
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2021-08-11 4:56 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-08-09 20:38 [phosphor-sel-logger] Watchdog Event Monitor Usage Willy Tu
2021-08-11 4:55 ` Willy Tu
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).