alarm: Do not hold the tchain lock during handlers
[akaros.git] / kern / src / schedule.c
index 5edd58b..4804885 100644 (file)
@@ -4,11 +4,8 @@
  *
  * Scheduling and dispatching. */
 
-#ifdef __SHARC__
-#pragma nosharc
-#endif
-
 #include <schedule.h>
+#include <corerequest.h>
 #include <process.h>
 #include <monitor.h>
 #include <stdio.h>
@@ -18,8 +15,8 @@
 #include <manager.h>
 #include <alarm.h>
 #include <sys/queue.h>
-#include <kmalloc.h>
 #include <arsc_server.h>
+#include <hashtable.h>
 
 /* Process Lists.  'unrunnable' is a holding list for SCPs that are running or
  * waiting or otherwise not considered for sched decisions. */
@@ -32,26 +29,12 @@ struct proc_list all_mcps_2 = TAILQ_HEAD_INITIALIZER(all_mcps_2);
 struct proc_list *primary_mcps = &all_mcps_1;
 struct proc_list *secondary_mcps = &all_mcps_2;
 
-/* The pcores in the system.  (array gets alloced in init()).  */
-struct sched_pcore *all_pcores;
-
-/* TAILQ of all unallocated, idle (CG) cores */
-struct sched_pcore_tailq idlecores = TAILQ_HEAD_INITIALIZER(idlecores);
-
 /* Helper, defined below */
 static void __core_request(struct proc *p, uint32_t amt_needed);
-static void __put_idle_cores(struct proc *p, uint32_t *pc_arr, uint32_t num);
 static void add_to_list(struct proc *p, struct proc_list *list);
 static void remove_from_list(struct proc *p, struct proc_list *list);
 static void switch_lists(struct proc *p, struct proc_list *old,
                          struct proc_list *new);
-static uint32_t spc2pcoreid(struct sched_pcore *spc);
-static struct sched_pcore *pcoreid2spc(uint32_t pcoreid);
-static bool is_ll_core(uint32_t pcoreid);
-static void __prov_track_alloc(struct proc *p, uint32_t pcoreid);
-static void __prov_track_dealloc(struct proc *p, uint32_t pcoreid);
-static void __prov_track_dealloc_bulk(struct proc *p, uint32_t *pc_arr,
-                                      uint32_t nr_cores);
 static void __run_mcp_ksched(void *arg);       /* don't call directly */
 static uint32_t get_cores_needed(struct proc *p);
 
@@ -60,7 +43,7 @@ static uint32_t get_cores_needed(struct proc *p);
 /* poke-style ksched - ensures the MCP ksched only runs once at a time.  since
  * only one mcp ksched runs at a time, while this is set, the ksched knows no
  * cores are being allocated by other code (though they could be dealloc, due to
- * yield). 
+ * yield).
  *
  * The main value to this sync method is to make the 'make sure the ksched runs
  * only once at a time and that it actually runs' invariant/desire wait-free, so
@@ -68,7 +51,7 @@ static uint32_t get_cores_needed(struct proc *p);
  *
  * As the ksched gets smarter, we'll probably embedd this poker in a bigger
  * struct that can handle the posting of different types of work. */
-struct poke_tracker ksched_poker = {0, 0, __run_mcp_ksched};
+struct poke_tracker ksched_poker = POKE_INITIALIZER(__run_mcp_ksched);
 
 /* this 'big ksched lock' protects a bunch of things, which i may make fine
  * grained: */
@@ -76,12 +59,10 @@ struct poke_tracker ksched_poker = {0, 0, __run_mcp_ksched};
  * of a proc on those lists.  proc lifetime within the ksched but outside this
  * lock is protected by the proc kref. */
 //spinlock_t proclist_lock = SPINLOCK_INITIALIZER; /* subsumed by bksl */
-/* - protects the provisioning assignment, membership of sched_pcores in
- * provision lists, and the integrity of all prov lists (the lists of each
- * proc).  does NOT protect spc->alloc_proc. */
+/* - protects the provisioning assignment, and the integrity of all prov
+ * lists (the lists of each proc). */
 //spinlock_t prov_lock = SPINLOCK_INITIALIZER;
-/* - protects allocation structures: spc->alloc_proc, the integrity and
- * membership of the idelcores tailq. */
+/* - protects allocation structures */
 //spinlock_t alloc_lock = SPINLOCK_INITIALIZER;
 spinlock_t sched_lock = SPINLOCK_INITIALIZER;
 
@@ -98,12 +79,6 @@ static void set_ksched_alarm(void)
        set_alarm(&per_cpu_info[core_id()].tchain, &ksched_waiter);
 }
 
