kth: Remove irq_state from sem_.*irqsave's interface
[akaros.git] / kern / src / process.c
index ebd043b..7197e11 100644 (file)
 #include <hashtable.h>
 #include <slab.h>
 #include <sys/queue.h>
-#include <frontend.h>
 #include <monitor.h>
 #include <elf.h>
 #include <arsc_server.h>
-#include <devfs.h>
 #include <kmalloc.h>
+#include <ros/procinfo.h>
+#include <init.h>
+#include <rcu.h>
 
 struct kmem_cache *proc_cache;
 
@@ -127,6 +128,7 @@ int __proc_set_state(struct proc *p, uint32_t state)
         * RGM -> RBS
         * RGS -> D
         * RGM -> D
+        * D   -> DA
         *
         * These ought to be implemented later (allowed, not thought through yet).
         * RBS -> D
@@ -153,9 +155,12 @@ int __proc_set_state(struct proc *p, uint32_t state)
                                panic("Invalid State Transition! PROC_WAITING to %02x", state);
                        break;
                case PROC_DYING:
-                       if (state != PROC_CREATED) // when it is reused (TODO)
+                       if (state != PROC_DYING_ABORT)
                                panic("Invalid State Transition! PROC_DYING to %02x", state);
                        break;
+               case PROC_DYING_ABORT:
+                       panic("Invalid State Transition! PROC_DYING to %02x", state);
+                       break;
                case PROC_RUNNABLE_M:
                        if (!(state & (PROC_RUNNING_M | PROC_DYING)))
                                panic("Invalid State Transition! PROC_RUNNABLE_M to %02x", state);
@@ -211,7 +216,7 @@ struct proc *pid_nth(unsigned int n)
                 * so continue
                 */
 
