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

2017-04-20 Thread Michael Ellerman
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 b

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

2017-04-19 Thread Michael Ellerman
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