-/* Need a kmsg to just run the sched, but not to rearm */
-static void __just_sched(uint32_t srcid, long a0, long a1, long a2)
-{
-       run_scheduler();
-}
-
 /* RKM alarm, to run the scheduler tick (not in interrupt context) and reset the
  * alarm.  Note that interrupts will be disabled, but this is not the same as
  * interrupt context.  We're a routine kmsg, which means the core is in a
@@ -112,36 +87,28 @@ static void __ksched_tick(struct alarm_waiter *waiter)
 {
        /* TODO: imagine doing some accounting here */
        run_scheduler();
-       /* Set our alarm to go off, incrementing from our last tick (instead of
-        * setting it relative to now, since some time has passed since the alarm
-        * first went off.  Note, this may be now or in the past! */
-       set_awaiter_inc(&ksched_waiter, TIMER_TICK_USEC);
+       /* Set our alarm to go off, relative to now.  This means we might lag a bit,
+        * and our ticks won't match wall clock time.  But if we do incremental,
+        * we'll actually punish the next process because the kernel took too long
+        * for the previous process.  Ultimately, if we really care, we should
+        * account for the actual time used. */
+       set_awaiter_rel(&ksched_waiter, TIMER_TICK_USEC);
        set_alarm(&per_cpu_info[core_id()].tchain, &ksched_waiter);
 }
 
 void schedule_init(void)
 {
        spin_lock(&sched_lock);
-       /* init provisioning stuff */
-       all_pcores = kmalloc(sizeof(struct sched_pcore) * num_cpus, 0);
-       memset(all_pcores, 0, sizeof(struct sched_pcore) * num_cpus);
        assert(!core_id());             /* want the alarm on core0 for now */
        init_awaiter(&ksched_waiter, __ksched_tick);
        set_ksched_alarm();
-       /* init the idlecore list.  if they turned off hyperthreading, give them the
-        * odds from 1..max-1.  otherwise, give them everything by 0 (default mgmt
-        * core).  TODO: (CG/LL) better LL/CG mgmt */
-#ifndef CONFIG_DISABLE_SMT
-       for (int i = 1; i < num_cpus; i++)
-               TAILQ_INSERT_TAIL(&idlecores, pcoreid2spc(i), alloc_next);
-#else
-       assert(!(num_cpus % 2));
-       for (int i = 1; i < num_cpus; i += 2)
-               TAILQ_INSERT_TAIL(&idlecores, pcoreid2spc(i), alloc_next);
-#endif /* CONFIG_DISABLE_SMT */
+       corealloc_init();
        spin_unlock(&sched_lock);
 
 #ifdef CONFIG_ARSC_SERVER
+       /* Most likely we'll have a syscall and a process that dedicates itself to
+        * running this.  Or if it's a kthread, we don't need a core. */
+       #error "Find a way to get a core.  Probably a syscall to run a server."
        int arsc_coreid = get_any_idle_core();
        assert(arsc_coreid >= 0);
        send_kernel_message(arsc_coreid, arsc_server, 0, 0, 0, KMSG_ROUTINE);
@@ -149,16 +116,6 @@ void schedule_init(void)
 #endif /* CONFIG_ARSC_SERVER */
 }
 
