CPU state tracking
[akaros.git] / kern / src / process.c
index cda5b5d..ee04506 100644 (file)
 #include <elf.h>
 #include <arsc_server.h>
 #include <devfs.h>
+#include <kmalloc.h>
 
 struct kmem_cache *proc_cache;
 
 /* Other helpers, implemented later. */
-static void __proc_startcore(struct proc *p, struct user_context *ctx);
 static bool is_mapped_vcore(struct proc *p, uint32_t pcoreid);
 static uint32_t get_vcoreid(struct proc *p, uint32_t pcoreid);
 static uint32_t try_get_pcoreid(struct proc *p, uint32_t vcoreid);
@@ -82,6 +82,32 @@ static void put_free_pid(pid_t pid)
        spin_unlock(&pid_bmask_lock);
 }
 
+/* 'resume' is the time int ticks of the most recent onlining.  'total' is the
+ * amount of time in ticks consumed up to and including the current offlining.
+ *
+ * We could move these to the map and unmap of vcores, though not every place
+ * uses that (SCPs, in particular).  However, maps/unmaps happen remotely;
+ * something to consider.  If we do it remotely, we can batch them up and do one
+ * rdtsc() for all of them.  For now, I want to do them on the core, around when
+ * we do the context change.  It'll also parallelize the accounting a bit. */
+void vcore_account_online(struct proc *p, uint32_t vcoreid)
+{
+       struct vcore *vc = &p->procinfo->vcoremap[vcoreid];
+       vc->resume_ticks = read_tsc();
+}
+
+void vcore_account_offline(struct proc *p, uint32_t vcoreid)
+{
+       struct vcore *vc = &p->procinfo->vcoremap[vcoreid];
+       vc->total_ticks += read_tsc() - vc->resume_ticks;
+}
+
+uint64_t vcore_account_gettotal(struct proc *p, uint32_t vcoreid)
+{
+       struct vcore *vc = &p->procinfo->vcoremap[vcoreid];
+       return vc->total_ticks;
+}
+
 /* While this could be done with just an assignment, this gives us the
  * opportunity to check for bad transitions.  Might compile these out later, so
  * we shouldn't rely on them for sanity checking from userspace.  */
@@ -166,6 +192,50 @@ struct proc *pid2proc(pid_t pid)
        return p;
 }
 
+/* Used by devproc for successive reads of the proc table.
+ * Returns a pointer to the nth proc, or 0 if there is none.
+ * This uses get_not_zero, since it is possible the refcnt is 0, which means the
+ * process is dying and we should not have the ref (and thus return 0).  We need
+ * to lock to protect us from getting p, (someone else removes and frees p),
+ * then get_not_zero() on p.
+ * Don't push the locking into the hashtable without dealing with this. */
+struct proc *pid_nth(unsigned int n)
+{
+       struct proc *p;
+       spin_lock(&pid_hash_lock);
+       if (!hashtable_count(pid_hash)) {
+               spin_unlock(&pid_hash_lock);
+               return NULL;
+       }
+       struct hashtable_itr *iter = hashtable_iterator(pid_hash);
+       p = hashtable_iterator_value(iter);
+
+       while (p) {
+               /* if this process is not valid, it doesn't count,
+                * so continue
+                */
+
+               if (kref_get_not_zero(&p->p_kref, 1)){
+                       /* this one counts */
+                       if (! n){
+                               printd("pid_nth: at end, p %p\n", p);
+                               break;
+                       }
+                       kref_put(&p->p_kref);
+                       n--;
+               }
+               if (!hashtable_iterator_advance(iter)){
+                       p = NULL;
+                       break;
+               }
+               p = hashtable_iterator_value(iter);
+       }
+
+       spin_unlock(&pid_hash_lock);
+       kfree(iter);
+       return p;
+}
+
 /* Performs any initialization related to processes, such as create the proc
  * cache, prep the scheduler, etc.  When this returns, we should be ready to use
  * any process related function. */
@@ -186,6 +256,14 @@ void proc_init(void)
        atomic_init(&num_envs, 0);
 }
 
