Changes in kernel/generic/src/proc/scheduler.c [43ac0cc:26aafe8] in mainline

File:

• kernel/generic/src/proc/scheduler.c (modified) (3 diffs)

kernel/generic/src/proc/scheduler.c

@@ -586 +586 @@
          * Searching least priority queues on all CPU's first and most priority
          * queues on all CPU's last.
+         *
          */
         size_t acpu;
@@ -619 +620 @@
                         
                         while (link != &(cpu->rq[rq].rq_head)) {
-                                thread = (thread_t *) list_get_instance(link,
-                                    thread_t, rq_link);
+                                thread = (thread_t *) list_get_instance(link, thread_t, rq_link);
                                 
                                 /*
-                                 * Do not steal CPU-wired threads, threads
-                                 * already stolen, threads for which migration
-                                 * was temporarily disabled or threads whose
-                                 * FPU context is still in the CPU.
+                                 * We don't want to steal CPU-wired threads
+                                 * neither threads already stolen. The latter
+                                 * prevents threads from migrating between CPU's
+                                 * without ever being run. We don't want to
+                                 * steal threads whose FPU context is still in
+                                 * CPU.
+                                 *
                                  */
                                 irq_spinlock_lock(&thread->lock, false);
                                 
-                                if (!(thread->flags & THREAD_FLAG_WIRED) &&
-                                    !(thread->flags & THREAD_FLAG_STOLEN) &&
-                                    !thread->nomigrate &&
-                                    !thread->fpu_context_engaged) {
+                                if ((!(thread->flags & (THREAD_FLAG_WIRED | THREAD_FLAG_STOLEN)))
+                                    && (!(thread->fpu_context_engaged))) {
                                         /*
                                          * Remove thread from ready queue.
                                          */
-                                        irq_spinlock_unlock(&thread->lock,
-                                            false);
+                                        irq_spinlock_unlock(&thread->lock, false);
                                         
                                         atomic_dec(&cpu->nrdy);
@@ -660 +660 @@
                                  */
                                 
-                                irq_spinlock_pass(&(cpu->rq[rq].lock),
-                                    &thread->lock);
+                                irq_spinlock_pass(&(cpu->rq[rq].lock), &thread->lock);
                                 
 #ifdef KCPULB_VERBOSE