[POWERPC] spufs: spu_find_victim may choose wrong victim

Need to re-check priority after dropping lock.  Otherwise, a
more favored context may be preempted.

Signed-off-by: Luke Browning <lukebr@linux.vnet.ibm.com>
Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
Signed-off-by: Paul Mackerras <paulus@samba.org>

1 files changed, 2 insertions, 2 deletions

diff --git a/arch/powerpc/platforms/cell/spufs/sched.c b/arch/powerpc/platforms/cell/spufs/sched.c
index ef0e5e230fbb..54b338f8363f 100644
--- a/arch/powerpc/platforms/cell/spufs/sched.c
+++ b/arch/powerpc/platforms/cell/spufs/sched.c
@@ -626,10 +626,10 @@ static struct spu *find_victim(struct spu_context *ctx)
                         }
 
                         spu = victim->spu;
-                        if (!spu) {
+                        if (!spu || victim->prio <= ctx->prio) {
                                 /*
                                  * This race can happen because we've dropped
-                                 * the active list mutex.  No a problem, just
+                                 * the active list mutex.  Not a problem, just
                                  * restart the search.
                                  */
                                 mutex_unlock(&victim->state_mutex);