powerpc/xmon: Fix invocation inside lock region

Michael Ellerman patch-notifications at ellerman.id.au
Sat Dec 8 00:07:01 AEDT 2018


On Thu, 2018-11-08 at 17:12:42 UTC, Breno Leitao wrote:
> Currently xmon needs to get devtree_lock (through rtas_token()) during its
> invocation (at crash time). If there is a crash while devtree_lock is being
> held, then xmon tries to get the lock but spins forever and never get into
> the interactive debugger, as in the following case:
> 
> 	int *ptr = NULL;
> 	raw_spin_lock_irqsave(&devtree_lock, flags);
> 	*ptr = 0xdeadbeef;
> 
> This patch avoids calling rtas_token(), thus trying to get the same lock,
> at crash time. This new mechanism proposes getting the token at
> initialization time (xmon_init()) and just consuming it at crash time.
> 
> This would allow xmon to be possible invoked independent of devtree_lock
> being held or not.
> 
> Signed-off-by: Breno Leitao <leitao at debian.org>
> Reviewed-by: Thiago Jung Bauermann <bauerman at linux.ibm.com>

Applied to powerpc next, thanks.

https://git.kernel.org/powerpc/c/8d4a862276a9c30a269d368d324fb5

cheers


More information about the Linuxppc-dev mailing list