[Skiboot] [PATCH] hw/p8-i2c: Fix deadlock in p9_i2c_bus_owner_change

[Stewart Smith]

Anton Blanchard <anton at ozlabs.org> writes:
> From: Anton Blanchard <anton at samba.org>
>
> When debugging a system where Linux was taking soft lockup errors, I
> noticed two CPUs were stuck in OPAL:
>
> CPU0
> lock
> p8_i2c_recover
> opal_handle_interrupt
>
> CPU1
> sync_timer
> cancel_timer
> p9_i2c_bus_owner_change
> occ_p9_interrupt
> xive_source_interrupt
> opal_handle_interrupt
>
> p8_i2c_recover() is a timer, and is stuck trying to take master->lock.
> p9_i2c_bus_owner_change() has taken master->lock, but then is stuck waiting
> for all timers to complete. We deadlock.
>
> Fix this by using cancel_timer_async(), as suggested by Oliver.
>
> Fixes: 201fd50f208d ("hw/p8-i2c: Fix OCC locking")
> Signed-off-by: Anton Blanchard <anton at samba.org>

Thanks!

Merged to master as of 4466a6ec89bf39afeb98b92d03bbf7fc30c55afe

Added the Suggested-by for Oliver as the patches were exactly the same.

-- 
Stewart Smith
OPAL Architect, IBM.