+void proc_set_progname(struct proc *p, char *name)
+{
+       /* might have an issue if a dentry name isn't null terminated, and we'd get
+        * extra junk up to progname_sz. */
+       strncpy(p->progname, name, PROC_PROGNAME_SZ);
+       p->progname[PROC_PROGNAME_SZ - 1] = '\0';
+}
+
 /* Be sure you init'd the vcore lists before calling this. */
 static void proc_init_procinfo(struct proc* p)
 {
@@ -194,7 +272,7 @@ static void proc_init_procinfo(struct proc* p)
        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 = (void*)UTEXT;
+       p->procinfo->heap_bottom = 0;
        /* 0'ing the arguments.  Some higher function will need to set them */
        memset(p->procinfo->argp, 0, sizeof(p->procinfo->argp));
        memset(p->procinfo->argbuf, 0, sizeof(p->procinfo->argbuf));
@@ -205,10 +283,8 @@ static void proc_init_procinfo(struct proc* p)
        p->procinfo->num_vcores = 0;
        p->procinfo->is_mcp = FALSE;
        p->procinfo->coremap_seqctr = SEQCTR_INITIALIZER;
-       /* For now, we'll go up to the max num_cpus (at runtime).  In the future,
-        * there may be cases where we can have more vcores than num_cpus, but for
-        * now we'll leave it like this. */
-       for (int i = 0; i < num_cpus; i++) {
+       /* It's a bug in the kernel if we let them ask for more than max */
+       for (int i = 0; i < p->procinfo->max_vcores; i++) {
                TAILQ_INSERT_TAIL(&p->inactive_vcs, &p->procinfo->vcoremap[i], list);
        }
 }
@@ -226,7 +302,7 @@ static void proc_init_procdata(struct proc *p)
  * Errors include:
  *  - ENOFREEPID if it can't get a PID
  *  - ENOMEM on memory exhaustion */
-error_t proc_alloc(struct proc **pp, struct proc *parent)
+error_t proc_alloc(struct proc **pp, struct proc *parent, int flags)
 {
        error_t r;
        struct proc *p;
@@ -258,6 +334,7 @@ error_t proc_alloc(struct proc **pp, struct proc *parent)
        p->exitcode = 1337;     /* so we can see processes killed by the kernel */
        if (parent) {
                p->ppid = parent->pid;
+               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);
                TAILQ_INSERT_TAIL(&parent->children, p, sibling_link);
@@ -270,11 +347,14 @@ error_t proc_alloc(struct proc **pp, struct proc *parent)
        p->state = PROC_CREATED; /* shouldn't go through state machine for init */
        p->env_flags = 0;
        p->env_entry = 0; // cheating.  this really gets set later
-       p->heap_top = (void*)UTEXT;     /* heap_bottom set in proc_init_procinfo */
-       spinlock_init(&p->mm_lock);
+       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 */
-       /* Initialize the vcore lists, we'll build the inactive list so that it includes
-        * all vcores when we initialize procinfo.  Do this before initing procinfo. */
+       p->vmr_history = 0;
+       /* Initialize the vcore lists, we'll build the inactive list so that it
+        * includes all vcores when we initialize procinfo.  Do this before initing
+        * procinfo. */
        TAILQ_INIT(&p->online_vcs);
        TAILQ_INIT(&p->bulk_preempted_vcs);
        TAILQ_INIT(&p->inactive_vcs);
@@ -304,12 +384,27 @@ error_t proc_alloc(struct proc **pp, struct proc *parent)
        p->open_files.max_fdset = NR_FILE_DESC_DEFAULT;
        p->open_files.fd = p->open_files.fd_array;
        p->open_files.open_fds = (struct fd_set*)&p->open_files.open_fds_init;
+       if (parent) {
+               if (flags & PROC_DUP_FGRP)
+                       clone_files(&parent->open_files, &p->open_files);
+       } else {
+               /* no parent, we're created from the kernel */
+               assert(insert_file(&p->open_files, dev_stdin,  0, TRUE) == 0);
+               assert(insert_file(&p->open_files, dev_stdout, 1, TRUE) == 1);
+               assert(insert_file(&p->open_files, dev_stderr, 2, TRUE) == 2);
+       }
        /* 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);
+       /* this does all the 9ns setup, much of which is done throughout this func
+        * for the VFS, including duping the fgrp */
+       plan9setup(p, parent, flags);
+       devalarm_init(p);
+       TAILQ_INIT(&p->abortable_sleepers);
+       spinlock_init_irqsave(&p->abort_list_lock);
        printd("[%08x] new process %08x\n", current ? current->pid : 0, p->pid);
        } // INIT_STRUCT
        *pp = p;
@@ -336,33 +431,46 @@ struct proc *proc_create(struct file *prog, char **argv, char **envp)
 {
        struct proc *p;
        error_t r;
-       if ((r = proc_alloc(&p, current)) < 0)
+       if ((r = proc_alloc(&p, current, 0 /* flags */)) < 0)
                panic("proc_create: %e", r);    /* one of 3 quaint usages of %e */
+       proc_set_progname(p, file_name(prog));
        procinfo_pack_args(p->procinfo, argv, envp);
        assert(load_elf(p, prog) == 0);
-       /* Connect to stdin, stdout, stderr */
-       assert(insert_file(&p->open_files, dev_stdin,  0) == 0);
-       assert(insert_file(&p->open_files, dev_stdout, 0) == 1);
-       assert(insert_file(&p->open_files, dev_stderr, 0) == 2);
        __proc_ready(p);
        return p;
 }
 
+static int __cb_assert_no_pg(struct proc *p, pte_t *pte, void *va, void *arg)
+{
+       assert(!*pte);
+       return 0;
+}
+
 /* This is called by kref_put(), once the last reference to the process is
  * gone.  Don't call this otherwise (it will panic).  It will clean up the
  * address space and deallocate any other used memory. */
 static void __proc_free(struct kref *kref)
 {
        struct proc *p = container_of(kref, struct proc, p_kref);
+       void *hash_ret;
        physaddr_t pa;
 
        printd("[PID %d] freeing proc: %d\n", current ? current->pid : 0, p->pid);
        // All parts of the kernel should have decref'd before __proc_free is called
        assert(kref_refcnt(&p->p_kref) == 0);
-
+       assert(TAILQ_EMPTY(&p->alarmset.list));
+
+       p->progname[0] = 0;
+       cclose(p->dot);
+       cclose(p->slash);
+       p->dot = p->slash = 0; /* catch bugs */
+       /* can safely free the fgrp, now that no one is accessing it */
+       kfree(p->fgrp->fd);
+       kfree(p->fgrp);
        kref_put(&p->fs_env.root->d_kref);
        kref_put(&p->fs_env.pwd->d_kref);
-       destroy_vmrs(p);
+       /* 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) {
@@ -372,13 +480,19 @@ static void __proc_free(struct kref *kref)
        }
        /* Remove us from the pid_hash and give our PID back (in that order). */
        spin_lock(&pid_hash_lock);
-       if (!hashtable_remove(pid_hash, (void*)(long)p->pid))
-               panic("Proc not in the pid table in %s", __FUNCTION__);
+       hash_ret = hashtable_remove(pid_hash, (void*)(long)p->pid);
        spin_unlock(&pid_hash_lock);
-       put_free_pid(p->pid);
-       /* Flush all mapped pages in the user portion of the address space */
-       env_user_mem_free(p, 0, UVPT);
-       /* These need to be free again, since they were allocated with a refcnt. */
+       /* might not be in the hash/ready, if we failed during proc creation */
+       if (hash_ret)
+               put_free_pid(p->pid);
+       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... */
+       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));
 
