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