<div dir="ltr">Just a friendly ping on this topic.<div><br></div><div>If no one has any concern, then I will replace it.</div><div><br></div><div>Adding `<a href="mailto:Charles.Hsu@quantatw.com">Charles.Hsu@quantatw.com</a>` since he added this feature.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Aug 9, 2021 at 1:38 PM Willy Tu <<a href="mailto:wltu@google.com">wltu@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">Hi all,<br><div><br></div><div>I am writing this to see if anyone is using the Watchdog Event Monitor option in phosphor-sel-logger.</div><div><br></div><div>Option: `log-watchdog`</div><div><br></div><div>I have made a few changes to `phosphor-watchdog` that sends out Timeout signals during timeout events.</div><div><br></div><div>See <a href="https://gerrit.openbmc-project.xyz/c/openbmc/phosphor-watchdog/+/44749" target="_blank">https://gerrit.openbmc-project.xyz/c/openbmc/phosphor-watchdog/+/44749</a><br></div><div><br></div><div>I am thinking of replacing the existing watchdog event to check that signal instead or checking for the `Enabled` property change.</div><div><br></div><div>See <a href="https://gerrit.openbmc-project.xyz/c/openbmc/phosphor-sel-logger/+/45622" target="_blank">https://gerrit.openbmc-project.xyz/c/openbmc/phosphor-sel-logger/+/45622</a></div><div><br></div><div>I think the existing change is wrong, since changing the `Enabled` property does not necessary mean the watchdog event got triggered.</div><div><br></div><div>So I am wondering if anyone needs the watchdog event monitor? or if I can replace it.</div><div><br></div><div>Best,</div><div><br></div><div>Willy Tu</div></div>
</blockquote></div>