[Skiboot] [PATCH 3/9] ipmi-watchdog: Don't reset the watchdog twice

Alistair Popple alistair at popple.id.au
Fri May 25 10:28:43 AEST 2018


On Wednesday, 23 May 2018 5:13:29 PM AEST William A. Kennington III wrote:
> There is no clarification for why this change was needed, but presumably
> this is due to a buggy BMC implementation where the Watchdog Set command

That is correct, although I'm not sure if the buggy BMC implementation
was ever fixed so we should probably test this on a P8 box as well.

- Alistair

> was processed concurrently or after the initial Watchdog Reset. This
> inversion would cause the watchdog to stop since the DONT_STOP bit was
> not set. Since we are now using the DONT_STOP bit during initialization,
> the watchdog should not be stopped even if an inversion occurs.
> 
> Signed-off-by: William A. Kennington III <wak at google.com>
> ---
>  hw/ipmi/ipmi-watchdog.c | 4 ----
>  1 file changed, 4 deletions(-)
> 
> diff --git a/hw/ipmi/ipmi-watchdog.c b/hw/ipmi/ipmi-watchdog.c
> index ee1608443..e8efba2b4 100644
> --- a/hw/ipmi/ipmi-watchdog.c
> +++ b/hw/ipmi/ipmi-watchdog.c
> @@ -143,9 +143,5 @@ void ipmi_wdt_init(void)
>  	 * could crash before the wdt has actually been started. */
>  	sync_reset_wdt();
>  
> -	/* For some reason we have to reset it twice to get it to
> -	 * actually start the first time. */
> -	sync_reset_wdt();
> -
>  	return;
>  }
> 




More information about the Skiboot mailing list