powerpc/xive: Fix missing check of rc != OPAL_BUSY

Michael Ellerman patch-notifications at ellerman.id.au
Fri Apr 21 14:09:43 AEST 2017


On Thu, 2017-04-20 at 05:55:55 UTC, Michael Ellerman wrote:
> Dan Carpenter noticed that the code in __xive_native_disable_queue() has a for
> loop with an unconditional break in the middle, which doesn't make a lot of
> sense.
> 
> What the code's supposed to do is loop as long as OPAL says it's busy, if we get
> any other return code, either success or failure, then we should break the loop.
> 
> So add the missing check.
> 
> Fixes: 243e25112d06 ("powerpc/xive: Native exploitation of the XIVE interrupt controller")
> Reported-by: Dan Carpenter <dan.carpenter at oracle.com>
> Signed-off-by: Michael Ellerman <mpe at ellerman.id.au>

Applied to powerpc next.

https://git.kernel.org/powerpc/c/686978b15cef8314c3b3f38f0124e0

cheers


More information about the Linuxppc-dev mailing list