@@ -392,11 +506,16 @@ static void __proc_free(struct kref *kref)
        kmem_cache_free(proc_cache, p);
 }
 
-/* Whether or not actor can control target.  Note we currently don't need
- * locking for this. TODO: think about that, esp wrt proc's dying. */
+/* Whether or not actor can control target.  TODO: do something reasonable here.
+ * Just checking for the parent is a bit limiting.  Could walk the parent-child
+ * tree, check user ids, or some combination.  Make sure actors can always
+ * control themselves. */
 bool proc_controls(struct proc *actor, struct proc *target)
 {
+       return TRUE;
+       #if 0 /* Example: */
        return ((actor == target) || (target->ppid == actor->pid));
+       #endif
 }
 
 /* Helper to incref by val.  Using the helper to help debug/interpose on proc
@@ -474,6 +593,7 @@ void proc_run_s(struct proc *p)
                         * lists).  This gets unmapped in resource.c and yield_s, and needs
                         * work. */
                        __map_vcore(p, 0, coreid); /* not treated like a true vcore */
+                       vcore_account_online(p, 0); /* VC# */
                        __seq_end_write(&p->procinfo->coremap_seqctr);
                        /* incref, since we're saving a reference in owning proc later */
                        proc_incref(p, 1);
@@ -499,7 +619,7 @@ void proc_run_s(struct proc *p)
                                pcpui->cur_ctx = &pcpui->actual_ctx;
                                memset(pcpui->cur_ctx, 0, sizeof(struct user_context));
                                proc_init_ctx(pcpui->cur_ctx, 0, p->env_entry,
-                                             vcpd->transition_stack);
+                                             vcpd->transition_stack, vcpd->vcore_tls_desc);
                        } else {
                                /* If they have no transition stack, then they can't receive
                                 * events.  The most they are getting is a wakeup from the
@@ -604,7 +724,9 @@ void __proc_run_m(struct proc *p)
        }
 }
 
-/* Actually runs the given context (trapframe) of process p on the core this
+/* You must disable IRQs and PRKM before calling this.
+ *
+ * Actually runs the given context (trapframe) of process p on the core this
  * code executes on.  This is called directly by __startcore, which needs to
  * bypass the routine_kmsg check.  Interrupts should be off when you call this.
  *
@@ -617,12 +739,17 @@ void __proc_run_m(struct proc *p)
  * 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. */
-static void __proc_startcore(struct proc *p, struct user_context *ctx)
+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);
        __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);
 }
 