-static uint32_t spc2pcoreid(struct sched_pcore *spc)
-{
-       return spc - all_pcores;
-}
-
-static struct sched_pcore *pcoreid2spc(uint32_t pcoreid)
-{
-       return &all_pcores[pcoreid];
-}
-
 /* Round-robins on whatever list it's on */
 static void add_to_list(struct proc *p, struct proc_list *new)
 {
@@ -202,12 +159,11 @@ static void remove_from_any_list(struct proc *p)
  *   DYING */
 void __sched_proc_register(struct proc *p)
 {
-       assert(p->state != PROC_DYING); /* shouldn't be abel to happen yet */
+       assert(!proc_is_dying(p));              /* shouldn't be able to happen yet */
        /* one ref for the proc's existence, cradle-to-grave */
        proc_incref(p, 1);      /* need at least this OR the 'one for existing' */
        spin_lock(&sched_lock);
-       TAILQ_INIT(&p->ksched_data.prov_alloc_me);
-       TAILQ_INIT(&p->ksched_data.prov_not_alloc_me);
+       corealloc_proc_init(p);
        add_to_list(p, &unrunnable_scps);
        spin_unlock(&sched_lock);
 }
@@ -219,7 +175,7 @@ void __sched_proc_change_to_m(struct proc *p)
        /* Need to make sure they aren't dying.  if so, we already dealt with their
         * list membership, etc (or soon will).  taking advantage of the 'immutable
         * state' of dying (so long as refs are held). */
-       if (p->state == PROC_DYING) {
+       if (proc_is_dying(p)) {
                spin_unlock(&sched_lock);
                return;
        }
@@ -237,20 +193,6 @@ void __sched_proc_change_to_m(struct proc *p)
        //poke_ksched(p, RES_CORES);
 }
 
-/* Helper for the destroy CB : unprovisions any pcores for the given list */
-static void unprov_pcore_list(struct sched_pcore_tailq *list_head)
-{
-       struct sched_pcore *spc_i;
-       /* We can leave them connected within the tailq, since the scps don't have a
-        * default list (if they aren't on a proc's list, then we don't care about
-        * them), and since the INSERTs don't care what list you were on before
-        * (chummy with the implementation).  Pretty sure this is right.  If there's
-        * suspected list corruption, be safer here. */
-       TAILQ_FOREACH(spc_i, list_head, prov_next)
-               spc_i->prov_proc = 0;
-       TAILQ_INIT(list_head);
-}
-
 /* Sched callback called when the proc dies.  pc_arr holds the cores the proc
  * had, if any, and nr_cores tells us how many are in the array.
  *
@@ -259,18 +201,15 @@ static void unprov_pcore_list(struct sched_pcore_tailq *list_head)
 void __sched_proc_destroy(struct proc *p, uint32_t *pc_arr, uint32_t nr_cores)
 {
        spin_lock(&sched_lock);
-       /* Unprovision any cores.  Note this is different than track_dealloc.
+       /* Unprovision any cores.  Note this is different than track_core_dealloc.
         * The latter does bookkeeping when an allocation changes.  This is a
         * bulk *provisioning* change. */
-       unprov_pcore_list(&p->ksched_data.prov_alloc_me);
-       unprov_pcore_list(&p->ksched_data.prov_not_alloc_me);
+       __unprovision_all_cores(p);
        /* Remove from whatever list we are on (if any - might not be on one if it
         * was in the middle of __run_mcp_sched) */
        remove_from_any_list(p);
-       if (nr_cores) {
-               __put_idle_cores(p, pc_arr, nr_cores);
-               __prov_track_dealloc_bulk(p, pc_arr, nr_cores);
-       }
+       if (nr_cores)
+               __track_core_dealloc_bulk(p, pc_arr, nr_cores);
        spin_unlock(&sched_lock);
        /* Drop the cradle-to-the-grave reference, jet-li */
        proc_decref(p);
@@ -280,7 +219,7 @@ void __sched_proc_destroy(struct proc *p, uint32_t *pc_arr, uint32_t nr_cores)
 void __sched_mcp_wakeup(struct proc *p)
 {
        spin_lock(&sched_lock);
-       if (p->state == PROC_DYING) {
+       if (proc_is_dying(p)) {
                spin_unlock(&sched_lock);
                return;
        }
@@ -294,7 +233,7 @@ void __sched_mcp_wakeup(struct proc *p)
 void __sched_scp_wakeup(struct proc *p)
 {
        spin_lock(&sched_lock);
-       if (p->state == PROC_DYING) {
+       if (proc_is_dying(p)) {
                spin_unlock(&sched_lock);
                return;
        }
@@ -308,10 +247,6 @@ void __sched_scp_wakeup(struct proc *p)
        if (!management_core()) {
                /* TODO: pick a better core and only send if halted.
                 *
-                * FYI, a POKE on x86 might lose a rare race with halt code, since the
-                * poke handler does not abort halts.  if this happens, the next timer
-                * IRQ would wake up the core.
-                *
                 * ideally, we'd know if a specific mgmt core is sleeping and wake it
                 * up.  o/w, we could interrupt an already-running mgmt core that won't
                 * get to our new proc anytime soon.  also, by poking core 0, a
@@ -328,33 +263,16 @@ void __sched_scp_wakeup(struct proc *p)
  * a scheduling decision (or at least plan to). */
 void __sched_put_idle_core(struct proc *p, uint32_t coreid)
 {
-       struct sched_pcore *spc = pcoreid2spc(coreid);
        spin_lock(&sched_lock);
-       TAILQ_INSERT_TAIL(&idlecores, spc, alloc_next);
-       __prov_track_dealloc(p, coreid);
+       __track_core_dealloc(p, coreid);
        spin_unlock(&sched_lock);
 }
 
-/* Helper for put_idle and core_req.  Note this does not track_dealloc.  When we
- * get rid of / revise proc_preempt_all and put_idle_cores, we can get rid of
- * this.  (the ksched will never need it - only external callers). */
-static void __put_idle_cores(struct proc *p, uint32_t *pc_arr, uint32_t num)
-{
-       struct sched_pcore *spc_i;
-       for (int i = 0; i < num; i++) {
-               spc_i = pcoreid2spc(pc_arr[i]);
-               TAILQ_INSERT_TAIL(&idlecores, spc_i, alloc_next);
-       }
-}
-
-/* Callback, bulk interface for put_idle.  Note this one also calls track_dealloc,
- * which the internal version does not.  The proclock is held for this. */
+/* Callback, bulk interface for put_idle. The proclock is held for this. */
 void __sched_put_idle_cores(struct proc *p, uint32_t *pc_arr, uint32_t num)
 {
        spin_lock(&sched_lock);
-       /* TODO: when we revise this func, look at __put_idle */
-       __put_idle_cores(p, pc_arr, num);
-       __prov_track_dealloc_bulk(p, pc_arr, num);
+       __track_core_dealloc_bulk(p, pc_arr, num);
        spin_unlock(&sched_lock);
        /* could trigger a sched decision here */
 }
@@ -368,23 +286,17 @@ static bool __schedule_scp(void)
        struct proc *p;
        uint32_t pcoreid = core_id();
        struct per_cpu_info *pcpui = &per_cpu_info[pcoreid];
-       int8_t state = 0;
        /* if there are any runnables, run them here and put any currently running
         * SCP on the tail of the runnable queue. */
        if ((p = TAILQ_FIRST(&runnable_scps))) {
-               /* protect owning proc, cur_ctx, etc.  note this nests with the
-                * calls in proc_yield_s */
-               disable_irqsave(&state);
                /* someone is currently running, dequeue them */
                if (pcpui->owning_proc) {
                        spin_lock(&pcpui->owning_proc->proc_lock);
                        /* process might be dying, with a KMSG to clean it up waiting on
                         * this core.  can't do much, so we'll attempt to restart */
-                       if (pcpui->owning_proc->state == PROC_DYING) {
-                               send_kernel_message(core_id(), __just_sched, 0, 0, 0,
-                                                   KMSG_ROUTINE);
+                       if (proc_is_dying(pcpui->owning_proc)) {
+                               run_as_rkm(run_scheduler);
                                spin_unlock(&pcpui->owning_proc->proc_lock);
-                               enable_irqsave(&state);
                                return FALSE;
                        }
                        printd("Descheduled %d in favor of %d\n", pcpui->owning_proc->pid,
@@ -393,8 +305,11 @@ static bool __schedule_scp(void)
                        /* Saving FP state aggressively.  Odds are, the SCP was hit by an
                         * IRQ and has a HW ctx, in which case we must save. */
                        __proc_save_fpu_s(pcpui->owning_proc);
-                       __proc_save_context_s(pcpui->owning_proc, pcpui->cur_ctx);
-                       vcore_account_offline(pcpui->owning_proc, 0); /* VC# */
+                       __proc_save_context_s(pcpui->owning_proc);
+                       vcore_account_offline(pcpui->owning_proc, 0);
+                       __seq_start_write(&p->procinfo->coremap_seqctr);
+                       __unmap_vcore(p, 0);
+                       __seq_end_write(&p->procinfo->coremap_seqctr);
                        spin_unlock(&pcpui->owning_proc->proc_lock);
                        /* round-robin the SCPs (inserts at the end of the queue) */
                        switch_lists(pcpui->owning_proc, &unrunnable_scps, &runnable_scps);
@@ -405,12 +320,11 @@ static bool __schedule_scp(void)
                         * proc_run_s would pick it up.  This way is a bit safer for
                         * future changes, but has an extra (empty) TLB flush.  */
                        abandon_core();
-               } 
+               }
                /* Run the new proc */
                switch_lists(p, &runnable_scps, &unrunnable_scps);
                printd("PID of the SCP i'm running: %d\n", p->pid);
                proc_run_s(p);  /* gives it core we're running on */
-               enable_irqsave(&state);
                return TRUE;
        }
        return FALSE;
@@ -499,7 +413,7 @@ static void __run_mcp_ksched(void *arg)
                         * DYING, it'll remain DYING until we decref.  And if there is a
                         * concurrent death, that will spin on the ksched lock (which we
                         * hold, and which protects the proc lists). */
-                       if (p->state != PROC_DYING)
+                       if (!proc_is_dying(p))
                                add_to_list(p, secondary_mcps);
                        proc_decref(p);                 /* fyi, this may trigger __proc_free */
                        /* need to break: the proc lists may have changed when we unlocked
@@ -517,7 +431,7 @@ static void __run_mcp_ksched(void *arg)
 }
 
 /* Something has changed, and for whatever reason the scheduler should
- * reevaluate things. 
+ * reevaluate things.
  *
  * Don't call this if you are processing a syscall or otherwise care about your
  * kthread variables, cur_proc/owning_proc, etc.
@@ -578,72 +492,6 @@ void avail_res_changed(int res_type, long change)
        printk("[kernel] ksched doesn't track any resources yet!\n");
 }
 
-int get_any_idle_core(void)
-{
-       struct sched_pcore *spc;
-       int ret = -1;
-       spin_lock(&sched_lock);
-       while ((spc = TAILQ_FIRST(&idlecores))) {
-               /* Don't take cores that are provisioned to a process */
-               if (spc->prov_proc)
-                       continue;
-               assert(!spc->alloc_proc);
-               TAILQ_REMOVE(&idlecores, spc, alloc_next);
-               ret = spc2pcoreid(spc);
-               break;
-       }
-       spin_unlock(&sched_lock);
-       return ret;
-}
-
-/* TODO: if we end up using this a lot, track CG-idleness as a property of the
- * SPC instead of doing a linear search. */
-static bool __spc_is_idle(struct sched_pcore *spc)
-{
-       struct sched_pcore *i;
-       TAILQ_FOREACH(i, &idlecores, alloc_next) {
-               if (spc == i)
-                       return TRUE;
-       }
-       return FALSE;
-}
-
-int get_this_idle_core(int coreid)
-{
-       struct sched_pcore *spc = pcoreid2spc(coreid);
-       int ret = -1;
-       assert((0 <= coreid) && (coreid < num_cpus));
-       spin_lock(&sched_lock);
-       if (__spc_is_idle(pcoreid2spc(coreid)) && !spc->prov_proc) {
-               assert(!spc->alloc_proc);
-               TAILQ_REMOVE(&idlecores, spc, alloc_next);
-               ret = coreid;
-       }
-       spin_unlock(&sched_lock);
-       return ret;
-}
-
-/* similar to __sched_put_idle_core, but without the prov tracking */
-void put_idle_core(int coreid)
-{
-       struct sched_pcore *spc = pcoreid2spc(coreid);
-       assert((0 <= coreid) && (coreid < num_cpus));
-       spin_lock(&sched_lock);
-       TAILQ_INSERT_TAIL(&idlecores, spc, alloc_next);
-       spin_unlock(&sched_lock);
-}
-
-/* Normally it'll be the max number of CG cores ever */
-uint32_t max_vcores(struct proc *p)
-{
-/* TODO: (CG/LL) */
-#ifdef CONFIG_DISABLE_SMT
-       return num_cpus >> 1;
-#else
-       return num_cpus - 1;    /* reserving core 0 */
-#endif /* CONFIG_DISABLE_SMT */
-}
-
 /* This deals with a request for more cores.  The amt of new cores needed is
  * passed in.  The ksched lock is held, but we are free to unlock if we want
  * (and we must, if calling out of the ksched to anything high-level).
@@ -656,8 +504,8 @@ uint32_t max_vcores(struct proc *p)
 static void __core_request(struct proc *p, uint32_t amt_needed)
 {
        uint32_t nr_to_grant = 0;
-       uint32_t corelist[num_cpus];
-       struct sched_pcore *spc_i, *temp;
+       uint32_t corelist[num_cores];
+       uint32_t pcoreid;
        struct proc *proc_to_preempt;
        bool success;
        /* we come in holding the ksched lock, and we hold it here to protect
@@ -665,23 +513,27 @@ static void __core_request(struct proc *p, uint32_t amt_needed)
        /* get all available cores from their prov_not_alloc list.  the list might
         * change when we unlock (new cores added to it, or the entire list emptied,
         * but no core allocations will happen (we hold the poke)). */
-       while (!TAILQ_EMPTY(&p->ksched_data.prov_not_alloc_me)) {
-               if (nr_to_grant == amt_needed)
+       while (nr_to_grant != amt_needed) {
+               /* Find the next best core to allocate to p. It may be a core
+                * provisioned to p, and it might not be. */
+               pcoreid = __find_best_core_to_alloc(p);
+               /* If no core is returned, we know that there are no more cores to give
+                * out, so we exit the loop. */
+               if (pcoreid == -1)
                        break;
-               /* picking the next victim (first on the not_alloc list) */
-               spc_i = TAILQ_FIRST(&p->ksched_data.prov_not_alloc_me);
-               /* someone else has this proc's pcore, so we need to try to preempt.
-                * after this block, the core will be tracked dealloc'd and on the idle
-                * list (regardless of whether we had to preempt or not) */
-               if (spc_i->alloc_proc) {
-                       proc_to_preempt = spc_i->alloc_proc;
+               /* If the pcore chosen currently has a proc allocated to it, we know
+                * it must be provisioned to p, but not allocated to it. We need to try
+                * to preempt. After this block, the core will be track_dealloc'd and
+                * on the idle list (regardless of whether we had to preempt or not) */
+               if (get_alloc_proc(pcoreid)) {
+                       proc_to_preempt = get_alloc_proc(pcoreid);
                        /* would break both preemption and maybe the later decref */
                        assert(proc_to_preempt != p);
                        /* need to keep a valid, external ref when we unlock */
                        proc_incref(proc_to_preempt, 1);
                        spin_unlock(&sched_lock);
                        /* sending no warning time for now - just an immediate preempt. */
-                       success = proc_preempt_core(proc_to_preempt, spc2pcoreid(spc_i), 0);
+                       success = proc_preempt_core(proc_to_preempt, pcoreid, 0);
                        /* reaquire locks to protect provisioning and idle lists */
                        spin_lock(&sched_lock);
                        if (success) {
@@ -690,20 +542,17 @@ static void __core_request(struct proc *p, uint32_t amt_needed)
                                 * idle CBs).  the core is not on the idle core list.  (if we
                                 * ever have proc alloc lists, it'll still be on the old proc's
                                 * list). */
-                               assert(spc_i->alloc_proc);
+                               assert(get_alloc_proc(pcoreid));
                                /* regardless of whether or not it is still prov to p, we need
                                 * to note its dealloc.  we are doing some excessive checking of
                                 * p == prov_proc, but using this helper is a lot clearer. */
-                               __prov_track_dealloc(proc_to_preempt, spc2pcoreid(spc_i));
-                               /* here, we rely on the fact that we are the only preemptor.  we
-                                * assume no one else preempted it, so we know it is available*/
-                               TAILQ_INSERT_TAIL(&idlecores, spc_i, alloc_next);
+                               __track_core_dealloc(proc_to_preempt, pcoreid);
                        } else {
                                /* the preempt failed, which should only happen if the pcore was
                                 * unmapped (could be dying, could be yielding, but NOT
                                 * preempted).  whoever unmapped it also triggered (or will soon
-                                * trigger) a track_dealloc and put it on the idle list.  our
-                                * signal for this is spc_i->alloc_proc being 0.  We need to
+                                * trigger) a track_core_dealloc and put it on the idle list.
+                                * Our signal for this is get_alloc_proc() being 0. We need to
                                 * spin and let whoever is trying to free the core grab the
                                 * ksched lock.  We could use an 'ignore_next_idle' flag per
                                 * sched_pcore, but it's not critical anymore.
@@ -715,7 +564,7 @@ static void __core_request(struct proc *p, uint32_t amt_needed)
                                 * allocator, the pcore could have been put on the idle list and
                                 * then quickly removed/allocated. */
                                cmb();
-                               while (spc_i->alloc_proc) {
+                               while (get_alloc_proc(pcoreid)) {
                                        /* this loop should be very rare */
                                        spin_unlock(&sched_lock);
                                        udelay(1);
@@ -724,35 +573,23 @@ static void __core_request(struct proc *p, uint32_t amt_needed)
                        }
                        /* no longer need to keep p_to_pre alive */
                        proc_decref(proc_to_preempt);
-                       /* might not be prov to p anymore (rare race).  spc_i is idle - we
+                       /* might not be prov to p anymore (rare race). pcoreid is idle - we
                         * might get it later, or maybe we'll give it to its rightful proc*/
-                       if (spc_i->prov_proc != p)
+                       if (get_prov_proc(pcoreid) != p)
                                continue;
                }
-               /* at this point, the pcore is idle, regardless of how we got here
+               /* At this point, the pcore is idle, regardless of how we got here
                 * (successful preempt, failed preempt, or it was idle in the first
-                * place.  the core is still provisioned.  lets pull from the idle list
-                * and add it to the pc_arr for p.  here, we rely on the fact that we
-                * are the only allocator (spc_i is still idle, despite unlocking). */
-               TAILQ_REMOVE(&idlecores, spc_i, alloc_next);
-               /* At this point, we have the core, ready to try to give it to the proc.
-                * It is on no alloc lists, and is track_dealloc'd() (regardless of how
-                * we got here).
-                *
-                * We'll give p its cores via a bulk list, which is better for the proc
-                * mgmt code (when going from runnable to running). */
-               corelist[nr_to_grant] = spc2pcoreid(spc_i);
-               nr_to_grant++;
-               __prov_track_alloc(p, spc2pcoreid(spc_i));
-       }
-       /* Try to get cores from the idle list that aren't prov to me (FCFS) */
-       TAILQ_FOREACH_SAFE(spc_i, &idlecores, alloc_next, temp) {
-               if (nr_to_grant == amt_needed)
-                       break;
-               TAILQ_REMOVE(&idlecores, spc_i, alloc_next);
-               corelist[nr_to_grant] = spc2pcoreid(spc_i);
+                * place).  We also know the core is still provisioned to us.  Lets add
+                * it to the corelist for p (so we can give it to p in bulk later), and
+                * track its allocation with p (so our internal data structures stay in
+                * sync). We rely on the fact that we are the only allocator (pcoreid is
+                * still idle, despite (potentially) unlocking during the preempt
+                * attempt above).  It is guaranteed to be track_dealloc'd()
+                * (regardless of how we got here). */
+               corelist[nr_to_grant] = pcoreid;
                nr_to_grant++;
-               __prov_track_alloc(p, spc2pcoreid(spc_i));
+               __track_core_alloc(p, pcoreid);
        }
        /* Now, actually give them out */
        if (nr_to_grant) {
@@ -774,8 +611,7 @@ static void __core_request(struct proc *p, uint32_t amt_needed)
                        /* we failed, put the cores and track their dealloc.  lock is
                         * protecting those structures. */
                        spin_lock(&sched_lock);
-                       __put_idle_cores(p, corelist, nr_to_grant);
-                       __prov_track_dealloc_bulk(p, corelist, nr_to_grant);
+                       __track_core_dealloc_bulk(p, corelist, nr_to_grant);
                } else {
                        /* at some point after giving cores, call proc_run_m() (harmless on
                         * RUNNING_Ms).  You can give small groups of cores, then run them
@@ -790,68 +626,13 @@ static void __core_request(struct proc *p, uint32_t amt_needed)
        /* note the ksched lock is still held */
 }
 
-/* TODO: need more thorough CG/LL management.  For now, core0 is the only LL
- * core.  This won't play well with the ghetto shit in schedule_init() if you do
- * anything like 'DEDICATED_MONITOR' or the ARSC server.  All that needs an
- * overhaul. */
-static bool is_ll_core(uint32_t pcoreid)
-{
-       if (pcoreid == 0)
-               return TRUE;
-       return FALSE;
-}
-
-/* Helper, makes sure the prov/alloc structures track the pcore properly when it
- * is allocated to p.  Might make this take a sched_pcore * in the future. */
-static void __prov_track_alloc(struct proc *p, uint32_t pcoreid)
-{
-       struct sched_pcore *spc;
-       assert(pcoreid < num_cpus);             /* catch bugs */
-       spc = pcoreid2spc(pcoreid);
-       assert(spc->alloc_proc != p);   /* corruption or double-alloc */
-       spc->alloc_proc = p;
-       /* if the pcore is prov to them and now allocated, move lists */
-       if (spc->prov_proc == p) {
-               TAILQ_REMOVE(&p->ksched_data.prov_not_alloc_me, spc, prov_next);
-               TAILQ_INSERT_TAIL(&p->ksched_data.prov_alloc_me, spc, prov_next);
-       }
-}
-
-/* Helper, makes sure the prov/alloc structures track the pcore properly when it
- * is deallocated from p. */
-static void __prov_track_dealloc(struct proc *p, uint32_t pcoreid)
-{
-       struct sched_pcore *spc;
-       assert(pcoreid < num_cpus);             /* catch bugs */
-       spc = pcoreid2spc(pcoreid);
-       spc->alloc_proc = 0;
-       /* if the pcore is prov to them and now deallocated, move lists */
-       if (spc->prov_proc == p) {
-               TAILQ_REMOVE(&p->ksched_data.prov_alloc_me, spc, prov_next);
-               /* this is the victim list, which can be sorted so that we pick the
-                * right victim (sort by alloc_proc reverse priority, etc).  In this
-                * case, the core isn't alloc'd by anyone, so it should be the first
-                * victim. */
-               TAILQ_INSERT_HEAD(&p->ksched_data.prov_not_alloc_me, spc, prov_next);
-       }
-}
-
-/* Bulk interface for __prov_track_dealloc */
-static void __prov_track_dealloc_bulk(struct proc *p, uint32_t *pc_arr,
-                                      uint32_t nr_cores)
-{
-       for (int i = 0; i < nr_cores; i++)
-               __prov_track_dealloc(p, pc_arr[i]);
-}
-
-/* P will get pcore if it needs more cores next time we look at it */
+/* Provision a core to a process. This function wraps the primary logic
+ * implemented in __provision_core, with a lock, error checking, etc. */
 int provision_core(struct proc *p, uint32_t pcoreid)
 {
-       struct sched_pcore *spc;
-       struct sched_pcore_tailq *prov_list;
        /* Make sure we aren't asking for something that doesn't exist (bounds check
         * on the pcore array) */
-       if (!(pcoreid < num_cpus)) {
+       if (!(pcoreid < num_cores)) {
                set_errno(ENXIO);
                return -1;
        }
@@ -860,34 +641,11 @@ int provision_core(struct proc *p, uint32_t pcoreid)
                set_errno(EBUSY);
                return -1;
        }
-       spc = pcoreid2spc(pcoreid);
-       /* Note the sched lock protects the spc tailqs for all procs in this code.
+       /* Note the sched lock protects the tailqs for all procs in this code.
         * If we need a finer grained sched lock, this is one place where we could
         * have a different lock */
        spin_lock(&sched_lock);
-       /* If the core is already prov to someone else, take it away.  (last write
-        * wins, some other layer or new func can handle permissions). */
-       if (spc->prov_proc) {
-               /* the list the spc is on depends on whether it is alloced to the
-                * prov_proc or not */
-               prov_list = (spc->alloc_proc == spc->prov_proc ?
-                            &spc->prov_proc->ksched_data.prov_alloc_me :
-                            &spc->prov_proc->ksched_data.prov_not_alloc_me);
-               TAILQ_REMOVE(prov_list, spc, prov_next);
-       }
-       /* Now prov it to p.  Again, the list it goes on depends on whether it is
-        * alloced to p or not.  Callers can also send in 0 to de-provision. */
-       if (p) {
-               if (spc->alloc_proc == p) {
-                       TAILQ_INSERT_TAIL(&p->ksched_data.prov_alloc_me, spc, prov_next);
-               } else {
-                       /* this is be the victim list, which can be sorted so that we pick
-                        * the right victim (sort by alloc_proc reverse priority, etc). */
-                       TAILQ_INSERT_TAIL(&p->ksched_data.prov_not_alloc_me, spc,
-                                         prov_next);
-               }
-       }
-       spc->prov_proc = p;
+       __provision_core(p, pcoreid);
        spin_unlock(&sched_lock);
        return 0;
 }
@@ -909,16 +667,6 @@ void sched_diag(void)
        return;
 }
 
-void print_idlecoremap(void)
-{
-       struct sched_pcore *spc_i;
-       /* not locking, so we can look at this without deadlocking. */
-       printk("Idle cores (unlocked!):\n");
-       TAILQ_FOREACH(spc_i, &idlecores, alloc_next)
-               printk("Core %d, prov to %d (%p)\n", spc2pcoreid(spc_i),
-                      spc_i->prov_proc ? spc_i->prov_proc->pid : 0, spc_i->prov_proc);
-}
-
 void print_resources(struct proc *p)
 {
        printk("--------------------\n");
@@ -932,83 +680,25 @@ void print_resources(struct proc *p)
 void print_all_resources(void)
 {
        /* Hash helper */
-       void __print_resources(void *item)
+       void __print_resources(void *item, void *opaque)
        {
                print_resources((struct proc*)item);
        }
        spin_lock(&pid_hash_lock);
-       hash_for_each(pid_hash, __print_resources);
+       hash_for_each(pid_hash, __print_resources, NULL);
        spin_unlock(&pid_hash_lock);
 }
 
-void print_prov_map(void)
-{
-       struct sched_pcore *spc_i;
-       /* Doing this unlocked, which is dangerous, but won't deadlock */
-       printk("Which cores are provisioned to which procs:\n------------------\n");
-       for (int i = 0; i < num_cpus; i++) {
-               spc_i = pcoreid2spc(i);
-               printk("Core %02d, prov: %d(%p) alloc: %d(%p)\n", i,
-                      spc_i->prov_proc ? spc_i->prov_proc->pid : 0, spc_i->prov_proc,
-                      spc_i->alloc_proc ? spc_i->alloc_proc->pid : 0,
-                      spc_i->alloc_proc);
-       }
-}
-
-void print_proc_prov(struct proc *p)
+void next_core_to_alloc(uint32_t pcoreid)
 {
-       struct sched_pcore *spc_i;
-       if (!p)
-               return;
-       printk("Prov cores alloced to proc %d (%p)\n----------\n", p->pid, p);
-       TAILQ_FOREACH(spc_i, &p->ksched_data.prov_alloc_me, prov_next)
-               printk("Pcore %d\n", spc2pcoreid(spc_i));
-       printk("Prov cores not alloced to proc %d (%p)\n----------\n", p->pid, p);
-       TAILQ_FOREACH(spc_i, &p->ksched_data.prov_not_alloc_me, prov_next)
-               printk("Pcore %d (alloced to %d (%p))\n", spc2pcoreid(spc_i),
-                      spc_i->alloc_proc ? spc_i->alloc_proc->pid : 0,
-                      spc_i->alloc_proc);
-}
-
-void next_core(uint32_t pcoreid)
-{
-       struct sched_pcore *spc_i;
-       bool match = FALSE;
        spin_lock(&sched_lock);
-       TAILQ_FOREACH(spc_i, &idlecores, alloc_next) {
-               if (spc2pcoreid(spc_i) == pcoreid) {
-                       match = TRUE;
-                       break;
-               }
-       }
-       if (match) {
-               TAILQ_REMOVE(&idlecores, spc_i, alloc_next);
-               TAILQ_INSERT_HEAD(&idlecores, spc_i, alloc_next);
-               printk("Pcore %d will be given out next (from the idles)\n", pcoreid);
-       }
+       __next_core_to_alloc(pcoreid);
        spin_unlock(&sched_lock);
 }
 
-void sort_idles(void)
+void sort_idle_cores(void)
 {
-       struct sched_pcore *spc_i, *spc_j, *temp;
-       struct sched_pcore_tailq sorter = TAILQ_HEAD_INITIALIZER(sorter);
-       bool added;
        spin_lock(&sched_lock);
-       TAILQ_CONCAT(&sorter, &idlecores, alloc_next);
-       TAILQ_FOREACH_SAFE(spc_i, &sorter, alloc_next, temp) {
-               TAILQ_REMOVE(&sorter, spc_i, alloc_next);
-               added = FALSE;
-               /* don't need foreach_safe since we break after we muck with the list */
-               TAILQ_FOREACH(spc_j, &idlecores, alloc_next) {
-                       if (spc_i < spc_j) {
-                               TAILQ_INSERT_BEFORE(spc_j, spc_i, alloc_next);
-                               added = TRUE;
-                               break;
-                       }
-               }
-               if (!added)
-                       TAILQ_INSERT_TAIL(&idlecores, spc_i, alloc_next);
-       }
+       __sort_idle_cores();
        spin_unlock(&sched_lock);
 }