-               if (kref_get_not_zero(&p->p_kref, 1)){
+               if (kref_get_not_zero(&p->p_kref, 1)) {
                        /* this one counts */
                        if (! n){
                                printd("pid_nth: at end, p %p\n", p);
@@ -220,7 +225,7 @@ struct proc *pid_nth(unsigned int n)
                        kref_put(&p->p_kref);
                        n--;
                }
-               if (!hashtable_iterator_advance(iter)){
+               if (!hashtable_iterator_advance(iter)) {
                        p = NULL;
                        break;
                }
@@ -240,7 +245,9 @@ void proc_init(void)
        /* Catch issues with the vcoremap and TAILQ_ENTRY sizes */
        static_assert(sizeof(TAILQ_ENTRY(vcore)) == sizeof(void*) * 2);
        proc_cache = kmem_cache_create("proc", sizeof(struct proc),
-                    MAX(ARCH_CL_SIZE, __alignof__(struct proc)), 0, 0, 0);
+                                      MAX(ARCH_CL_SIZE,
+                                      __alignof__(struct proc)), 0, NULL, 0,
+                                      0, NULL);
        /* Init PID mask and hash.  pid 0 is reserved. */
        SET_BITMASK_BIT(pid_bmask, 0);
        spinlock_init(&pid_hash_lock);
@@ -252,6 +259,29 @@ void proc_init(void)
        atomic_init(&num_envs, 0);
 }
 
+void proc_set_username(struct proc *p, char *name)
+{
+       set_username(&p->user, name);
+}
+
+/*
+ * Copies username from the parent process. This is the only case where a
+ * reader blocks writing, just to be extra safe during process initialization.
+ *
+ * Note that since this is intended to be called during initialization, the
+ * child's name lock is NOT used for writing. Nothing else should be able to
+ * read or write yet, so this can be a simple memcpy once the parent is locked.
+ */
+void proc_inherit_parent_username(struct proc *child, struct proc *parent)
+{
+       spin_lock(&parent->user.name_lock);
+
+       // copy entire parent buffer for constant runtime
+       memcpy(child->user.name, parent->user.name, sizeof(child->user.name));
+
+       spin_unlock(&parent->user.name_lock);
+}
+
 void proc_set_progname(struct proc *p, char *name)
 {
        if (name == NULL)
@@ -275,9 +305,9 @@ void proc_init_procinfo(struct proc* p)
        p->procinfo->pid = p->pid;
        p->procinfo->ppid = p->ppid;
        p->procinfo->max_vcores = max_vcores(p);
-       p->procinfo->tsc_freq = system_timing.tsc_freq;
-       p->procinfo->timing_overhead = system_timing.timing_overhead;
-       p->procinfo->heap_bottom = 0;
+       p->procinfo->tsc_freq = __proc_global_info.tsc_freq;
+       p->procinfo->timing_overhead = __proc_global_info.tsc_overhead;
+       p->procinfo->program_end = 0;
        /* 0'ing the arguments.  Some higher function will need to set them */
        memset(p->procinfo->res_grant, 0, sizeof(p->procinfo->res_grant));
        /* 0'ing the vcore/pcore map.  Will link the vcores later. */
@@ -300,6 +330,26 @@ void proc_init_procdata(struct proc *p)
        atomic_set(&p->procdata->vcore_preempt_data[0].flags, VC_SCP_NOVCCTX);
 }
 
+static void proc_open_stdfds(struct proc *p)
+{
+       int fd;
+       struct proc *old_current = current;
+
+       /* Due to the way the syscall helpers assume the target process is current,
+        * we need to set current temporarily.  We don't use switch_to, since that
+        * actually loads the process's address space, which might be empty or
+        * incomplete.  These syscalls shouldn't access user memory, especially
+        * considering how we're probably in the boot pgdir. */
+       current = p;
+       fd = sysopenat(AT_FDCWD, "#cons/stdin", O_READ);
+       assert(fd == 0);
+       fd = sysopenat(AT_FDCWD, "#cons/stdout", O_WRITE);
+       assert(fd == 1);
+       fd = sysopenat(AT_FDCWD, "#cons/stderr", O_WRITE);
+       assert(fd == 2);
+       current = old_current;
+}
+
 /* Allocates and initializes a process, with the given parent.  Currently
  * writes the *p into **pp, and returns 0 on success, < 0 for an error.
  * Errors include:
@@ -318,9 +368,6 @@ error_t proc_alloc(struct proc **pp, struct proc *parent, int flags)
        /* only one ref, which we pass back.  the old 'existence' ref is managed by
         * the ksched */
        kref_init(&p->p_kref, __proc_free, 1);
-       // Setup the default map of where to get cache colors from
-       p->cache_colors_map = global_cache_colors_map;
-       p->next_cache_color = 0;
        /* Initialize the address space */
        if ((r = env_setup_vm(p)) < 0) {
                kmem_cache_free(proc_cache, p);
@@ -334,9 +381,11 @@ error_t proc_alloc(struct proc **pp, struct proc *parent, int flags)
                kstrdup(&p->binary_path, parent->binary_path);
        /* Set the basic status variables. */
        spinlock_init(&p->proc_lock);
+       spinlock_init(&p->user.name_lock);
        p->exitcode = 1337;     /* so we can see processes killed by the kernel */
        if (parent) {
                p->ppid = parent->pid;
+               proc_inherit_parent_username(p, parent);
                proc_incref(p, 1);      /* storing a ref in the parent */
                /* using the CV's lock to protect anything related to child waiting */
                cv_lock(&parent->child_wait);
@@ -344,12 +393,13 @@ error_t proc_alloc(struct proc **pp, struct proc *parent, int flags)
                cv_unlock(&parent->child_wait);
        } else {
                p->ppid = 0;
+               strlcpy(p->user.name, eve.name, sizeof(p->user.name));
+               printk("Parentless process assigned username '%s'\n", p->user.name);
        }
        TAILQ_INIT(&p->children);
        cv_init(&p->child_wait);
        p->state = PROC_CREATED; /* shouldn't go through state machine for init */
        p->env_flags = 0;
-       p->heap_top = 0;
        spinlock_init(&p->vmr_lock);
        spinlock_init(&p->pte_lock);
        TAILQ_INIT(&p->vm_regions); /* could init this in the slab */
@@ -372,14 +422,7 @@ error_t proc_alloc(struct proc **pp, struct proc *parent, int flags)
                        SYSEVENTRINGSIZE);
 
        /* Init FS structures TODO: cleanup (might pull this out) */
-       kref_get(&default_ns.kref, 1);
-       p->ns = &default_ns;
-       spinlock_init(&p->fs_env.lock);
-       p->fs_env.umask = parent ? parent->fs_env.umask : S_IWGRP | S_IWOTH;
-       p->fs_env.root = p->ns->root->mnt_root;
-       kref_get(&p->fs_env.root->d_kref, 1);
-       p->fs_env.pwd = parent ? parent->fs_env.pwd : p->fs_env.root;
-       kref_get(&p->fs_env.pwd->d_kref, 1);
+       p->umask = parent ? parent->umask : S_IWGRP | S_IWOTH;
        memset(&p->open_files, 0, sizeof(p->open_files));       /* slightly ghetto */
        spinlock_init(&p->open_files.lock);
        p->open_files.max_files = NR_OPEN_FILES_DEFAULT;
@@ -391,25 +434,19 @@ error_t proc_alloc(struct proc **pp, struct proc *parent, int flags)
                        clone_fdt(&parent->open_files, &p->open_files);
        } else {
                /* no parent, we're created from the kernel */
-               int fd;
-               fd = insert_file(&p->open_files, dev_stdin,  0, TRUE, FALSE);
-               assert(fd == 0);
-               fd = insert_file(&p->open_files, dev_stdout, 1, TRUE, FALSE);
-               assert(fd == 1);
-               fd = insert_file(&p->open_files, dev_stderr, 2, TRUE, FALSE);
-               assert(fd == 2);
+               proc_open_stdfds(p);
        }
        /* Init the ucq hash lock */
        p->ucq_hashlock = (struct hashlock*)&p->ucq_hl_noref;
        hashlock_init_irqsave(p->ucq_hashlock, HASHLOCK_DEFAULT_SZ);
 
        atomic_inc(&num_envs);
-       frontend_proc_init(p);
        plan9setup(p, parent, flags);
        devalarm_init(p);
        TAILQ_INIT(&p->abortable_sleepers);
        spinlock_init_irqsave(&p->abort_list_lock);
        memset(&p->vmm, 0, sizeof(struct vmm));
+       spinlock_init(&p->vmm.lock);
        qlock_init(&p->vmm.qlock);
        printd("[%08x] new process %08x\n", current ? current->pid : 0, p->pid);
        *pp = p;
@@ -430,14 +467,13 @@ void __proc_ready(struct proc *p)
        spin_unlock(&pid_hash_lock);
 }
 
