Removed set_current_proc()
authorBarret Rhoden <brho@cs.berkeley.edu>
Fri, 19 Nov 2010 20:21:14 +0000 (12:21 -0800)
committerKevin Klues <klueska@cs.berkeley.edu>
Thu, 3 Nov 2011 00:35:57 +0000 (17:35 -0700)
It was just confusing (what with __set_proc_current(), which actually
did something).  Also finally moved those macros to smp.h.

kern/arch/i686/process.c
kern/arch/sparc/process.c
kern/include/process.h
kern/include/smp.h

index 86ae4d6..3b82308 100644 (file)
@@ -56,8 +56,9 @@ void proc_secure_trapframe(struct trapframe *tf)
  * unmapping the address space!) */
 void __abandon_core(void)
 {
+       struct per_cpu_info *pcpui = &per_cpu_info[core_id()];
        asm volatile ("movw %%ax,%%gs; lldt %%ax" :: "a"(0));
        lcr3(boot_cr3);
-       kref_put(&current->kref);
-       set_current_proc(NULL);
+       kref_put(&pcpui->cur_proc->kref);
+       pcpui->cur_proc = 0;
 }
index 65dc46d..e7ccceb 100644 (file)
@@ -40,7 +40,8 @@ void proc_secure_trapframe(struct trapframe *tf)
  * protects the cr3). */
 void __abandon_core(void)
 {
+       struct per_cpu_info *pcpui = &per_cpu_info[core_id()];
        lcr3(boot_cr3);
-       kref_put(&current->kref);
-       set_current_proc(NULL);
+       kref_put(&pcpui->cur_proc->kref);
+       pcpui->cur_proc = 0;
 }
index 36042ff..f193217 100644 (file)
@@ -128,20 +128,6 @@ bool __proc_preempt_all(struct proc *p);
 void proc_preempt_core(struct proc *p, uint32_t pcoreid, uint64_t usec);
 void proc_preempt_all(struct proc *p, uint64_t usec);
 
-/* Allows the kernel to figure out what process is running on this core.  Can be
- * used just like a pointer to a struct proc.  Need these to be macros due to
- * some circular dependencies with smp.h. */
-#define current per_cpu_info[core_id()].cur_proc
-#define set_current_proc(p) per_cpu_info[core_id()].cur_proc = (p)
-
-/* Allows the kernel to figure out what *user* tf is on this core's stack.  Can
- * be used just like a pointer to a struct Trapframe.  Need these to be macros
- * due to some circular dependencies with smp.h.  This is done here instead of
- * elsewhere (like trap.h) for other elliptical reasons.  Note the distinction
- * between kernel and user contexts.  The kernel always returns to its nested,
- * interrupted contexts via iret/etc.  We never do that for user contexts. */
-#define current_tf per_cpu_info[core_id()].cur_tf
-
 void abandon_core(void);
 /* Hold the proc_lock, since it'll use the vcoremapping to send an unmapping
  * message for the region from start to end.  */
index 1c36123..bc0bb9a 100644 (file)
@@ -28,8 +28,8 @@ struct per_cpu_info {
        struct proc *cur_proc;
        struct trapframe *cur_tf;       /* user tf we came in on (can be 0) */
        struct trapframe actual_tf;     /* storage for cur_tf */
-       struct kthread *spare;          /* useful when restarting */
        struct syscall *cur_sysc;       /* ptr is into cur_proc's address space */
+       struct kthread *spare;          /* useful when restarting */
 
 #ifdef __SHARC__
        // held spin-locks. this will have to go elsewhere if multiple kernel
@@ -48,6 +48,15 @@ struct per_cpu_info {
        struct kernel_msg_list NTPTV(a0t) NTPTV(a1t) NTPTV(a2t) routine_amsgs;
 }__attribute__((aligned(HW_CACHE_ALIGN)));
 
+/* Allows the kernel to figure out what process is running on this core.  Can be
+ * used just like a pointer to a struct proc. */
+#define current per_cpu_info[core_id()].cur_proc
+/* Allows the kernel to figure out what *user* tf is on this core's stack.  Can
+ * be used just like a pointer to a struct Trapframe.  Note the distinction
+ * between kernel and user contexts.  The kernel always returns to its nested,
+ * interrupted contexts via iret/etc.  We never do that for user contexts. */
+#define current_tf per_cpu_info[core_id()].cur_tf
+
 typedef struct per_cpu_info NTPTV(t) NTPTV(a0t) NTPTV(a1t) NTPTV(a2t) per_cpu_info_t;
 
 extern per_cpu_info_t (RO per_cpu_info)[MAX_NUM_CPUS];