[Skiboot] [PATCH 0/9] ipmi-watchdog: Fixes for error handling and general cleanups

Stewart Smith stewart at linux.ibm.com
Tue Jun 5 14:23:44 AEST 2018

"William A. Kennington III" <wak at google.com> writes:
> The current watchdog implementation doesn't do a good job of handling
> error cases when the BMC side restarts or crashes. This set of patches
> tries to improve the robustness of the ipmi watchdog code such that
> recovery happens when possible.
> This series also adds a patch which enables the watchdog for the KEXEC
> payload since the kernel being executed is guaranteed to support handling
> the watchdog.
> It also adds some general cleanups to the code that made the above
> easier to implement.
> William A. Kennington III (9):
>   ipmi-watchdog: Make it possible to set DONT_STOP
>   ipmi-watchdog: Don't reset the watchdog twice

I kind of meant to try this on a p8 with dodgy BMC before I hit merge,
but I didn't. So, I'll try now and see how it goes.

>   ipmi-watchdog: Don't disable at shutdown

I've added in 5b70462c73a803d15982fe6f2c6dad89b8a9c962 to not run with
it enabled as we exit by default, as I just want to give some time for
people to catch up with their BOOTKERNELs.

I'm not sure when is teh best time to revert my patch and force the
issue though... maybe in an op-build cycle or two?

>   ipmi-watchdog: Add a flag to determine if we are still ticking
>   ipmi-watchdog: The stop action should disable reset
>   ipmi-watchdog: Simplify our completion function
>   ipmi-watchdog: Support resetting the watchdog after set
>   ipmi-watchdog: Support handling re-initialization

Anyway, series merged to master as of e6e74c53ed64eb029cb669fbb6715ee4077cf0b2

Stewart Smith
OPAL Architect, IBM.

More information about the Skiboot mailing list