Fixes memory leak in dev stdout
[akaros.git] / kern / src / process.c
index cda5b5d..5404f92 100644 (file)
@@ -32,7 +32,6 @@
 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);
@@ -194,7 +193,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));
@@ -270,11 +269,12 @@ 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 */
+       p->heap_top = 0;
        spinlock_init(&p->mm_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. */
+       /* 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);
@@ -310,6 +310,10 @@ error_t proc_alloc(struct proc **pp, struct proc *parent)
 
        atomic_inc(&num_envs);
        frontend_proc_init(p);
+       //plan9setup(p, parent);
+       //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;
@@ -359,7 +363,9 @@ static void __proc_free(struct kref *kref)
        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));
 
+       /* close plan9 dot and slash and free fgrp fd and fgrp */
        kref_put(&p->fs_env.root->d_kref);
        kref_put(&p->fs_env.pwd->d_kref);
        destroy_vmrs(p);
@@ -392,11 +398,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
@@ -499,7 +510,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 +615,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,9 +630,13 @@ 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... */
@@ -632,22 +649,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 +764,17 @@ 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) */
+       //close_9ns_files(p);
        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);
@@ -1095,6 +1113,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
@@ -1623,7 +1643,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();
 }
@@ -1755,7 +1776,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;
        }
@@ -1985,7 +2006,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 */
 }
 
@@ -2100,7 +2121,7 @@ void print_proc_info(pid_t pid)
        printk("State: %s (%p)\n", procstate2str(p->state), p->state);
        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");
@@ -2121,14 +2142,14 @@ void print_proc_info(pid_t pid)
        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,
+                       printk("\tFD: %02d, File: %p, File name: %s\n", i,
                               files->fd_array[i].fd_file,
                               file_name(files->fd_array[i].fd_file));
                }
        spin_unlock(&files->lock);
        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 +2174,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);