Set errno when only errstr was called
[akaros.git] / kern / src / syscall.c
index 95443e8..e506d76 100644 (file)
@@ -9,7 +9,7 @@
 #include <arch/arch.h>
 #include <arch/mmu.h>
 #include <arch/console.h>
-#include <ros/timer.h>
+#include <time.h>
 #include <error.h>
 
 #include <elf.h>
@@ -24,7 +24,6 @@
 #include <syscall.h>
 #include <kmalloc.h>
 #include <stdio.h>
-#include <resource.h>
 #include <frontend.h>
 #include <colored_caches.h>
 #include <hashtable.h>
 #include <smp.h>
 #include <arsc_server.h>
 #include <event.h>
+#include <termios.h>
+#include <socket.h>
 
 
-#ifdef __CONFIG_NETWORKING__
-#include <arch/nic_common.h>
+#ifdef CONFIG_NETWORKING
+#include <net/nic_common.h>
 extern int (*send_frame)(const char *CT(len) data, size_t len);
 extern unsigned char device_mac[6];
 #endif
@@ -45,10 +46,10 @@ extern unsigned char device_mac[6];
 /* Tracing Globals */
 int systrace_flags = 0;
 struct systrace_record *systrace_buffer = 0;
-unsigned int systrace_bufidx = 0;
+uint32_t systrace_bufidx = 0;
 size_t systrace_bufsize = 0;
 struct proc *systrace_procs[MAX_NUM_TRACED] = {0};
-spinlock_t systrace_lock = SPINLOCK_INITIALIZER;
+spinlock_t systrace_lock = SPINLOCK_INITIALIZER_IRQSAVE;
 
 /* Not enforcing the packing of systrace_procs yet, but don't rely on that */
 static bool proc_is_traced(struct proc *p)
@@ -59,10 +60,24 @@ static bool proc_is_traced(struct proc *p)
        return false;
 }
 
-/* Helper that "finishes" the current async syscall.  This should be used when
- * we are calling a function in a syscall that might not return and won't be
- * able to use the normal syscall return path, such as proc_yield() and
- * resource_req().  Call this from within syscall.c (I don't want it global).
+/* Helper to finish a syscall, signalling if appropriate */
+static void finish_sysc(struct syscall *sysc, struct proc *p)
+{
+       /* Atomically turn on the LOCK and SC_DONE flag.  The lock tells userspace
+        * we're messing with the flags and to not proceed.  We use it instead of
+        * CASing with userspace.  We need the atomics since we're racing with
+        * userspace for the event_queue registration.  The 'lock' tells userspace
+        * to not muck with the flags while we're signalling. */
+       atomic_or(&sysc->flags, SC_K_LOCK | SC_DONE); 
+       __signal_syscall(sysc, p);
+       atomic_and(&sysc->flags, ~SC_K_LOCK); 
+}
+
+/* Helper that "finishes" the current async syscall.  This should be used with
+ * care when we are not using the normal syscall completion path.
+ *
+ * Do *NOT* complete the same syscall twice.  This is catastrophic for _Ms, and
+ * a bad idea for _S.
  *
  * It is possible for another user thread to see the syscall being done early -
  * they just need to be careful with the weird proc management calls (as in,
@@ -70,11 +85,12 @@ static bool proc_is_traced(struct proc *p)
  *
  * *sysc is in user memory, and should be pinned (TODO: UMEM).  There may be
  * issues with unpinning this if we never return. */
-static void signal_current_sc(int retval)
+static void finish_current_sysc(int retval)
 {
        struct per_cpu_info *pcpui = &per_cpu_info[core_id()];
+       assert(pcpui->cur_sysc);
        pcpui->cur_sysc->retval = retval;
-       pcpui->cur_sysc->flags |= SC_DONE;
+       finish_sysc(pcpui->cur_sysc, pcpui->cur_proc);
 }
 
 /* Callable by any function while executing a syscall (or otherwise, actually).
@@ -86,6 +102,36 @@ void set_errno(int errno)
                pcpui->cur_sysc->err = errno;
 }
 
+void set_errstr(char *errstr)
+{
+       struct per_cpu_info *pcpui = &per_cpu_info[core_id()];
+       if (!pcpui->cur_sysc)
+               return;
+       strncpy(pcpui->cur_sysc->errstr, errstr, MAX_ERRSTR_LEN);
+       /* enforce null termination */
+       pcpui->cur_sysc->errstr[MAX_ERRSTR_LEN - 1] = '\0';
+}
+
+char *current_errstr(void)
+{
+       struct per_cpu_info *pcpui = &per_cpu_info[core_id()];
+       /* no one should call this that doesn't have a sysc */
+       assert(!pcpui->cur_sysc);
+       return pcpui->cur_sysc->errstr;
+}
+
+struct errbuf *get_cur_errbuf(void)
+{
+       struct per_cpu_info *pcpui = &per_cpu_info[core_id()];
+       return (struct errbuf*)pcpui->cur_errbuf;
+}
+
+void set_cur_errbuf(struct errbuf *ebuf)
+{
+       struct per_cpu_info *pcpui = &per_cpu_info[core_id()];
+       pcpui->cur_errbuf = ebuf;
+}
+
 /************** Utility Syscalls **************/
 
 static int sys_null(void)