@@ -632,22 +759,18 @@ static void __proc_startcore(struct proc *p, struct user_context *ctx)
  * 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.
- *
- * Refcnting: this will not return, and it assumes that you've accounted for
- * your reference as if it was the ref for "current" (which is what happens when
- * returning from local traps and such. */
+ * but that would have crappy overhead. */
 void proc_restartcore(void)
 {
        struct per_cpu_info *pcpui = &per_cpu_info[core_id()];
-       assert(!pcpui->cur_sysc);
+       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 processing (race on missing
+       /* Need ints disabled when we return from PRKM (race on missing
         * messages/IPIs) */
        disable_irq();
        process_routine_kmsg();
@@ -751,12 +874,20 @@ void proc_destroy(struct proc *p)
        spin_unlock(&p->proc_lock);
        /* Wake any of our kthreads waiting on children, so they can abort */
        cv_broadcast(&p->child_wait);
-       /* This prevents processes from accessing their old files while dying, and
-        * will help if these files (or similar objects in the future) hold
-        * references to p (preventing a __proc_free()).  Need to unlock before
-        * doing this - the proclock doesn't protect the files (not proc state), and
-        * closing these might block (can't block while spinning). */
-       /* TODO: might need some sync protection */
+       /* 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).
+        *
+        * Also note that any mmap'd files will still be mmapped.  You can close the
+        * file after mmapping, with no effect. */
+       close_9ns_files(p, FALSE);
        close_all_files(&p->open_files, FALSE);
        /* Tell the ksched about our death, and which cores we freed up */
        __sched_proc_destroy(p, pc_arr, nr_cores_revoked);
@@ -794,7 +925,7 @@ 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 after dying */
+       proc_decref(child);     /* ref that was keeping the child alive on the list */
        return 0;
 }
 
@@ -835,6 +966,7 @@ int proc_change_to_m(struct proc *p)
                        // TODO: (VC#) might need to adjust num_vcores
                        // TODO: (ACR) will need to unmap remotely (receive-side)
                        __unmap_vcore(p, 0);    /* VC# keep in sync with proc_run_s */
+                       vcore_account_offline(p, 0); /* VC# */
                        __seq_end_write(&p->procinfo->coremap_seqctr);
                        /* change to runnable_m (it's TF is already saved) */
                        __proc_set_state(p, PROC_RUNNABLE_M);
@@ -869,6 +1001,7 @@ uint32_t __proc_change_to_s(struct proc *p, uint32_t *pc_arr)
 {
        struct preempt_data *vcpd = &p->procdata->vcore_preempt_data[0];
        uint32_t num_revoked;
+       /* Not handling vcore accounting.  Do so if we ever use this */
        printk("[kernel] trying to transition _M -> _S (deprecated)!\n");
        assert(p->state == PROC_RUNNING_M); // TODO: (ACR) async core req
        /* save the context, to be restarted in _S mode */
@@ -960,6 +1093,7 @@ void __proc_save_context_s(struct proc *p, struct user_context *ctx)
 {
        p->scp_ctx = *ctx;
        __unmap_vcore(p, 0);    /* VC# keep in sync with proc_run_s */
+       vcore_account_offline(p, 0); /* VC# */
 }
 
 /* Yields the calling core.  Must be called locally (not async) for now.
@@ -1095,6 +1229,8 @@ void proc_yield(struct proc *SAFE p, bool being_nice)
                TAILQ_INSERT_TAIL(&p->online_vcs, vc, list); /* could go HEAD */
                goto out_failed;
        }
+       /* Not really a kmsg, but it acts like one w.r.t. proc mgmt */
+       pcpui_trace_kmsg(pcpui, (uintptr_t)proc_yield);
        /* We won the race with event sending, we can safely yield */
        TAILQ_INSERT_HEAD(&p->inactive_vcs, vc, list);
        /* Note this protects stuff userspace should look at, which doesn't
@@ -1106,6 +1242,7 @@ void proc_yield(struct proc *SAFE p, bool being_nice)
        p->procinfo->num_vcores--;
        p->procinfo->res_grant[RES_CORES] = p->procinfo->num_vcores;
        __seq_end_write(&p->procinfo->coremap_seqctr);
+       vcore_account_offline(p, vcoreid);
        /* No more vcores?  Then we wait on an event */
        if (p->procinfo->num_vcores == 0) {
                /* consider a ksched op to tell it about us WAITING */
@@ -1623,7 +1760,8 @@ void 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_sysc = 0;
+       pcpui->cur_kthread->sysc = 0;
+       pcpui->cur_kthread->errbuf = 0; /* just in case */
        if (pcpui->cur_proc)
                __abandon_core();
 }
@@ -1655,7 +1793,10 @@ struct proc *switch_to(struct proc *new_p)
        /* If we aren't the proc already, then switch to it */
        if (old_proc != new_p) {
                pcpui->cur_proc = new_p;                                /* uncounted ref */
-               lcr3(new_p->env_cr3);
+               if (new_p)
+                       lcr3(new_p->env_cr3);
+               else
+                       lcr3(boot_cr3);
        }
        return old_proc;
 }
@@ -1686,6 +1827,8 @@ void switch_back(struct proc *new_p, struct proc *old_proc)
  * immediate message. */
 void proc_tlbshootdown(struct proc *p, uintptr_t start, uintptr_t end)
 {
+       /* TODO: need a better way to find cores running our address space.  we can
+        * have kthreads running syscalls, async calls, processes being created. */
        struct vcore *vc_i;
        /* TODO: we might be able to avoid locking here in the future (we must hit
         * all online, and we can check __mapped).  it'll be complicated. */
@@ -1701,14 +1844,11 @@ void proc_tlbshootdown(struct proc *p, uintptr_t start, uintptr_t end)
                                                    0, KMSG_IMMEDIATE);
                        }
                        break;
