On Sat Nov 26, 2022 at 3:32 AM AEST, Laurent Dufour wrote:
> The RCU watchdog timer should be reset when restarting the CPU after a Live
> Partition Mobility operation.
>
> Signed-off-by: Laurent Dufour <lduf...@linux.ibm.com>

Looks okay to me. xmon touches the softlockup watchdog explicitly but
is that for architectures with unsynchronized clocks maybe.

Acked-by: Nicholas Piggin <npig...@gmail.com>

> ---
>  arch/powerpc/platforms/pseries/mobility.c | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/arch/powerpc/platforms/pseries/mobility.c 
> b/arch/powerpc/platforms/pseries/mobility.c
> index 634fac5db3f9..9e10f38dd9ad 100644
> --- a/arch/powerpc/platforms/pseries/mobility.c
> +++ b/arch/powerpc/platforms/pseries/mobility.c
> @@ -636,8 +636,10 @@ static int do_join(void *arg)
>       }
>       /*
>        * Execution may have been suspended for several seconds, so
> -      * reset the watchdog.
> +      * reset the watchdogs.
>        */
> +     rcu_cpu_stall_reset();
> +     /* touch_nmi_watchdog() also touch the soft lockup watchdog */
>       touch_nmi_watchdog();
>       return ret;
>  }
> -- 
> 2.38.1

Reply via email to