@@ -94,44 +140,19 @@ static int sys_null(void)
 }
 
 /* Diagnostic function: blocks the kthread/syscall, to help userspace test its
- * async I/O handling.  Don't mix this with things that mess with the interrupt
- * handler, like other sys_blocks or the current blockdev crap. */
-static int sys_block(void)
-{
-       struct semaphore local_sem, *sem = &local_sem;
-       init_sem(sem, 0);
-#ifdef __i386__        /* Sparc can't register interrupt handlers yet */
-       /* Faking an interrupt.  The handler runs in interrupt context btw */
-       void x86_unblock_handler(struct trapframe *tf, void *data)
-       {
-               /* Turn off the interrupt, Re-register the old dumb handler */
-               set_core_timer(0);
-               register_interrupt_handler(interrupt_handlers,
-                                          LAPIC_TIMER_DEFAULT_VECTOR, timer_interrupt,
-                                          NULL);
-               struct semaphore *sem = (struct semaphore*)data;
-               struct kthread *sleeper = __up_sem(sem);
-               if (!sleeper) {
-                       warn("No one sleeping!");
-                       return;
-               }
-               kthread_runnable(sleeper);
-               assert(TAILQ_EMPTY(&sem->waiters));
-       }
-
-       register_interrupt_handler(interrupt_handlers, LAPIC_TIMER_DEFAULT_VECTOR,
-                                  x86_unblock_handler, sem);
-       /* This fakes a 100ms delay.  Though it might be less, esp in _M mode.  TODO
-        * KVM-timing. */
-       set_core_timer(100000); /* in microseconds */
-       printk("[kernel] sys_block(), sleeping at %llu\n", read_tsc());
-       sleep_on(sem);
-       printk("[kernel] sys_block(), waking up at %llu\n", read_tsc());
+ * async I/O handling. */
+static int sys_block(struct proc *p, unsigned int usec)
+{
+       struct timer_chain *tchain = &per_cpu_info[core_id()].tchain;
+       struct alarm_waiter a_waiter;
+       init_awaiter(&a_waiter, 0);
+       /* Note printing takes a few ms, so your printds won't be perfect. */
+       printd("[kernel] sys_block(), sleeping at %llu\n", read_tsc());
+       set_awaiter_rel(&a_waiter, usec);
+       set_alarm(tchain, &a_waiter);
+       sleep_on_awaiter(&a_waiter);
+       printd("[kernel] sys_block(), waking up at %llu\n", read_tsc());
        return 0;
-#else /* sparc */
-       set_errno(ENOSYS);
-       return -1;
-#endif
 }
 
 // Writes 'val' to 'num_writes' entries of the well-known array in the kernel
@@ -141,7 +162,7 @@ static int sys_block(void)
 static int sys_cache_buster(struct proc *p, uint32_t num_writes,
                              uint32_t num_pages, uint32_t flags)
 { TRUSTEDBLOCK /* zra: this is not really part of the kernel */
-       #define BUSTER_ADDR             0xd0000000  // around 512 MB deep
+       #define BUSTER_ADDR             0xd0000000L  // around 512 MB deep
        #define MAX_WRITES              1048576*8
        #define MAX_PAGES               32
        #define INSERT_ADDR     (UINFO + 2*PGSIZE) // should be free for these tests
@@ -210,7 +231,7 @@ static int sys_cache_buster(struct proc *p, uint32_t num_writes,
 
 static int sys_cache_invalidate(void)
 {
-       #ifdef __i386__
+       #ifdef CONFIG_X86
                wbinvd();
        #endif
        return 0;
@@ -233,20 +254,21 @@ static ssize_t sys_cputs(struct proc *p, const char *DANGEROUS string,
 
 // Read a character from the system console.
 // Returns the character.
+/* TODO: remove me */
 static uint16_t sys_cgetc(struct proc *p)
 {
        uint16_t c;
 
-       // The cons_getc() primitive doesn't wait for a character,
+       // The cons_get_any_char() primitive doesn't wait for a character,
        // but the sys_cgetc() system call does.
-       while ((c = cons_getc()) == 0)
+       while ((c = cons_get_any_char()) == 0)
                cpu_relax();
 
        return c;
 }
 
-/* Returns the id of the cpu this syscall is executed on. */
-static uint32_t sys_getcpuid(void)
+/* Returns the id of the physical core this syscall is executed on. */
+static uint32_t sys_getpcoreid(void)
 {
        return core_id();
 }
@@ -255,7 +277,7 @@ static uint32_t sys_getcpuid(void)
 // this is removed from the user interface
 static size_t sys_getvcoreid(struct proc *p)
 {
-       return proc_get_vcoreid(p, core_id());
+       return proc_get_vcoreid(p);
 }
 
 /************** Process management syscalls **************/
@@ -310,6 +332,7 @@ static int sys_proc_create(struct proc *p, char *path, size_t path_l,
        return pid;
 late_error:
        proc_destroy(new_p);
+       proc_decref(new_p);     /* give up the reference created in proc_create() */
 mid_error:
        kref_put(&program->f_kref);
        return -1;
@@ -321,29 +344,31 @@ static error_t sys_proc_run(struct proc *p, unsigned pid)
        struct proc *target = pid2proc(pid);
        error_t retval = 0;
 
-       if (!target)
-               return -EBADPROC;
-       // note we can get interrupted here. it's not bad.
-       spin_lock(&p->proc_lock);
-       // make sure we have access and it's in the right state to be activated
+       if (!target) {
+               set_errno(ESRCH);
+               return -1;
+       }
+       /* make sure we have access and it's in the right state to be activated */
        if (!proc_controls(p, target)) {
-               proc_decref(target);
-               retval = -EPERM;
+               set_errno(EPERM);
+               goto out_error;
        } else if (target->state != PROC_CREATED) {
-               proc_decref(target);
-               retval = -EINVAL;
-       } else {
-               __proc_set_state(target, PROC_RUNNABLE_S);
-               schedule_proc(target);
+               set_errno(EINVAL);
+               goto out_error;
        }
-       spin_unlock(&p->proc_lock);
+       /* Note a proc can spam this for someone it controls.  Seems safe - if it
+        * isn't we can change it. */
+       proc_wakeup(target);
        proc_decref(target);
-       return retval;
+       return 0;
+out_error:
+       proc_decref(target);
+       return -1;
 }
 
 /* Destroy proc pid.  If this is called by the dying process, it will never
  * return.  o/w it will return 0 on success, or an error.  Errors include:
- * - EBADPROC: if there is no such process with pid
+ * - ESRCH: if there is no such process with pid
  * - EPERM: if caller does not control pid */
 static error_t sys_proc_destroy(struct proc *p, pid_t pid, int exitcode)
 {
@@ -363,28 +388,43 @@ static error_t sys_proc_destroy(struct proc *p, pid_t pid, int exitcode)
                p->exitcode = exitcode;
                printd("[PID %d] proc exiting gracefully (code %d)\n", p->pid,exitcode);
        } else {
+               p_to_die->exitcode = exitcode;  /* so its parent has some clue */
                printd("[%d] destroying proc %d\n", p->pid, p_to_die->pid);
        }
        proc_destroy(p_to_die);
        /* we only get here if we weren't the one to die */
        proc_decref(p_to_die);
-       return ESUCCESS;
+       return 0;
 }
 
 static int sys_proc_yield(struct proc *p, bool being_nice)
 {
+       struct per_cpu_info *pcpui = &per_cpu_info[core_id()];
        /* proc_yield() often doesn't return - we need to set the syscall retval
         * early.  If it doesn't return, it expects to eat our reference (for now).
         */
-       signal_current_sc(0);
+       finish_sysc(pcpui->cur_sysc, pcpui->cur_proc);
+       pcpui->cur_sysc = 0;    /* don't touch sysc again */
        proc_incref(p, 1);
        proc_yield(p, being_nice);
        proc_decref(p);
-       return 0;
+       /* Shouldn't return, to prevent the chance of mucking with cur_sysc. */
+       smp_idle();
+       assert(0);
+}
+
+static int sys_change_vcore(struct proc *p, uint32_t vcoreid,
+                             bool enable_my_notif)
+{
+       /* Note retvals can be negative, but we don't mess with errno in case
+        * callers use this in low-level code and want to extract the 'errno'. */
+       return proc_change_to_vcore(p, vcoreid, enable_my_notif);
 }
 
 static ssize_t sys_fork(env_t* e)
 {
+       struct proc *temp;
+       int8_t state = 0;
        // TODO: right now we only support fork for single-core processes
        if (e->state != PROC_RUNNING_S) {
                set_errno(EINVAL);
@@ -396,81 +436,58 @@ static ssize_t sys_fork(env_t* e)
 
        env->heap_top = e->heap_top;
        env->ppid = e->pid;
-       /* Can't really fork if we don't have a current_tf to fork */
-       if (!current_tf) {
+       disable_irqsave(&state);        /* protect cur_ctx */
+       /* Can't really fork if we don't have a current_ctx to fork */
+       if (!current_ctx) {
                set_errno(EINVAL);
                return -1;
        }
-       env->env_tf = *current_tf;
-
-       /* We need to speculatively say the syscall worked before copying the memory
-        * out, since the 'forked' process's call never actually goes through the
-        * syscall return path, and will never think it is done.  This violates a
-        * few things.  Just be careful with fork. */
-       signal_current_sc(0);
+       env->scp_ctx = *current_ctx;
+       enable_irqsave(&state);
 
        env->cache_colors_map = cache_colors_map_alloc();
        for(int i=0; i < llc_cache->num_colors; i++)
                if(GET_BITMASK_BIT(e->cache_colors_map,i))
                        cache_color_alloc(llc_cache, env->cache_colors_map);
 
-       duplicate_vmrs(e, env);
-
-       int copy_page(env_t* e, pte_t* pte, void* va, void* arg)
-       {
-               env_t* env = (env_t*)arg;
-
-               if(PAGE_PRESENT(*pte))
-               {
-                       page_t* pp;
-                       if(upage_alloc(env,&pp,0))
-                               return -1;
-                       if(page_insert(env->env_pgdir,pp,va,*pte & PTE_PERM))
-                       {
-                               page_decref(pp);
-                               return -1;
-                       }
-                       pagecopy(page2kva(pp),ppn2kva(PTE2PPN(*pte)));
-                       page_decref(pp);
-               } else {
-                       assert(PAGE_PAGED_OUT(*pte));
-                       /* TODO: (SWAP) will need to either make a copy or CoW/refcnt the
-                        * backend store.  For now, this PTE will be the same as the
-                        * original PTE */
-                       panic("Swapping not supported!");
-                       pte_t* newpte = pgdir_walk(env->env_pgdir,va,1);
-                       if(!newpte)
-                               return -1;
-                       *newpte = *pte;
-               }
-               return 0;
-       }
-
-       // TODO: (PC) this won't work.  Needs revisiting.
-       // copy procdata and procinfo
-       memcpy(env->procdata,e->procdata,sizeof(struct procdata));
-       memcpy(env->procinfo,e->procinfo,sizeof(struct procinfo));
-       env->procinfo->pid = env->pid;
-       env->procinfo->ppid = env->ppid;
-
-       /* for now, just copy the contents of every present page in the entire
-        * address space. */
-       if (env_user_mem_walk(e, 0, UMAPTOP, &copy_page, env)) {
+       /* Make the new process have the same VMRs as the older.  This will copy the
+        * contents of non MAP_SHARED pages to the new VMRs. */
+       if (duplicate_vmrs(e, env)) {
                proc_destroy(env);      /* this is prob what you want, not decref by 2 */
+               proc_decref(env);
                set_errno(ENOMEM);
                return -1;
        }
+       /* Switch to the new proc's address space and finish the syscall.  We'll
+        * never naturally finish this syscall for the new proc, since its memory
+        * is cloned before we return for the original process.  If we ever do CoW
+        * for forked memory, this will be the first place that gets CoW'd. */
+       temp = switch_to(env);
+       finish_current_sysc(0);
+       switch_back(env, temp);
+
+       /* In general, a forked process should be a fresh process, and we copy over
+        * whatever stuff is needed between procinfo/procdata. */
+       /* Copy over the procinfo argument stuff in case they don't exec */
+       memcpy(env->procinfo->argp, e->procinfo->argp, sizeof(e->procinfo->argp));
+       memcpy(env->procinfo->argbuf, e->procinfo->argbuf,
+              sizeof(e->procinfo->argbuf));
+       #ifdef CONFIG_X86
+       /* new guy needs to know about ldt (everything else in procdata is fresh */
+       env->procdata->ldt = e->procdata->ldt;
+       #endif
+
        clone_files(&e->open_files, &env->open_files);
+       /* FYI: once we call ready, the proc is open for concurrent usage */
        __proc_ready(env);
-       __proc_set_state(env, PROC_RUNNABLE_S);
-       schedule_proc(env);
+       proc_wakeup(env);
 
        // don't decref the new process.
        // that will happen when the parent waits for it.
        // TODO: if the parent doesn't wait, we need to change the child's parent
        // when the parent dies, or at least decref it
 
-       printd("[PID %d] fork PID %d\n",e->pid,env->pid);
+       printd("[PID %d] fork PID %d\n", e->pid, env->pid);
        return env->pid;
 }
 
@@ -488,7 +505,7 @@ static int sys_exec(struct proc *p, char *path, size_t path_l,
        char *t_path;
        struct file *program;
        struct per_cpu_info *pcpui = &per_cpu_info[core_id()];
-       struct trapframe *old_cur_tf = pcpui->cur_tf;
+       int8_t state = 0;
 
        /* We probably want it to never be allowed to exec if it ever was _M */
        if (p->state != PROC_RUNNING_S) {
@@ -499,21 +516,27 @@ static int sys_exec(struct proc *p, char *path, size_t path_l,
                set_errno(EINVAL);
                return -1;
        }
-       /* Can't exec if we don't have a current_tf to restart (if we fail).  This
-        * isn't 100% true, but I'm okay with it. */
-       if (!old_cur_tf) {
-               set_errno(EINVAL);
-               return -1;
-       }
        /* Copy in the path.  Consider putting an upper bound on path_l. */
        t_path = user_strdup_errno(p, path, path_l);
        if (!t_path)
                return -1;
-       /* Clear the current_tf.  We won't be returning the 'normal' way.  Even if
+       disable_irqsave(&state);        /* protect cur_ctx */
+       /* Can't exec if we don't have a current_ctx to restart (if we fail).  This
+        * isn't 100% true, but I'm okay with it. */
+       if (!pcpui->cur_ctx) {
+               enable_irqsave(&state);
+               set_errno(EINVAL);
+               return -1;
+       }
+       /* Preemptively copy out the cur_ctx, in case we fail later (easier on
+        * cur_ctx if we do this now) */
+       p->scp_ctx = *pcpui->cur_ctx;
+       /* Clear the current_ctx.  We won't be returning the 'normal' way.  Even if
         * we want to return with an error, we need to go back differently in case
         * we succeed.  This needs to be done before we could possibly block, but
         * unfortunately happens before the point of no return. */
-       pcpui->cur_tf = 0;
+       pcpui->cur_ctx = 0;
+       enable_irqsave(&state);
        /* This could block: */
        program = do_file_open(t_path, 0, 0);
        user_memdup_free(p, t_path);
@@ -527,21 +550,17 @@ static int sys_exec(struct proc *p, char *path, size_t path_l,
                                   sizeof(pi->argbuf)))
                goto mid_error;
        /* This is the point of no return for the process. */
-       /* TODO: issues with this: Need to also assert there are no outstanding
-        * users of the sysrings.  the ldt page will get freed shortly, so that's
-        * okay.  Potentially issues with the nm and vcpd if we were in _M before
-        * and someone is trying to notify. */
-       memset(p->procdata, 0, sizeof(procdata_t));
+       #ifdef CONFIG_X86
+       /* clear this, so the new program knows to get an LDT */
+       p->procdata->ldt = 0;
+       #endif
        destroy_vmrs(p);
        close_all_files(&p->open_files, TRUE);
        env_user_mem_free(p, 0, UMAPTOP);
        if (load_elf(p, program)) {
                kref_put(&program->f_kref);
-               /* Need an edible reference for proc_destroy in case it doesn't return.
-                * sys_exec was given current's ref (counted once just for current) */
-               proc_incref(p, 1);
+               /* Note this is an inedible reference, but proc_destroy now returns */
                proc_destroy(p);
-               proc_decref(p);
                /* We don't want to do anything else - we just need to not accidentally
                 * return to the user (hence the all_out) */
                goto all_out;
@@ -557,67 +576,182 @@ mid_error:
         * error value (errno is already set). */
        kref_put(&program->f_kref);
 early_error:
-       p->env_tf = *old_cur_tf;
-       signal_current_sc(-1);
+       finish_current_sysc(-1);
 success:
-       /* Here's how we'll restart the new (or old) process: */
+       /* Here's how we restart the new (on success) or old (on failure) proc: */
        spin_lock(&p->proc_lock);
-       __proc_set_state(p, PROC_RUNNABLE_S);
-       schedule_proc(p);
+       __unmap_vcore(p, 0);    /* VC# keep in sync with proc_run_s */
+       __proc_set_state(p, PROC_WAITING);      /* fake a yield */
        spin_unlock(&p->proc_lock);
+       proc_wakeup(p);
 all_out:
        /* we can't return, since we'd write retvals to the old location of the
-        * sycall struct (which has been freed and is in the old userspace) (or has
+        * syscall struct (which has been freed and is in the old userspace) (or has
         * already been written to).*/
+       disable_irq();                  /* abandon_core/clear_own wants irqs disabled */
+       clear_owning_proc(core_id());
        abandon_core();
-       smp_idle();
-       assert(0);
+       smp_idle();                             /* will reenable interrupts */
+}
+
+/* Helper, will attempt a particular wait on a proc.  Returns the pid of the
+ * process if we waited on it successfully, and the status will be passed back
+ * in ret_status (kernel memory).  Returns 0 if the wait failed and we should
+ * try again.  Returns -1 if we should abort.  Only handles DYING.  Callers
+ * need to lock to protect the children tailq and reaping bits. */
+static pid_t try_wait(struct proc *parent, struct proc *child, int *ret_status,
+                      int options)
+{
+       if (child->state == PROC_DYING) {
+               /* Disown returns -1 if it's already been disowned or we should o/w
+                * abort.  This can happen if we have concurrent waiters, both with
+                * pointers to the child (only one should reap).  Note that if we don't
+                * do this, we could go to sleep and never receive a cv_signal. */
+               if (__proc_disown_child(parent, child))
+                       return -1;
+               /* despite disowning, the child won't be freed til we drop this ref
+                * held by this function, so it is safe to access the memory.
+                *
+                * Note the exit code one byte in the 0xff00 spot.  Check out glibc's
+                * posix/sys/wait.h and bits/waitstatus.h for more info.  If we ever
+                * deal with signalling and stopping, we'll need to do some more work
+                * here.*/
+               *ret_status = (child->exitcode & 0xff) << 8;
+               return child->pid;
+       }
+       return 0;
 }
 
-static ssize_t sys_trywait(env_t* e, pid_t pid, int* status)
+/* Helper, like try_wait, but attempts a wait on any of the children, returning
+ * the specific PID we waited on, 0 to try again (a waitable exists), and -1 to
+ * abort (no children/waitables exist).  Callers need to lock to protect the
+ * children tailq and reaping bits.*/
+static pid_t try_wait_any(struct proc *parent, int *ret_status, int options)
 {
-       struct proc* p = pid2proc(pid);
-
-       // TODO: this syscall is racy, so we only support for single-core procs
-       if(e->state != PROC_RUNNING_S)
+       struct proc *i, *temp;
+       pid_t retval;
+       if (TAILQ_EMPTY(&parent->children))
                return -1;
+       /* Could have concurrent waiters mucking with the tailq, caller must lock */
+       TAILQ_FOREACH_SAFE(i, &parent->children, sibling_link, temp) {
+               retval = try_wait(parent, i, ret_status, options);
+               /* This catches a thread causing a wait to fail but not taking the
+                * child off the list before unlocking.  Should never happen. */
+               assert(retval != -1);
+               /* Succeeded, return the pid of the child we waited on */
+               if (retval)
+                       return retval;
+       }
+       assert(retval == 0);
+       return 0;
+}
 
-       // TODO: need to use errno properly.  sadly, ROS error codes conflict..
-
-       if(p)
-       {
-               ssize_t ret;
-
-               if(current->pid == p->ppid)
-               {
-                       if(p->state == PROC_DYING)
-                       {
-                               memcpy_to_user(e,status,&p->exitcode,sizeof(int));
-                               printd("[PID %d] waited for PID %d (code %d)\n",
-                                      e->pid,p->pid,p->exitcode);
-                               ret = 0;
-                       }
-                       else // not dead yet
-                       {
-                               set_errno(ESUCCESS);
-                               ret = -1;
-                       }
-               }
-               else // not a child of the calling process
-               {
-                       set_errno(EPERM);
-                       ret = -1;
-               }
+/* Waits on a particular child, returns the pid of the child waited on, and
+ * puts the ret status in *ret_status.  Returns the pid if we succeeded, 0 if
+ * the child was not waitable and WNOHANG, and -1 on error. */
+static pid_t wait_one(struct proc *parent, struct proc *child, int *ret_status,
+                      int options)
+{
+       pid_t retval;
+       cv_lock(&parent->child_wait);
+       /* retval == 0 means we should block */
+       retval = try_wait(parent, child, ret_status, options);
+       if ((retval == 0) && (options & WNOHANG))
+               goto out_unlock;
+       while (!retval) {
+               cpu_relax();
+               cv_wait(&parent->child_wait);
+               /* If we're dying, then we don't need to worry about waiting.  We don't
+                * do this yet, but we'll need this outlet when we deal with orphaned
+                * children and having init inherit them. */
+               if (parent->state == PROC_DYING)
+                       goto out_unlock;
+               /* Any child can wake us up, but we check for the particular child we
+                * care about */
+               retval = try_wait(parent, child, ret_status, options);
+       }
+       if (retval == -1) {
+               /* Child was already waited on by a concurrent syscall. */
+               set_errno(ECHILD);
+       }
+       /* Fallthrough */
+out_unlock:
+       cv_unlock(&parent->child_wait);
+       return retval;
+}
 
-               // if the wait succeeded, decref twice
-               if (ret == 0)
-                       proc_decref(p);
-               proc_decref(p);
-               return ret;
+/* Waits on any child, returns the pid of the child waited on, and puts the ret
+ * status in *ret_status.  Is basically a waitpid(-1, ... );  See wait_one for
+ * more details.  Returns -1 if there are no children to wait on, and returns 0
+ * if there are children and we need to block but WNOHANG was set. */
+static pid_t wait_any(struct proc *parent, int *ret_status, int options)
+{
+       pid_t retval;
+       cv_lock(&parent->child_wait);
+       retval = try_wait_any(parent, ret_status, options);
+       if ((retval == 0) && (options & WNOHANG))
+               goto out_unlock;
+       while (!retval) {
+               cpu_relax();
+               cv_wait(&parent->child_wait);
+               if (parent->state == PROC_DYING)
+                       goto out_unlock;
+               /* Any child can wake us up from the CV.  This is a linear try_wait
+                * scan.  If we have a lot of children, we could optimize this. */
+               retval = try_wait_any(parent, ret_status, options);
        }
+       if (retval == -1)
+               assert(TAILQ_EMPTY(&parent->children));
+       /* Fallthrough */
+out_unlock:
+       cv_unlock(&parent->child_wait);
+       return retval;
+}
 
-       set_errno(EPERM);
-       return -1;
+/* Note: we only allow waiting on children (no such thing as threads, for
+ * instance).  Right now we only allow waiting on termination (not signals),
+ * and we don't have a way for parents to disown their children (such as
+ * ignoring SIGCHLD, see man 2 waitpid's Notes).
+ *
+ * We don't bother with stop/start signals here, though we can probably build
+ * it in the helper above.
+ *
+ * Returns the pid of who we waited on, or -1 on error, or 0 if we couldn't
+ * wait (WNOHANG). */
+static pid_t sys_waitpid(struct proc *parent, pid_t pid, int *status,
+                         int options)
+{
+       struct proc *child;
+       pid_t retval = 0;
+       int ret_status = 0;
+
+       /* -1 is the signal for 'any child' */
+       if (pid == -1) {
+               retval = wait_any(parent, &ret_status, options);
+               goto out;
+       }
+       child = pid2proc(pid);
+       if (!child) {
+               set_errno(ECHILD);      /* ECHILD also used for no proc */
+               retval = -1;
+               goto out;
+       }
+       if (!(parent->pid == child->ppid)) {
+               set_errno(ECHILD);
+               retval = -1;
+               goto out_decref;
+       }
+       retval = wait_one(parent, child, &ret_status, options);
+       /* fall-through */
+out_decref:
+       proc_decref(child);
+out:
+       /* ignoring / don't care about memcpy's retval here. */
+       if (status)
+               memcpy_to_user(parent, status, &ret_status, sizeof(ret_status));
+       printd("[PID %d] waited for PID %d, got retval %d (status 0x%x)\n",
+              parent->pid, pid, retval, ret_status);
+       return retval;
 }
 
 /************** Memory Management Syscalls **************/
@@ -652,16 +786,40 @@ static int sys_shared_page_free(env_t* p1, void*DANGEROUS addr, pid_t p2)
        return -1;
 }
 
+/* Helper, to do the actual provisioning of a resource to a proc */
+static int prov_resource(struct proc *target, unsigned int res_type,
+                         long res_val)
+{
+       switch (res_type) {
+               case (RES_CORES):
+                       /* in the off chance we have a kernel scheduler that can't
+                        * provision, we'll need to change this. */
+                       return provision_core(target, res_val);
+               default:
+                       printk("[kernel] received provisioning for unknown resource %d\n",
+                              res_type);
+                       set_errno(ENOENT);      /* or EINVAL? */
+                       return -1;
+       }
+}
 
-static int sys_resource_req(struct proc *p, int type, unsigned int amt_wanted,
-                            unsigned int amt_wanted_min, int flags)
+/* Rough syscall to provision res_val of type res_type to target_pid */
+static int sys_provision(struct proc *p, int target_pid,
+                         unsigned int res_type, long res_val)
 {
+       struct proc *target = pid2proc(target_pid);
        int retval;
-       signal_current_sc(0);
-       /* this might not return (if it's a _S -> _M transition) */
-       proc_incref(p, 1);
-       retval = resource_req(p, type, amt_wanted, amt_wanted_min, flags);
-       proc_decref(p);
+       if (!target) {
+               if (target_pid == 0)
+                       return prov_resource(0, res_type, res_val);
+               /* debugging interface */
+               if (target_pid == -1)
+                       print_prov_map();
+               set_errno(ESRCH);
+               return -1;
+       }
+       retval = prov_resource(target, res_type, res_val);
+       proc_decref(target);
        return retval;
 }
 
@@ -673,7 +831,7 @@ static int sys_notify(struct proc *p, int target_pid, unsigned int ev_type,
        struct event_msg local_msg = {0};
        struct proc *target = pid2proc(target_pid);
        if (!target) {
-               set_errno(EBADPROC);
+               set_errno(ESRCH);
                return -1;
        }
        if (!proc_controls(p, target)) {
@@ -688,6 +846,8 @@ static int sys_notify(struct proc *p, int target_pid, unsigned int ev_type,
                        set_errno(EINVAL);
                        return -1;
                }
+       } else {
+               local_msg.ev_type = ev_type;
        }
        send_kernel_event(target, &local_msg, 0);
        proc_decref(target);
@@ -695,41 +855,118 @@ static int sys_notify(struct proc *p, int target_pid, unsigned int ev_type,
 }
 
 /* Will notify the calling process on the given vcore, independently of WANTED
- * or advertised vcoreid.  If you change the parameters, change pop_ros_tf() */
+ * or advertised vcoreid.  If you change the parameters, change pop_user_ctx().
+ */
 static int sys_self_notify(struct proc *p, uint32_t vcoreid,
-                           unsigned int ev_type, struct event_msg *u_msg)
+                           unsigned int ev_type, struct event_msg *u_msg,
+                           bool priv)
 {
        struct event_msg local_msg = {0};
-
-       printd("[kernel] received self notify for vcoreid %d, type %d, msg %08p\n",
-              vcoreid, ev_type, u_msg);
        /* if the user provided an ev_msg, copy it in and use that */
        if (u_msg) {
                if (memcpy_from_user(p, &local_msg, u_msg, sizeof(struct event_msg))) {
                        set_errno(EINVAL);
                        return -1;
                }
+       } else {
+               local_msg.ev_type = ev_type;
+       }
+       if (local_msg.ev_type >= MAX_NR_EVENT) {
+               printk("[kernel] received self-notify for vcoreid %d, ev_type %d, "
+                      "u_msg %p, u_msg->type %d\n", vcoreid, ev_type, u_msg,
+                      u_msg ? u_msg->ev_type : 0);
+               return -1;
        }
        /* this will post a message and IPI, regardless of wants/needs/debutantes.*/
-       post_vcore_event(p, &local_msg, vcoreid);
+       post_vcore_event(p, &local_msg, vcoreid, priv ? EVENT_VCORE_PRIVATE : 0);
        proc_notify(p, vcoreid);
        return 0;
 }
 
-/* This will set a local timer for usec, then shut down the core */
-static int sys_halt_core(struct proc *p, unsigned int usec)
+/* Puts the calling core into vcore context, if it wasn't already, via a
+ * self-IPI / active notification.  Barring any weird unmappings, we just send
+ * ourselves a __notify. */
+static int sys_vc_entry(struct proc *p)
 {
-       /* TODO: ought to check and see if a timer was already active, etc, esp so
-        * userspace can't turn off timers.  also note we will also call whatever
-        * timer_interrupt() will do, though all we care about is just
-        * self_ipi/interrupting. */
-       set_core_timer(usec);
-       cpu_halt();
-       set_core_timer(0);              /* Disable the timer (we don't have a 0-shot yet) */
+       send_kernel_message(core_id(), __notify, (long)p, 0, 0, KMSG_ROUTINE);
+       return 0;
+}
 
+/* This will set a local timer for usec, then shut down the core.  There's a
+ * slight race between spinner and halt.  For now, the core will wake up for
+ * other interrupts and service them, but will not process routine messages or
+ * do anything other than halt until the alarm goes off.  We could just unset
+ * the alarm and return early.  On hardware, there are a lot of interrupts that
+ * come in.  If we ever use this, we can take a closer look.  */
+static int sys_halt_core(struct proc *p, unsigned int usec)
+{
+       struct timer_chain *tchain = &per_cpu_info[core_id()].tchain;
+       struct alarm_waiter a_waiter;
+       bool spinner = TRUE;
+       void unblock(struct alarm_waiter *waiter)
+       {
+               spinner = FALSE;
+       }
+       init_awaiter(&a_waiter, unblock);
+       set_awaiter_rel(&a_waiter, MAX(usec, 100));
+       set_alarm(tchain, &a_waiter);
+       enable_irq();
+       /* Could wake up due to another interrupt, but we want to sleep still. */
+       while (spinner) {
+               cpu_halt();     /* slight race between spinner and halt */
+               cpu_relax();
+       }
+       printd("Returning from halting\n");
        return 0;
 }
 
+/* Changes a process into _M mode, or -EINVAL if it already is an mcp.
+ * __proc_change_to_m() returns and we'll eventually finish the sysc later.  The
+ * original context may restart on a remote core before we return and finish,
+ * but that's fine thanks to the async kernel interface. */
+static int sys_change_to_m(struct proc *p)
+{
+       int retval = proc_change_to_m(p);
+       /* convert the kernel error code into (-1, errno) */
+       if (retval) {
+               set_errno(-retval);
+               retval = -1;
+       }
+       return retval;
+}
+
+/* Pokes the ksched for the given resource for target_pid.  If the target pid
+ * == 0, we just poke for the calling process.  The common case is poking for
+ * self, so we avoid the lookup. 
+ *
+ * Not sure if you could harm someone via asking the kernel to look at them, so
+ * we'll do a 'controls' check for now.  In the future, we might have something
+ * in the ksched that limits or penalizes excessive pokes. */
+static int sys_poke_ksched(struct proc *p, int target_pid,
+                           unsigned int res_type)
+{
+       struct proc *target;
+       int retval = 0;
+       if (!target_pid) {
+               poke_ksched(p, res_type);
+               return 0;
+       }
+       target = pid2proc(target_pid);
+       if (!target) {
+               set_errno(ESRCH);
+               return -1;
+       }
+       if (!proc_controls(p, target)) {
+               set_errno(EPERM);
+               retval = -1;
+               goto out;
+       }
+       poke_ksched(target, res_type);
+out:
+       proc_decref(target);
+       return retval;
+}
+
 /************** Platform Specific Syscalls **************/
 
 //Read a buffer over the serial port
@@ -739,8 +976,8 @@ static ssize_t sys_serial_read(env_t* e, char *DANGEROUS _buf, size_t len)
        if (len == 0)
                return 0;
 
-       #ifdef __CONFIG_SERIAL_IO__
-           char *COUNT(len) buf = user_mem_assert(e, _buf, len, PTE_USER_RO);
+       #ifdef CONFIG_SERIAL_IO
+           char *COUNT(len) buf = user_mem_assert(e, _buf, len, 1, PTE_USER_RO);
                size_t bytes_read = 0;
                int c;
                while((c = serial_read_byte()) != -1) {
@@ -759,8 +996,8 @@ static ssize_t sys_serial_write(env_t* e, const char *DANGEROUS buf, size_t len)
        printk("[kernel] serial writing is deprecated.\n");
        if (len == 0)
                return 0;
-       #ifdef __CONFIG_SERIAL_IO__
-               char *COUNT(len) _buf = user_mem_assert(e, buf, len, PTE_USER_RO);
+       #ifdef CONFIG_SERIAL_IO
+               char *COUNT(len) _buf = user_mem_assert(e, buf, len, 1, PTE_USER_RO);
                for(int i =0; i<len; i++)
                        serial_send_byte(buf[i]);
                return (ssize_t)len;
@@ -769,7 +1006,7 @@ static ssize_t sys_serial_write(env_t* e, const char *DANGEROUS buf, size_t len)
        #endif
 }
 
-#ifdef __CONFIG_NETWORKING__
+#ifdef CONFIG_NETWORKING
 // This is not a syscall we want. Its hacky. Here just for syscall stuff until get a stack.
 static ssize_t sys_eth_read(env_t* e, char *DANGEROUS buf)
 {
@@ -793,7 +1030,7 @@ static ssize_t sys_eth_read(env_t* e, char *DANGEROUS buf)
 
                spin_unlock(&packet_buffers_lock);
 
-               char* _buf = user_mem_assert(e, buf, len, PTE_U);
+               char* _buf = user_mem_assert(e, buf, len, 1, PTE_U);
 
                memcpy(_buf, ptr, len);
 
@@ -975,7 +1212,6 @@ static intreg_t sys_fstat(struct proc *p, int fd, struct kstat *u_stat)
        /* TODO: UMEM: pin the memory, copy directly, and skip the kernel buffer */
        if (memcpy_to_user_errno(p, u_stat, kbuf, sizeof(struct kstat))) {
                kfree(kbuf);
-               set_errno(EINVAL);
                return -1;
        }
        kfree(kbuf);
@@ -1008,7 +1244,6 @@ static intreg_t stat_helper(struct proc *p, const char *path, size_t path_l,
        /* TODO: UMEM: pin the memory, copy directly, and skip the kernel buffer */
        if (memcpy_to_user_errno(p, u_stat, kbuf, sizeof(struct kstat))) {
                kfree(kbuf);
-               set_errno(EINVAL);
                return -1;
        }
        kfree(kbuf);
@@ -1106,17 +1341,30 @@ intreg_t sys_chmod(struct proc *p, const char *path, size_t path_l, int mode)
        return retval;
 }
 
-static intreg_t sys_lseek(struct proc *p, int fd, off_t offset, int whence)
+/* 64 bit seek, with the off64_t passed in via two (potentially 32 bit) off_ts.
+ * We're supporting both 32 and 64 bit kernels/userspaces, but both use the
+ * llseek syscall with 64 bit parameters. */
+static intreg_t sys_llseek(struct proc *p, int fd, off_t offset_hi,
+                           off_t offset_lo, off64_t *result, int whence)
 {
-       off_t ret;
+       off64_t retoff = 0;
+       off64_t tempoff = 0;
+       int ret = 0;
        struct file *file = get_file_from_fd(&p->open_files, fd);
        if (!file) {
                set_errno(EBADF);
                return -1;
        }
-       ret = file->f_op->llseek(file, offset, whence);
+       tempoff = offset_hi;
+       tempoff <<= 32;
+       tempoff |= offset_lo;
+       ret = file->f_op->llseek(file, tempoff, &retoff, whence);
        kref_put(&file->f_kref);
-       return ret;
+       if (ret)
+               return -1;
+       if (memcpy_to_user_errno(p, result, &retoff, sizeof(off64_t)))
+               return -1;
+       return 0;
 }
 
 intreg_t sys_link(struct proc *p, char *old_path, size_t old_l,
@@ -1160,7 +1408,7 @@ intreg_t sys_symlink(struct proc *p, char *old_path, size_t old_l,
                user_memdup_free(p, t_oldpath);
                return -1;
        }
-       ret = do_symlink(new_path, old_path, S_IRWXU | S_IRWXG | S_IRWXO);
+       ret = do_symlink(t_newpath, t_oldpath, S_IRWXU | S_IRWXG | S_IRWXO);
        user_memdup_free(p, t_oldpath);
        user_memdup_free(p, t_newpath);
        return ret;
@@ -1183,7 +1431,6 @@ intreg_t sys_readlink(struct proc *p, char *path, size_t path_l,
        copy_amt = strnlen(symname, buf_l - 1) + 1;
        if (memcpy_to_user_errno(p, u_buf, symname, copy_amt)) {
                kref_put(&path_d->d_kref);
-               set_errno(EINVAL);
                return -1;
        }
        kref_put(&path_d->d_kref);
@@ -1243,6 +1490,45 @@ intreg_t sys_rmdir(struct proc *p, const char *path, size_t path_l)
        return retval;
 }
 
+intreg_t sys_pipe(struct proc *p, int *u_pipefd, int flags)
+{
+       int pipefd[2] = {0};
+       int fd;
+       int retval = 0;
+       struct file *pipe_files[2] = {0};
+
+       if (do_pipe(pipe_files, flags))
+               return -1;
+       fd = insert_file(&p->open_files, pipe_files[0], 0);
+       if (!fd) {
+               set_errno(ENFILE);
+               goto failed_first;
+       }
+       pipefd[0] = fd;
+       fd = insert_file(&p->open_files, pipe_files[1], 0);
+       if (!fd) {
+               set_errno(ENFILE);
+               goto failed_second;
+       }
+       pipefd[1] = fd;
+       if (memcpy_to_user_errno(p, u_pipefd, pipefd, sizeof(pipefd))) {
+               set_errno(EFAULT);
+               goto failed_memcpy;
+       }
+       goto all_out;
+
+failed_memcpy:
+       put_file_from_fd(&p->open_files, pipefd[1]);
+failed_second:
+       put_file_from_fd(&p->open_files, pipefd[0]);
+failed_first:
+       retval = -1;
+all_out:
+       kref_put(&pipe_files[0]->f_kref);
+       kref_put(&pipe_files[1]->f_kref);
+       return retval;
+}
+
 intreg_t sys_gettimeofday(struct proc *p, int *buf)
 {
        static spinlock_t gtod_lock = SPINLOCK_INITIALIZER;
@@ -1251,7 +1537,7 @@ intreg_t sys_gettimeofday(struct proc *p, int *buf)
        spin_lock(&gtod_lock);
        if(t0 == 0)
 
-#if (defined __CONFIG_APPSERVER__)
+#if (defined CONFIG_APPSERVER)
        t0 = ufe(time,0,0,0,0);
 #else
        // Nanwan's birthday, bitches!!
@@ -1260,32 +1546,70 @@ intreg_t sys_gettimeofday(struct proc *p, int *buf)
        spin_unlock(&gtod_lock);
 
        long long dt = read_tsc();
-       int kbuf[2] = {t0+dt/system_timing.tsc_freq,
+       /* TODO: This probably wants its own function, using a struct timeval */
+       long kbuf[2] = {t0+dt/system_timing.tsc_freq,
            (dt%system_timing.tsc_freq)*1000000/system_timing.tsc_freq};
 
        return memcpy_to_user_errno(p,buf,kbuf,sizeof(kbuf));
 }
 
-#define SIZEOF_STRUCT_TERMIOS 60
 intreg_t sys_tcgetattr(struct proc *p, int fd, void *termios_p)
 {
-       int* kbuf = kmalloc(SIZEOF_STRUCT_TERMIOS,0);
-       int ret = ufe(tcgetattr,fd,PADDR(kbuf),0,0);
-       if(ret != -1 && memcpy_to_user_errno(p,termios_p,kbuf,SIZEOF_STRUCT_TERMIOS))
-               ret = -1;
+       int retval = 0;
+       /* TODO: actually support this call on tty FDs.  Right now, we just fake
+        * what my linux box reports for a bash pty. */
+       struct termios *kbuf = kmalloc(sizeof(struct termios), 0);
+       kbuf->c_iflag = 0x2d02;
+       kbuf->c_oflag = 0x0005;
+       kbuf->c_cflag = 0x04bf;
+       kbuf->c_lflag = 0x8a3b;
+       kbuf->c_line = 0x0;
+       kbuf->c_ispeed = 0xf;
+       kbuf->c_ospeed = 0xf;
+       kbuf->c_cc[0] = 0x03;
+       kbuf->c_cc[1] = 0x1c;
+       kbuf->c_cc[2] = 0x7f;
+       kbuf->c_cc[3] = 0x15;
+       kbuf->c_cc[4] = 0x04;
+       kbuf->c_cc[5] = 0x00;
+       kbuf->c_cc[6] = 0x01;
+       kbuf->c_cc[7] = 0xff;
+       kbuf->c_cc[8] = 0x11;
+       kbuf->c_cc[9] = 0x13;
+       kbuf->c_cc[10] = 0x1a;
+       kbuf->c_cc[11] = 0xff;
+       kbuf->c_cc[12] = 0x12;
+       kbuf->c_cc[13] = 0x0f;
+       kbuf->c_cc[14] = 0x17;
+       kbuf->c_cc[15] = 0x16;
+       kbuf->c_cc[16] = 0xff;
+       kbuf->c_cc[17] = 0x00;
+       kbuf->c_cc[18] = 0x00;
+       kbuf->c_cc[19] = 0x00;
+       kbuf->c_cc[20] = 0x00;
+       kbuf->c_cc[21] = 0x00;
+       kbuf->c_cc[22] = 0x00;
+       kbuf->c_cc[23] = 0x00;
+       kbuf->c_cc[24] = 0x00;
+       kbuf->c_cc[25] = 0x00;
+       kbuf->c_cc[26] = 0x00;
+       kbuf->c_cc[27] = 0x00;
+       kbuf->c_cc[28] = 0x00;
+       kbuf->c_cc[29] = 0x00;
+       kbuf->c_cc[30] = 0x00;
+       kbuf->c_cc[31] = 0x00;
+
+       if (memcpy_to_user_errno(p, termios_p, kbuf, sizeof(struct termios)))
+               retval = -1;
        kfree(kbuf);
-       return ret;
+       return retval;
 }
 
 intreg_t sys_tcsetattr(struct proc *p, int fd, int optional_actions,
                        const void *termios_p)
 {
-       void* kbuf = user_memdup_errno(p,termios_p,SIZEOF_STRUCT_TERMIOS);
-       if(kbuf == NULL)
-               return -1;
-       int ret = ufe(tcsetattr,fd,optional_actions,PADDR(kbuf),0);
-       user_memdup_free(p,kbuf);
-       return ret;
+       /* TODO: do this properly too.  For now, we just say 'it worked' */
+       return 0;
 }
 
 /* TODO: we don't have any notion of UIDs or GIDs yet, but don't let that stop a
@@ -1312,38 +1636,56 @@ const static struct sys_table_entry syscall_table[] = {
        [SYS_reboot] = {(syscall_t)reboot, "reboot!"},
        [SYS_cputs] = {(syscall_t)sys_cputs, "cputs"},
        [SYS_cgetc] = {(syscall_t)sys_cgetc, "cgetc"},
-       [SYS_getcpuid] = {(syscall_t)sys_getcpuid, "getcpuid"},
+       [SYS_getpcoreid] = {(syscall_t)sys_getpcoreid, "getpcoreid"},
        [SYS_getvcoreid] = {(syscall_t)sys_getvcoreid, "getvcoreid"},
        [SYS_getpid] = {(syscall_t)sys_getpid, "getpid"},
        [SYS_proc_create] = {(syscall_t)sys_proc_create, "proc_create"},
        [SYS_proc_run] = {(syscall_t)sys_proc_run, "proc_run"},
        [SYS_proc_destroy] = {(syscall_t)sys_proc_destroy, "proc_destroy"},
        [SYS_yield] = {(syscall_t)sys_proc_yield, "proc_yield"},
+       [SYS_change_vcore] = {(syscall_t)sys_change_vcore, "change_vcore"},
        [SYS_fork] = {(syscall_t)sys_fork, "fork"},
        [SYS_exec] = {(syscall_t)sys_exec, "exec"},
-       [SYS_trywait] = {(syscall_t)sys_trywait, "trywait"},
+       [SYS_waitpid] = {(syscall_t)sys_waitpid, "waitpid"},
        [SYS_mmap] = {(syscall_t)sys_mmap, "mmap"},
        [SYS_munmap] = {(syscall_t)sys_munmap, "munmap"},
        [SYS_mprotect] = {(syscall_t)sys_mprotect, "mprotect"},
        [SYS_shared_page_alloc] = {(syscall_t)sys_shared_page_alloc, "pa"},
        [SYS_shared_page_free] = {(syscall_t)sys_shared_page_free, "pf"},
-       [SYS_resource_req] = {(syscall_t)sys_resource_req, "resource_req"},
+       [SYS_provision] = {(syscall_t)sys_provision, "provision"},
        [SYS_notify] = {(syscall_t)sys_notify, "notify"},
        [SYS_self_notify] = {(syscall_t)sys_self_notify, "self_notify"},
+       [SYS_vc_entry] = {(syscall_t)sys_vc_entry, "vc_entry"},
        [SYS_halt_core] = {(syscall_t)sys_halt_core, "halt_core"},
-#ifdef __CONFIG_SERIAL_IO__
+#ifdef CONFIG_SERIAL_IO
        [SYS_serial_read] = {(syscall_t)sys_serial_read, "ser_read"},
        [SYS_serial_write] = {(syscall_t)sys_serial_write, "ser_write"},
 #endif
-#ifdef __CONFIG_NETWORKING__
+#ifdef CONFIG_NETWORKING
        [SYS_eth_read] = {(syscall_t)sys_eth_read, "eth_read"},
        [SYS_eth_write] = {(syscall_t)sys_eth_write, "eth_write"},
        [SYS_eth_get_mac_addr] = {(syscall_t)sys_eth_get_mac_addr, "get_mac"},
        [SYS_eth_recv_check] = {(syscall_t)sys_eth_recv_check, "recv_check"},
 #endif
-#ifdef __CONFIG_ARSC_SERVER__
+#ifdef CONFIG_ARSC_SERVER
        [SYS_init_arsc] = {(syscall_t)sys_init_arsc, "init_arsc"},
 #endif
+       [SYS_change_to_m] = {(syscall_t)sys_change_to_m, "change_to_m"},
+       [SYS_poke_ksched] = {(syscall_t)sys_poke_ksched, "poke_ksched"},
+
+// socket related syscalls
+       [SYS_socket] ={(syscall_t)sys_socket, "socket"},
+       [SYS_sendto] ={(syscall_t)sys_sendto, "sendto"},
+       [SYS_recvfrom] ={(syscall_t)sys_recvfrom, "recvfrom"},
+       [SYS_select] ={(syscall_t)sys_select, "select"},
+       [SYS_connect] = {(syscall_t)sys_connect, "connect"},
+       [SYS_send] ={(syscall_t)sys_send, "send"},
+       [SYS_recv] ={(syscall_t)sys_recv, "recvfrom"},
+       [SYS_bind] ={(syscall_t)sys_bind, "bind"},
+       [SYS_accept] ={(syscall_t)sys_accept, "accept"},
+       [SYS_listen] ={(syscall_t)sys_listen, "listen"},
+
+
        [SYS_read] = {(syscall_t)sys_read, "read"},
        [SYS_write] = {(syscall_t)sys_write, "write"},
        [SYS_open] = {(syscall_t)sys_open, "open"},
@@ -1355,7 +1697,7 @@ const static struct sys_table_entry syscall_table[] = {
        [SYS_access] = {(syscall_t)sys_access, "access"},
        [SYS_umask] = {(syscall_t)sys_umask, "umask"},
        [SYS_chmod] = {(syscall_t)sys_chmod, "chmod"},
-       [SYS_lseek] = {(syscall_t)sys_lseek, "lseek"},
+       [SYS_llseek] = {(syscall_t)sys_llseek, "llseek"},
        [SYS_link] = {(syscall_t)sys_link, "link"},
        [SYS_unlink] = {(syscall_t)sys_unlink, "unlink"},
        [SYS_symlink] = {(syscall_t)sys_symlink, "symlink"},
@@ -1364,6 +1706,7 @@ const static struct sys_table_entry syscall_table[] = {
        [SYS_getcwd] = {(syscall_t)sys_getcwd, "getcwd"},
        [SYS_mkdir] = {(syscall_t)sys_mkdir, "mkdri"},
        [SYS_rmdir] = {(syscall_t)sys_rmdir, "rmdir"},
+       [SYS_pipe] = {(syscall_t)sys_pipe, "pipe"},
        [SYS_gettimeofday] = {(syscall_t)sys_gettimeofday, "gettime"},
        [SYS_tcgetattr] = {(syscall_t)sys_tcgetattr, "tcgetattr"},
        [SYS_tcsetattr] = {(syscall_t)sys_tcsetattr, "tcsetattr"},
@@ -1388,19 +1731,19 @@ intreg_t syscall(struct proc *p, uintreg_t sc_num, uintreg_t a0, uintreg_t a1,
        if (systrace_flags & SYSTRACE_ON) {
                if ((systrace_flags & SYSTRACE_ALLPROC) || (proc_is_traced(p))) {
                        coreid = core_id();
-                       vcoreid = proc_get_vcoreid(p, coreid);
+                       vcoreid = proc_get_vcoreid(p);
                        if (systrace_flags & SYSTRACE_LOUD) {
-                               printk("[%16llu] Syscall %3d (%12s):(%08p, %08p, %08p, %08p, "
-                                      "%08p, %08p) proc: %d core: %d vcore: %d\n", read_tsc(),
+                               printk("[%16llu] Syscall %3d (%12s):(%p, %p, %p, %p, "
+                                      "%p, %p) proc: %d core: %d vcore: %d\n", read_tsc(),
                                       sc_num, syscall_table[sc_num].name, a0, a1, a2, a3,
                                       a4, a5, p->pid, coreid, vcoreid);
                        } else {
                                struct systrace_record *trace;
-                               unsigned int idx, new_idx;
+                               uintptr_t idx, new_idx;
                                do {
                                        idx = systrace_bufidx;
                                        new_idx = (idx + 1) % systrace_bufsize;
-                               } while (!atomic_comp_swap(&systrace_bufidx, idx, new_idx));
+                               } while (!atomic_cas_u32(&systrace_bufidx, idx, new_idx));
                                trace = &systrace_buffer[idx];
                                trace->timestamp = read_tsc();
                                trace->syscallno = sc_num;
@@ -1417,24 +1760,34 @@ intreg_t syscall(struct proc *p, uintreg_t sc_num, uintreg_t a0, uintreg_t a1,
                }
        }
        if (sc_num > max_syscall || syscall_table[sc_num].call == NULL)
-               panic("Invalid syscall number %d for proc %x!", sc_num, *p);
+               panic("Invalid syscall number %d for proc %x!", sc_num, p);
 
        return syscall_table[sc_num].call(p, a0, a1, a2, a3, a4, a5);
 }
 
 /* Execute the syscall on the local core */
-static void run_local_syscall(struct syscall *sysc)
+void run_local_syscall(struct syscall *sysc)
 {
        struct per_cpu_info *pcpui = &per_cpu_info[core_id()];
 
        /* TODO: (UMEM) assert / pin the memory for the sysc */
-       user_mem_assert(pcpui->cur_proc, sysc, sizeof(struct syscall), PTE_USER_RW);
+       assert(irq_is_enabled());       /* in case we proc destroy */
+       /* Abort on mem check failure, for now */
+       if (!user_mem_check(pcpui->cur_proc, sysc, sizeof(struct syscall),
+                           sizeof(uintptr_t), PTE_USER_RW))
+               return;
        pcpui->cur_sysc = sysc;                 /* let the core know which sysc it is */
        sysc->retval = syscall(pcpui->cur_proc, sysc->num, sysc->arg0, sysc->arg1,
                               sysc->arg2, sysc->arg3, sysc->arg4, sysc->arg5);
-       sysc->flags |= SC_DONE;
-       signal_syscall(sysc, pcpui->cur_proc);
-       /* Can unpin at this point */
+       /* Need to re-load pcpui, in case we migrated */
+       pcpui = &per_cpu_info[core_id()];
+       /* Some 9ns paths set errstr, but not errno.  glibc will ignore errstr.
+        * this is somewhat hacky, since errno might get set unnecessarily */
+       if ((current_errstr()[0] != 0) && (!sysc->err))
+               sysc->err = EUNSPECIFIED;
+       finish_sysc(sysc, pcpui->cur_proc);
+       /* Can unpin (UMEM) at this point */
+       pcpui->cur_sysc = 0;    /* no longer working on sysc */
 }
 
 /* A process can trap and call this function, which will set up the core to
@@ -1443,7 +1796,7 @@ static void run_local_syscall(struct syscall *sysc)
 void prep_syscalls(struct proc *p, struct syscall *sysc, unsigned int nr_syscs)
 {
        int retval;
-       struct per_cpu_info *pcpui = &per_cpu_info[core_id()];
+       /* Careful with pcpui here, we could have migrated */
        if (!nr_syscs)
                return;
        /* For all after the first call, send ourselves a KMSG (TODO). */
@@ -1456,17 +1809,23 @@ void prep_syscalls(struct proc *p, struct syscall *sysc, unsigned int nr_syscs)
 
 /* Call this when something happens on the syscall where userspace might want to
  * get signaled.  Passing p, since the caller should know who the syscall
- * belongs to (probably is current). */
-void signal_syscall(struct syscall *sysc, struct proc *p)
+ * belongs to (probably is current). 
+ *
+ * You need to have SC_K_LOCK set when you call this. */
+void __signal_syscall(struct syscall *sysc, struct proc *p)
 {
        struct event_queue *ev_q;
        struct event_msg local_msg;
-       ev_q = sysc->ev_q;
-       if (ev_q) {
-               memset(&local_msg, 0, sizeof(struct event_msg));
-               local_msg.ev_type = EV_SYSCALL;
-               local_msg.ev_arg3 = sysc;
-               send_event(p, ev_q, &local_msg, 0);
+       /* User sets the ev_q then atomically sets the flag (races with SC_DONE) */
+       if (atomic_read(&sysc->flags) & SC_UEVENT) {
+               rmb();  /* read the ev_q after reading the flag */
+               ev_q = sysc->ev_q;
+               if (ev_q) {
+                       memset(&local_msg, 0, sizeof(struct event_msg));
+                       local_msg.ev_type = EV_SYSCALL;
+                       local_msg.ev_arg3 = sysc;
+                       send_event(p, ev_q, &local_msg, 0);
+               }
        }
 }
 
@@ -1554,8 +1913,8 @@ void systrace_print(bool all, struct proc *p)
         * timestamp and loop around.  Careful of concurrent writes. */
        for (int i = 0; i < systrace_bufsize; i++)
                if (systrace_buffer[i].timestamp)
-                       printk("[%16llu] Syscall %3d (%12s):(%08p, %08p, %08p, %08p, %08p,"
-                              "%08p) proc: %d core: %d vcore: %d\n",
+                       printk("[%16llu] Syscall %3d (%12s):(%p, %p, %p, %p, %p,"
+                              "%p) proc: %d core: %d vcore: %d\n",
                               systrace_buffer[i].timestamp,
                               systrace_buffer[i].syscallno,
                               syscall_table[systrace_buffer[i].syscallno].name,