-/* Creates a process from the specified file, argvs, and envps.  Tempted to get
- * rid of proc_alloc's style, but it is so quaint... */
-struct proc *proc_create(struct file *prog, char **argv, char **envp)
+/* Creates a process from the specified file, argvs, and envps. */
+struct proc *proc_create(struct file_or_chan *prog, char **argv, char **envp)
 {
        struct proc *p;
        error_t r;
        if ((r = proc_alloc(&p, current, 0 /* flags */)) < 0)
-               panic("proc_create: %e", r);    /* one of 3 quaint usages of %e */
+               panic("proc_create: %d", r);
        int argc = 0, envc = 0;
        if(argv) while(argv[argc]) argc++;
        if(envp) while(envp[envc]) envc++;
@@ -467,23 +503,18 @@ static void __proc_free(struct kref *kref)
        assert(kref_refcnt(&p->p_kref) == 0);
        assert(TAILQ_EMPTY(&p->alarmset.list));
 
+       if (p->strace) {
+               kref_put(&p->strace->procs);
+               kref_put(&p->strace->users);
+       }
        __vmm_struct_cleanup(p);
        p->progname[0] = 0;
        free_path(p, p->binary_path);
        cclose(p->dot);
        cclose(p->slash);
        p->dot = p->slash = 0; /* catch bugs */
-       kref_put(&p->fs_env.root->d_kref);
-       kref_put(&p->fs_env.pwd->d_kref);
        /* now we'll finally decref files for the file-backed vmrs */
        unmap_and_destroy_vmrs(p);
-       frontend_proc_free(p);  /* TODO: please remove me one day */
-       /* Free any colors allocated to this process */
-       if (p->cache_colors_map != global_cache_colors_map) {
-               for(int i = 0; i < llc_cache->num_colors; i++)
-                       cache_color_free(llc_cache, p->cache_colors_map);
-               cache_colors_map_free(p->cache_colors_map);
-       }
        /* Remove us from the pid_hash and give our PID back (in that order). */
        spin_lock(&pid_hash_lock);
        hash_ret = hashtable_remove(pid_hash, (void*)(long)p->pid);
@@ -494,13 +525,14 @@ static void __proc_free(struct kref *kref)
        else
                printd("[kernel] pid %d not in the PID hash in %s\n", p->pid,
                       __FUNCTION__);
-       /* all memory below UMAPTOP should have been freed via the VMRs.  the stuff
-        * above is the global page and procinfo/procdata */
-       env_user_mem_free(p, (void*)UMAPTOP, UVPT - UMAPTOP); /* 3rd arg = len... */
+       /* All memory below UMAPTOP should have been freed via the VMRs.  The stuff
+        * above is the global info/page and procinfo/procdata.  We free procinfo
+        * and procdata, but not the global memory - that's system wide.  We could
+        * clear the PTEs of the upper stuff (UMAPTOP to UVPT), but we shouldn't
+        * need to. */
        env_user_mem_walk(p, 0, UMAPTOP, __cb_assert_no_pg, 0);
-       /* These need to be freed again, since they were allocated with a refcnt. */
-       free_cont_pages(p->procinfo, LOG2_UP(PROCINFO_NUM_PAGES));
-       free_cont_pages(p->procdata, LOG2_UP(PROCDATA_NUM_PAGES));
+       kpages_free(p->procinfo, PROCINFO_NUM_PAGES * PGSIZE);
+       kpages_free(p->procdata, PROCDATA_NUM_PAGES * PGSIZE);
 
        env_pagetable_free(p);
        arch_pgdir_clear(&p->env_pgdir);
@@ -545,6 +577,8 @@ static void __set_proc_current(struct proc *p)
        /* We use the pcpui to access 'current' to cut down on the core_id() calls,
         * though who know how expensive/painful they are. */
        struct per_cpu_info *pcpui = &per_cpu_info[core_id()];
+       struct proc *old_proc;
+
        /* If the process wasn't here, then we need to load its address space. */
        if (p != pcpui->cur_proc) {
                proc_incref(p, 1);
@@ -553,9 +587,10 @@ static void __set_proc_current(struct proc *p)
                 * previous lcr3 unloaded the previous proc's context.  This should
                 * rarely happen, since we usually proactively leave process context,
                 * but this is the fallback. */
-               if (pcpui->cur_proc)
-                       proc_decref(pcpui->cur_proc);
+               old_proc = pcpui->cur_proc;
                pcpui->cur_proc = p;
+               if (old_proc)
+                       proc_decref(old_proc);
        }
 }
 