-               case (PROC_DYING):
-                       /* if it is dying, death messages are already on the way to all
-                        * cores, including ours, which will clear the TLB. */
-                       break;
                default:
-                       /* will probably get this when we have the short handlers */
-                       warn("Unexpected case %s in %s", procstate2str(p->state),
-                            __FUNCTION__);
+                       /* TODO: til we fix shootdowns, there are some odd cases where we
+                        * have the address space loaded, but the state is in transition. */
+                       if (p == current)
+                               tlbflush();
        }
        spin_unlock(&p->proc_lock);
 }
@@ -1755,7 +1895,7 @@ static void __set_curctx_to_vcoreid(struct proc *p, uint32_t vcoreid,
        } else { /* not restarting from a preemption, use a fresh vcore */
                assert(vcpd->transition_stack);
                proc_init_ctx(&pcpui->actual_ctx, vcoreid, p->env_entry,
-                             vcpd->transition_stack);
+                             vcpd->transition_stack, vcpd->vcore_tls_desc);
                /* Disable/mask active notifications for fresh vcores */
                vcpd->notif_disabled = TRUE;
        }
@@ -1776,6 +1916,7 @@ static void __set_curctx_to_vcoreid(struct proc *p, uint32_t vcoreid,
        /* cur_ctx was built above (in actual_ctx), now use it */
        pcpui->cur_ctx = &pcpui->actual_ctx;
        /* this cur_ctx will get run when the kernel returns / idles */
+       vcore_account_online(p, vcoreid);
 }
 
 /* Changes calling vcore to be vcoreid.  enable_my_notif tells us about how the
@@ -1876,6 +2017,7 @@ int proc_change_to_vcore(struct proc *p, uint32_t new_vcoreid,
        __unmap_vcore(p, caller_vcoreid);
        __map_vcore(p, new_vcoreid, pcoreid);
        __seq_end_write(&p->procinfo->coremap_seqctr);
+       vcore_account_offline(p, caller_vcoreid);
        /* Send either a PREEMPT msg or a CHECK_MSGS msg.  If they said to
         * enable_my_notif, then all userspace needs is to check messages, not a
         * full preemption recovery. */
