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.carpen...@oracle.com>
> Signed-off-by: Michael Ellerman <m...@ellerman.id.au>
Applied to powerpc next.
https://git.kernel.org/powerpc/c/686978b15cef8314c3b3f38f0124e0
cheers