@@ -568,7 +603,7 @@ static bool scp_is_vcctx_ready(struct preempt_data *vcpd)
 }
 
 /* Dispatches a _S process to run on the current core.  This should never be
- * called to "restart" a core.   
+ * called to "restart" a core.
  *
  * This will always return, regardless of whether or not the calling core is
  * being given to a process. (it used to pop the tf directly, before we had
@@ -584,6 +619,7 @@ void proc_run_s(struct proc *p)
        spin_lock(&p->proc_lock);
        switch (p->state) {
                case (PROC_DYING):
+               case (PROC_DYING_ABORT):
                        spin_unlock(&p->proc_lock);
                        printk("[kernel] _S %d not starting due to async death\n", p->pid);
                        return;
@@ -686,6 +722,7 @@ void __proc_run_m(struct proc *p)
        switch (p->state) {
                case (PROC_WAITING):
                case (PROC_DYING):
+               case (PROC_DYING_ABORT):
                        warn("ksched tried to run proc %d in state %s\n", p->pid,
                             procstate2str(p->state));
                        return;
@@ -735,58 +772,54 @@ void __proc_run_m(struct proc *p)
  * bypass the routine_kmsg check.  Interrupts should be off when you call this.
  *
  * A note on refcnting: this function will not return, and your proc reference
- * will end up stored in current.  This will make no changes to p's refcnt, so
- * do your accounting such that there is only the +1 for current.  This means if
- * it is already in current (like in the trap return path), don't up it.  If
- * it's already in current and you have another reference (like pid2proc or from
- * an IPI), then down it (which is what happens in __startcore()).  If it's not
- * in current and you have one reference, like proc_run(non_current_p), then
- * also do nothing.  The refcnt for your *p will count for the reference stored
- * in current. */
+ * will be ignored (not decreffed).  It may be incref'd, if cur_proc was not
+ * set.  Pass in an already-accounted-for ref, such as owning_proc. */
 void __proc_startcore(struct proc *p, struct user_context *ctx)
 {
        struct per_cpu_info *pcpui = &per_cpu_info[core_id()];
        assert(!irq_is_enabled());
        /* Should never have ktask still set.  If we do, future syscalls could try
         * to block later and lose track of our address space. */
-       assert(!pcpui->cur_kthread->is_ktask);
+       assert(!is_ktask(pcpui->cur_kthread));
        __set_proc_current(p);
-       /* Clear the current_ctx, since it is no longer used */
-       current_ctx = 0;        /* TODO: might not need this... */
        __set_cpu_state(pcpui, CPU_STATE_USER);
        proc_pop_ctx(ctx);
 }
 
 /* Restarts/runs the current_ctx, which must be for the current process, on the
- * core this code executes on.  Calls an internal function to do the work.
+ * core this code executes on.
  *
- * In case there are pending routine messages, like __death, __preempt, or
- * __notify, we need to run them.  Alternatively, if there are any, we could
- * self_ipi, and run the messages immediately after popping back to userspace,
- * but that would have crappy overhead. */
+ * For now, we just smp_idle.  We used to do something similar, but customized
+ * for expecting to return to the process.  But it was a source of bugs.  If we
+ * want to optimize for the case where we know we had a process current, then we
+ * can do so here.
+ *
+ * Note that PRKM currently calls smp_idle() if it ever has a message, so the
+ * value of optimizing may depend on the semantics of PRKM. */
 void proc_restartcore(void)
 {
-       struct per_cpu_info *pcpui = &per_cpu_info[core_id()];
-       assert(!pcpui->cur_kthread->sysc);
-       /* TODO: can probably remove this enable_irq.  it was an optimization for
-        * RKMs */
-       /* Try and get any interrupts before we pop back to userspace.  If we didn't
-        * do this, we'd just get them in userspace, but this might save us some
-        * effort/overhead. */
-       enable_irq();
-       /* Need ints disabled when we return from PRKM (race on missing
-        * messages/IPIs) */
-       disable_irq();
-       process_routine_kmsg();
-       /* If there is no owning process, just idle, since we don't know what to do.
-        * This could be because the process had been restarted a long time ago and
-        * has since left the core, or due to a KMSG like __preempt or __death. */
-       if (!pcpui->owning_proc) {
-               abandon_core();
-               smp_idle();
+       smp_idle();
+}
+
+/* Helper for proc_destroy.  Disowns any children. */
+static void proc_disown_children(struct proc *parent)
+{
+       struct proc *child_i, *temp;
+       struct proc_list todo = TAILQ_HEAD_INITIALIZER(todo);
+       int ret;
+
+       cv_lock(&parent->child_wait);
+       TAILQ_FOREACH_SAFE(child_i, &parent->children, sibling_link, temp) {
+               ret = __proc_disown_child(parent, child_i);
+               /* should never fail, lock should cover the race.  invariant: any child
+                * on the list should have us as a parent */
+               assert(!ret);
+               TAILQ_INSERT_TAIL(&todo, child_i, sibling_link);
        }
-       assert(pcpui->cur_ctx);
-       __proc_startcore(pcpui->owning_proc, pcpui->cur_ctx);
+       cv_unlock(&parent->child_wait);
+
+       TAILQ_FOREACH_SAFE(child_i, &todo, sibling_link, temp)
+               proc_decref(child_i);
 }
 
 /* Destroys the process.  It will destroy the process and return any cores
@@ -815,15 +848,13 @@ void proc_destroy(struct proc *p)
        uint32_t nr_cores_revoked = 0;
        struct kthread *sleeper;
        struct proc *child_i, *temp;
-       /* Can't spin on the proc lock with irq disabled.  This is a problem for all
-        * places where we grab the lock, but it is particularly bad for destroy,
-        * since we tend to call this from trap and irq handlers */
-       assert(irq_is_enabled());
+
        spin_lock(&p->proc_lock);
        /* storage for pc_arr is alloced at decl, which is after grabbing the lock*/
        uint32_t pc_arr[p->procinfo->num_vcores];
        switch (p->state) {
                case PROC_DYING: /* someone else killed this already. */
+               case (PROC_DYING_ABORT):
                        spin_unlock(&p->proc_lock);
                        return;
                case PROC_CREATED:
@@ -841,11 +872,11 @@ void proc_destroy(struct proc *p)
                        // here's how to do it manually
                        if (current == p) {
                                lcr3(boot_cr3);
-                               proc_decref(p);         /* this decref is for the cr3 */
                                current = NULL;
+                               proc_decref(p);         /* this decref is for the cr3 */
                        }
                        #endif
-                       send_kernel_message(get_pcoreid(p, 0), __death, 0, 0, 0,
+                       send_kernel_message(get_pcoreid(p, 0), __death, (long)p, 0, 0,
                                            KMSG_ROUTINE);
                        __seq_start_write(&p->procinfo->coremap_seqctr);
                        __unmap_vcore(p, 0);
@@ -864,33 +895,26 @@ void proc_destroy(struct proc *p)
         * interrupts should be on when you call proc_destroy locally, but currently
         * aren't for all things (like traphandlers). */
        __proc_set_state(p, PROC_DYING);
-       /* Disown any children.  If we want to have init inherit or something,
-        * change __disown to set the ppid accordingly and concat this with init's
-        * list (instead of emptying it like disown does).  Careful of lock ordering
-        * between procs (need to lock to protect lists) */
-       TAILQ_FOREACH_SAFE(child_i, &p->children, sibling_link, temp) {
-               int ret = __proc_disown_child(p, child_i);
-               /* should never fail, lock should cover the race.  invariant: any child
-                * on the list should have us as a parent */
-               assert(!ret);
-       }
        spin_unlock(&p->proc_lock);
+       proc_disown_children(p);
        /* Wake any of our kthreads waiting on children, so they can abort */
        cv_broadcast(&p->child_wait);
-       /* Abort any abortable syscalls.  This won't catch every sleeper, but future
-        * abortable sleepers are already prevented via the DYING state.  (signalled
-        * DYING, no new sleepers will block, and now we wake all old sleepers). */
-       abort_all_sysc(p);
        /* we need to close files here, and not in free, since we could have a
         * refcnt indirectly related to one of our files.  specifically, if we have
         * a parent sleeping on our pipe, that parent won't wake up to decref until
         * the pipe closes.  And if the parent doesnt decref, we don't free.
-        * alternatively, we could send a SIGCHILD to the parent, but that would
-        * require parent's to never ignore that signal (or risk never reaping).
+        * Even if we send a SIGCHLD to the parent, that would require that the
+        * parent to never ignores that signal (or we risk never reaping).
         *
         * Also note that any mmap'd files will still be mmapped.  You can close the
         * file after mmapping, with no effect. */
        close_fdt(&p->open_files, FALSE);
+       /* Abort any abortable syscalls.  This won't catch every sleeper, but future
+        * abortable sleepers are already prevented via the DYING_ABORT state.
+        * (signalled DYING_ABORT, no new sleepers will block, and now we wake all
+        * old sleepers). */
+       __proc_set_state(p, PROC_DYING_ABORT);
+       abort_all_sysc(p);
        /* Tell the ksched about our death, and which cores we freed up */
        __sched_proc_destroy(p, pc_arr, nr_cores_revoked);
        /* Tell our parent about our state change (to DYING) */
@@ -898,14 +922,15 @@ void proc_destroy(struct proc *p)
 }
 
 /* Can use this to signal anything that might cause a parent to wait on the
- * child, such as termination, or (in the future) signals.  Change the state or
- * whatever before calling. */
+ * child, such as termination, or signals.  Change the state or whatever before
+ * calling. */
 void proc_signal_parent(struct proc *child)
 {
        struct kthread *sleeper;
        struct proc *parent = pid2proc(child->ppid);
        if (!parent)
                return;
+       send_posix_signal(parent, SIGCHLD);
        /* there could be multiple kthreads sleeping for various reasons.  even an
         * SCP could have multiple async syscalls. */
        cv_broadcast(&parent->child_wait);
@@ -915,7 +940,9 @@ void proc_signal_parent(struct proc *child)
 
 /* Called when a parent is done with its child, and no longer wants to track the
  * child, nor to allow the child to track it.  Call with a lock (cv) held.
- * Returns 0 if we disowned, -1 on failure. */
+ * Returns 0 if we disowned, -1 on failure.
+ *
+ * If we disowned, (ret == 0), the caller must decref the child. */
 int __proc_disown_child(struct proc *parent, struct proc *child)
 {
        /* Bail out if the child has already been reaped */
@@ -927,7 +954,6 @@ int __proc_disown_child(struct proc *parent, struct proc *child)
        /* After this, the child won't be able to get more refs to us, but it may
         * still have some references in running code. */
        child->ppid = 0;
-       proc_decref(child);     /* ref that was keeping the child alive on the list */
        return 0;
 }
 
@@ -983,6 +1009,7 @@ int proc_change_to_m(struct proc *p)
                        warn("Not supporting RUNNABLE_S -> RUNNABLE_M yet.\n");
                        goto error_out;
                case (PROC_DYING):
+               case (PROC_DYING_ABORT):
                        warn("Dying, core request coming from %d\n", core_id());
                        goto error_out;
                default:
@@ -1142,7 +1169,7 @@ void proc_yield(struct proc *p, bool being_nice)
                                 * WAITING.  one (or both) of us will see and make sure the proc
                                 * wakes up.  */
                                __proc_set_state(p, PROC_WAITING);
-                               wrmb(); /* don't let the state write pass the notif read */ 
+                               wrmb(); /* don't let the state write pass the notif read */
                                if (vcpd->notif_pending) {
                                        __proc_set_state(p, PROC_RUNNING_S);
                                        /* they can't handle events, just need to prevent a yield.
@@ -1169,6 +1196,7 @@ void proc_yield(struct proc *p, bool being_nice)
                case (PROC_RUNNING_M):
                        break;                          /* will handle this stuff below */
                case (PROC_DYING):              /* incoming __death */
+               case (PROC_DYING_ABORT):
                case (PROC_RUNNABLE_M): /* incoming (bulk) preempt/myield TODO:(BULK) */
                        goto out_failed;
                default:
@@ -1280,6 +1308,15 @@ out_yield_core:                          /* successfully yielded the core */
 void proc_notify(struct proc *p, uint32_t vcoreid)
 {
        struct preempt_data *vcpd = &p->procdata->vcore_preempt_data[vcoreid];
+
+       /* If you're thinking about checking notif_pending and then returning if it
+        * is already set, note that some callers (e.g. the event system) set
+        * notif_pending when they deliver a message, regardless of whether there is
+        * an IPI or not.  Those callers assume that we don't care about
+        * notif_pending, only notif_disabled.  So don't change this without
+        * changing them (probably can't without a lot of thought - that
+        * notif_pending is about missing messages.  It might be possible to say "no
+        * IPI, but don't let me miss messages that were delivered." */
        vcpd->notif_pending = TRUE;
        wrmb(); /* must write notif_pending before reading notif_disabled */
        if (!vcpd->notif_disabled) {
@@ -1327,6 +1364,7 @@ void proc_wakeup(struct proc *p)
                        case (PROC_RUNNABLE_S):
                        case (PROC_RUNNING_S):
                        case (PROC_DYING):
+                       case (PROC_DYING_ABORT):
                                spin_unlock(&p->proc_lock);
                                return;
                        case (PROC_RUNNABLE_M):
@@ -1584,7 +1622,7 @@ static void __proc_give_cores_running(struct proc *p, uint32_t *pc_arr,
        for (int i = 0; i < num; i++) {
                assert(__proc_give_a_pcore(p, pc_arr[i], &p->inactive_vcs, &vc_i));
                send_kernel_message(pc_arr[i], __startcore, (long)p,
-                                   (long)vcore2vcoreid(p, vc_i), 
+                                   (long)vcore2vcoreid(p, vc_i),
                                    (long)vc_i->nr_preempts_sent, KMSG_ROUTINE);
        }
        __seq_end_write(&p->procinfo->coremap_seqctr);
@@ -1616,6 +1654,7 @@ int __proc_give_cores(struct proc *p, uint32_t *pc_arr, uint32_t num)
                        warn("Don't give cores to a process in a *_S state!\n");
                        return -1;
                case (PROC_DYING):
+               case (PROC_DYING_ABORT):
                case (PROC_WAITING):
                        /* can't accept, just fail */
                        return -1;
@@ -1647,7 +1686,7 @@ static void __proc_revoke_core(struct proc *p, uint32_t vcoreid, bool preempt)
                atomic_or(&vcpd->flags, VC_K_LOCK);
                send_kernel_message(pcoreid, __preempt, (long)p, 0, 0, KMSG_ROUTINE);
        } else {
-               send_kernel_message(pcoreid, __death, 0, 0, 0, KMSG_ROUTINE);
+               send_kernel_message(pcoreid, __death, (long)p, 0, 0, KMSG_ROUTINE);
        }
 }
 
@@ -1764,16 +1803,21 @@ void __unmap_vcore(struct proc *p, uint32_t vcoreid)
  * Note this leaves no trace of what was running. This "leaves the process's
  * context.
  *
- * This does not clear the owning proc.  Use the other helper for that. */
-void abandon_core(void)
+ * This does not clear the owning proc.  Use the other helper for that.
+ *
+ * Returns whether or not there was a process present. */
+bool abandon_core(void)
 {
        struct per_cpu_info *pcpui = &per_cpu_info[core_id()];
        /* Syscalls that don't return will ultimately call abadon_core(), so we need
         * to make sure we don't think we are still working on a syscall. */
        pcpui->cur_kthread->sysc = 0;
        pcpui->cur_kthread->errbuf = 0; /* just in case */
-       if (pcpui->cur_proc)
+       if (pcpui->cur_proc) {
                __abandon_core();
+               return true;
+       }
+       return false;
 }
 
 /* Helper to clear the core's owning processor and manage refcnting.  Pass in
@@ -1782,6 +1826,8 @@ void clear_owning_proc(uint32_t coreid)
 {
        struct per_cpu_info *pcpui = &per_cpu_info[coreid];
        struct proc *p = pcpui->owning_proc;
+
+       __clear_owning_proc(coreid);
        pcpui->owning_proc = 0;
        pcpui->owning_vcoreid = 0xdeadbeef;
        pcpui->cur_ctx = 0;                     /* catch bugs for now (may go away) */
@@ -1791,14 +1837,16 @@ void clear_owning_proc(uint32_t coreid)
 
 /* Switches to the address space/context of new_p, doing nothing if we are
  * already in new_p.  This won't add extra refcnts or anything, and needs to be
- * paired with switch_back() at the end of whatever function you are in.  Don't
- * migrate cores in the middle of a pair.  Specifically, the uncounted refs are
- * one for the old_proc, which is passed back to the caller, and new_p is
- * getting placed in cur_proc. */
-struct proc *switch_to(struct proc *new_p)
+ * paired with switch_back() at the end of whatever function you are in.
+ * Specifically, the uncounted refs are one for the old_proc, which is passed
+ * back to the caller, and new_p is getting placed in cur_proc. */
+uintptr_t switch_to(struct proc *new_p)
 {
        struct per_cpu_info *pcpui = &per_cpu_info[core_id()];
+       struct kthread *kth = pcpui->cur_kthread;
        struct proc *old_proc;
+       uintptr_t ret;
+
        old_proc = pcpui->cur_proc;                                     /* uncounted ref */
        /* If we aren't the proc already, then switch to it */
        if (old_proc != new_p) {
@@ -1808,14 +1856,33 @@ struct proc *switch_to(struct proc *new_p)
                else
                        lcr3(boot_cr3);
        }
-       return old_proc;
+       ret = (uintptr_t)old_proc;
+       if (is_ktask(kth)) {
+               if (!(kth->flags & KTH_SAVE_ADDR_SPACE)) {
+                       kth->flags |= KTH_SAVE_ADDR_SPACE;
+                       /* proc pointers are aligned; we can use the lower bit as a signal
+                        * to turn off SAVE_ADDR_SPACE. */
+                       ret |= 0x1;
+               }
+       }
+       return ret;
 }
 
-/* This switches back to old_proc from new_p.  Pair it with switch_to(), and
- * pass in its return value for old_proc. */
-void switch_back(struct proc *new_p, struct proc *old_proc)
+/* This switches back from new_p to the original process.  Pair it with
+ * switch_to(), and pass in its return value for old_ret. */
+void switch_back(struct proc *new_p, uintptr_t old_ret)
 {
        struct per_cpu_info *pcpui = &per_cpu_info[core_id()];
+       struct kthread *kth = pcpui->cur_kthread;
+       struct proc *old_proc;
+
+       if (is_ktask(kth)) {
+               if (old_ret & 0x1) {
+                       kth->flags &= ~KTH_SAVE_ADDR_SPACE;
+                       old_ret &= ~0x1;
+               }
+       }
+       old_proc = (struct proc*)old_ret;
        if (old_proc != new_p) {
                pcpui->cur_proc = old_proc;
                if (old_proc)
@@ -1848,7 +1915,12 @@ void proc_tlbshootdown(struct proc *p, uintptr_t start, uintptr_t end)
                        tlbflush();
                        break;
                case (PROC_RUNNING_M):
-                       /* TODO: (TLB) sanity checks and rounding on the ranges */
+                       /* TODO: (TLB) sanity checks and rounding on the ranges.
+                        *
+                        * We need to make sure that once a core that was online has been
+                        * removed from the online list, then it must receive a TLB flush
+                        * (abandon_core()) before running the process again.  Either that,
+                        * or make other decisions about who to TLB-shootdown. */
                        TAILQ_FOREACH(vc_i, &p->online_vcs, list) {
                                send_kernel_message(vc_i->pcoreid, __tlbshootdown, start, end,
                                                    0, KMSG_IMMEDIATE);
@@ -1967,6 +2039,7 @@ int proc_change_to_vcore(struct proc *p, uint32_t new_vcoreid,
                        break;                          /* the only case we can proceed */
                case (PROC_RUNNING_S):  /* user bug, just return */
                case (PROC_DYING):              /* incoming __death */
+               case (PROC_DYING_ABORT):
                case (PROC_RUNNABLE_M): /* incoming (bulk) preempt/myield TODO:(BULK) */
                        goto out_locked;
                default:
@@ -2209,18 +2282,24 @@ void __death(uint32_t srcid, long a0, long a1, long a2)
 {
        uint32_t vcoreid, coreid = core_id();
        struct per_cpu_info *pcpui = &per_cpu_info[coreid];
-       struct proc *p = pcpui->owning_proc;
-       if (p) {
-               vcoreid = pcpui->owning_vcoreid;
-               printd("[kernel] death on physical core %d for process %d's vcore %d\n",
-                      coreid, p->pid, vcoreid);
-               vcore_account_offline(p, vcoreid);      /* in case anyone is counting */
-               /* We won't restart the process later.  current gets cleared later when
-                * we notice there is no owning_proc and we have nothing to do
-                * (smp_idle, restartcore, etc). */
-               arch_finalize_ctx(pcpui->cur_ctx);
-               clear_owning_proc(coreid);
+       struct proc *p = (struct proc*)a0;
+
+       assert(p);
+       if (p != pcpui->owning_proc) {
+               /* Older versions of Akaros thought it was OK to have a __death hit a
+                * core that no longer had a process.  I think it's a bug now. */
+               panic("__death arrived for a process (%p) that was not owning (%p)!",
+                     p, pcpui->owning_proc);
        }
+       vcoreid = pcpui->owning_vcoreid;
+       printd("[kernel] death on physical core %d for process %d's vcore %d\n",
+              coreid, p->pid, vcoreid);
+       vcore_account_offline(p, vcoreid);      /* in case anyone is counting */
+       /* We won't restart the process later.  current gets cleared later when
+        * we notice there is no owning_proc and we have nothing to do
+        * (smp_idle, restartcore, etc). */
+       arch_finalize_ctx(pcpui->cur_ctx);
+       clear_owning_proc(coreid);
 }
 
 /* Kernel message handler, usually sent IMMEDIATE, to shoot down virtual
@@ -2278,9 +2357,9 @@ void proc_get_set(struct process_set *pset)
                pset->size = atomic_read(&num_envs) + num_extra_alloc;
                pset->num_processes = 0;
                pset->procs = (struct proc **)
-                       kzmalloc(pset->size * sizeof(struct proc *), KMALLOC_WAIT);
+                       kzmalloc(pset->size * sizeof(struct proc *), MEM_WAIT);
                if (!pset->procs)
-                       error(-ENOMEM, NULL);
+                       error(-ENOMEM, ERROR_FIXME);
 
                spin_lock(&pid_hash_lock);
                hash_for_each(pid_hash, enum_proc, pset);
@@ -2296,16 +2375,20 @@ void proc_free_set(struct process_set *pset)
        kfree(pset->procs);
 }
 
-void print_proc_info(pid_t pid)
+void print_proc_info(pid_t pid, int verbosity)
 {
        int j = 0;
        uint64_t total_time = 0;
        struct proc *child, *p = pid2proc(pid);
        struct vcore *vc_i;
+       struct preempt_data *vcpd;
+
        if (!p) {
                printk("Bad PID.\n");
                return;
        }
+       vcpd = &p->procdata->vcore_preempt_data[0];
+       print_lock();
        spinlock_debug(&p->proc_lock);
        //spin_lock(&p->proc_lock); // No locking!!
        printk("struct proc: %p\n", p);
@@ -2314,6 +2397,12 @@ void print_proc_info(pid_t pid)
        printk("PPID: %d\n", p->ppid);
        printk("State: %s (%p)\n", procstate2str(p->state), p->state);
        printk("\tIs %san MCP\n", p->procinfo->is_mcp ? "" : "not ");
+       if (!scp_is_vcctx_ready(vcpd))
+               printk("\tIs NOT vcctx ready\n");
+       if (verbosity > 0 && !p->procinfo->is_mcp) {
+               printk("Last saved SCP context:");
+               backtrace_user_ctx(p, &p->scp_ctx);
+       }
        printk("Refcnt: %d\n", atomic_read(&p->p_kref.refcount) - 1);
        printk("Flags: 0x%08x\n", p->env_flags);
        printk("CR3(phys): %p\n", p->env_cr3);
@@ -2328,16 +2417,20 @@ void print_proc_info(pid_t pid)
        printk("Inactive / Yielded:\n");
        TAILQ_FOREACH(vc_i, &p->inactive_vcs, list)
                printk("\tVcore %d\n", vcore2vcoreid(p, vc_i));
-       printk("Nsec Online, up to the last offlining:\n------------------------");
-       for (int i = 0; i < p->procinfo->max_vcores; i++) {
-               uint64_t vc_time = tsc2nsec(vcore_account_gettotal(p, i));
-               if (i % 4 == 0)
-                       printk("\n");
-               printk("  VC %3d: %14llu", i, vc_time);
-               total_time += vc_time;
+       if (verbosity > 0) {
+               printk("Nsec Online, up to the last offlining:\n");
+               printk("------------------------");
+               for (int i = 0; i < p->procinfo->max_vcores; i++) {
+                       uint64_t vc_time = tsc2nsec(vcore_account_gettotal(p, i));
+
+                       if (i % 4 == 0)
+                               printk("\n");
+                       printk("  VC %3d: %14llu", i, vc_time);
+                       total_time += vc_time;
+               }
+               printk("\n");
+               printk("Total CPU-NSEC: %llu\n", total_time);
        }
-       printk("\n");
-       printk("Total CPU-NSEC: %llu\n", total_time);
        printk("Resources:\n------------------------\n");
        for (int i = 0; i < MAX_NUM_RESOURCES; i++)
                printk("\tRes type: %02d, amt wanted: %08d, amt granted: %08d\n", i,
@@ -2347,6 +2440,7 @@ void print_proc_info(pid_t pid)
        if (spin_locked(&files->lock)) {
                spinlock_debug(&files->lock);
                printk("FILE LOCK HELD, ABORTING\n");
+               print_unlock();
                proc_decref(p);
                return;
        }
@@ -2354,19 +2448,15 @@ void print_proc_info(pid_t pid)
        for (int i = 0; i < files->max_files; i++) {
                if (GET_BITMASK_BIT(files->open_fds->fds_bits, i)) {
                        printk("\tFD: %02d, ", i);
-                       if (files->fd[i].fd_file) {
-                               printk("File: %p, File name: %s\n", files->fd[i].fd_file,
-                                      file_name(files->fd[i].fd_file));
-                       } else {
-                               assert(files->fd[i].fd_chan);
-                               print_chaninfo(files->fd[i].fd_chan);
-                       }
+                       assert(files->fd[i].fd_chan);
+                       print_chaninfo(files->fd[i].fd_chan);
                }
        }
        spin_unlock(&files->lock);
        printk("Children: (PID (struct proc *))\n");
        TAILQ_FOREACH(child, &p->children, sibling_link)
                printk("\t%d (%p)\n", child->pid, child);
+       print_unlock();
        /* no locking / unlocking or refcnting */
        // spin_unlock(&p->proc_lock);
        proc_decref(p);
@@ -2401,7 +2491,6 @@ void check_my_owner(void)
                spin_unlock(&p->proc_lock);
        }
        assert(!irq_is_enabled());
-       extern int booting;
        if (!booting && !pcpui->owning_proc) {
                spin_lock(&pid_hash_lock);
                hash_for_each(pid_hash, shazbot, NULL);