@@ -1985,7 +2127,7 @@ void __notify(uint32_t srcid, long a0, long a1, long a2)
        vcpd->uthread_ctx = *pcpui->cur_ctx;
        memset(pcpui->cur_ctx, 0, sizeof(struct user_context));
        proc_init_ctx(pcpui->cur_ctx, vcoreid, p->env_entry,
-                     vcpd->transition_stack);
+                     vcpd->transition_stack, vcpd->vcore_tls_desc);
        /* this cur_ctx will get run when the kernel returns / idles */
 }
 
@@ -2031,6 +2173,7 @@ void __preempt(uint32_t srcid, long a0, long a1, long a2)
        atomic_and(&vcpd->flags, ~VC_K_LOCK);
        /* either __preempt or proc_yield() ends the preempt phase. */
        p->procinfo->vcoremap[vcoreid].preempt_pending = 0;
+       vcore_account_offline(p, vcoreid);
        wmb();  /* make sure everything else hits before we finish the preempt */
        /* up the nr_done, which signals the next __startcore for this vc */
        p->procinfo->vcoremap[vcoreid].nr_preempts_done++;
@@ -2053,6 +2196,7 @@ void __death(uint32_t srcid, long a0, long a1, long a2)
                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) */
@@ -2074,10 +2218,18 @@ void print_allpids(void)
        {
                struct proc *p = (struct proc*)item;
                assert(p);
-               printk("%8d %-10s %6d\n", p->pid, procstate2str(p->state), p->ppid);
+               /* this actually adds an extra space, since no progname is ever
+                * PROGNAME_SZ bytes, due to the \0 counted in PROGNAME. */
+               printk("%8d %-*s %-10s %6d\n", p->pid, PROC_PROGNAME_SZ, p->progname,
+                      procstate2str(p->state), p->ppid);
        }
-       printk("     PID STATE      Parent    \n");
-       printk("------------------------------\n");
+       char dashes[PROC_PROGNAME_SZ];
+       memset(dashes, '-', PROC_PROGNAME_SZ);
+       dashes[PROC_PROGNAME_SZ - 1] = '\0';
+       /* -5, for 'Name ' */
+       printk("     PID Name %-*s State      Parent    \n",
+              PROC_PROGNAME_SZ - 5, "");
+       printk("------------------------------%s\n", dashes);
        spin_lock(&pid_hash_lock);
        hash_for_each(pid_hash, print_proc_state);
        spin_unlock(&pid_hash_lock);
@@ -2086,6 +2238,7 @@ void print_allpids(void)
 void print_proc_info(pid_t pid)
 {
        int j = 0;
+       uint64_t total_time = 0;
        struct proc *child, *p = pid2proc(pid);
        struct vcore *vc_i;
        if (!p) {
@@ -2095,12 +2248,14 @@ void print_proc_info(pid_t pid)
        spinlock_debug(&p->proc_lock);
        //spin_lock(&p->proc_lock); // No locking!!
        printk("struct proc: %p\n", p);
+       printk("Program name: %s\n", p->progname);
        printk("PID: %d\n", p->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 ");
        printk("Refcnt: %d\n", atomic_read(&p->p_kref.refcount) - 1);
        printk("Flags: 0x%08x\n", p->env_flags);
-       printk("CR3(phys): 0x%08x\n", p->env_cr3);
+       printk("CR3(phys): %p\n", p->env_cr3);
        printk("Num Vcores: %d\n", p->procinfo->num_vcores);
        printk("Vcore Lists (may be in flux w/o locking):\n----------------------\n");
        printk("Online:\n");
@@ -2112,6 +2267,16 @@ 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;
+       }
+       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,
@@ -2120,15 +2285,16 @@ void print_proc_info(pid_t pid)
        struct files_struct *files = &p->open_files;
        spin_lock(&files->lock);
        for (int i = 0; i < files->max_files; i++)
-               if (files->fd_array[i].fd_file) {
-                       printk("\tFD: %02d, File: %08p, File name: %s\n", i,
-                              files->fd_array[i].fd_file,
-                              file_name(files->fd_array[i].fd_file));
+               if (GET_BITMASK_BIT(files->open_fds->fds_bits, i) &&
+                   (files->fd[i].fd_file)) {
+                       printk("\tFD: %02d, File: %p, File name: %s\n", i,
+                              files->fd[i].fd_file, file_name(files->fd[i].fd_file));
                }
        spin_unlock(&files->lock);
+       print_9ns_files(p);
        printk("Children: (PID (struct proc *))\n");
        TAILQ_FOREACH(child, &p->children, sibling_link)
-               printk("\t%d (%08p)\n", child->pid, child);
+               printk("\t%d (%p)\n", child->pid, child);
        /* no locking / unlocking or refcnting */
        // spin_unlock(&p->proc_lock);
        proc_decref(p);
@@ -2153,7 +2319,7 @@ void check_my_owner(void)
                                 * interrupts, which should cause us to skip cpu_halt() */
                                if (!STAILQ_EMPTY(&pcpui->immed_amsgs))
                                        continue;
-                               printk("Owned pcore (%d) has no owner, by %08p, vc %d!\n",
+                               printk("Owned pcore (%d) has no owner, by %p, vc %d!\n",
                                       core_id(), p, vcore2vcoreid(p, vc_i));
                                spin_unlock(&p->proc_lock);
                                spin_unlock(&pid_hash_lock);
@@ -2170,3 +2336,16 @@ void check_my_owner(void)
                spin_unlock(&pid_hash_lock);
        }
 }
+
+/* Use this via kfunc */
+void print_9ns(void)
+{
+       void print_proc_9ns(void *item)
+       {
+               struct proc *p = (struct proc*)item;
+               print_9ns_files(p);
+       }
+       spin_lock(&pid_hash_lock);
+       hash_for_each(pid_hash, print_proc_9ns);
+       spin_unlock(&pid_hash_lock);
+}