Changeset 335 for trunk/hal/x86_64/core/hal_context.c

Timestamp:

Aug 7, 2017, 11:19:27 AM (6 years ago)

Author:

max@…

Message:

Separate the CPU context from the trap frame.

File:

• trunk/hal/x86_64/core/hal_context.c (modified) (5 diffs)

trunk/hal/x86_64/core/hal_context.c

@@ -37 +37 @@
 error_t hal_cpu_context_create(struct thread_s *thread)
 {
-        hal_cpu_context_t *context;
+        hal_trapframe_t *tf;
+        hal_cpu_context_t *ctx;
         kmem_req_t req;
 
@@ -45 +46 @@
         req.flags = AF_KERNEL | AF_ZERO;
 
-        context = (hal_cpu_context_t *)kmem_alloc(&req);
-        if (context == NULL)
+        ctx = (hal_cpu_context_t *)kmem_alloc(&req);
+        if (ctx == NULL)
                 return ENOMEM;
 
         /* set cpu context pointer in thread */
-        thread->cpu_context = (void *)context;
+        thread->cpu_context = (void *)ctx;
 
-        /* build the context */
-        /* tf_gs */
-        /* tf_fs */
-        context->tf_es = GDT_FIXED_SEL(GDT_UDATA_SEL, SEL_UPL);
-        context->tf_ds = GDT_FIXED_SEL(GDT_UDATA_SEL, SEL_UPL);
-        context->tf_rip = (uint64_t)thread->entry_func;
-        context->tf_rflags = PSL_USERSET;
+        /*
+         * Build the context
+         */
+        ctx->ctx_rsp0 = ((uint64_t)thread->k_stack_base) + thread->k_stack_size;
+        ctx->ctx_rsp0 &= ~0xF;
+        /* XXX: ctx_rsp */
+
+        /*
+         * Build the trap frame
+         */
+        tf = (void *)(((uint64_t)thread->k_stack_base +
+            thread->k_stack_size) & ~0xF);
+        tf->tf_gs = GDT_FIXED_SEL(GDT_UDATA_SEL, SEL_UPL);
+        tf->tf_fs = GDT_FIXED_SEL(GDT_UDATA_SEL, SEL_UPL);
+        tf->tf_es = GDT_FIXED_SEL(GDT_UDATA_SEL, SEL_UPL);
+        tf->tf_ds = GDT_FIXED_SEL(GDT_UDATA_SEL, SEL_UPL);
+        tf->tf_rip = (uint64_t)thread->entry_func;
+        tf->tf_rflags = PSL_USERSET;
 
         if (thread->type == THREAD_USER) {
-                context->tf_cs = GDT_FIXED_SEL(GDT_UCODE_SEL, SEL_UPL);
-                context->tf_rsp = ((uint64_t)thread->u_stack_base) +
+                tf->tf_cs = GDT_FIXED_SEL(GDT_UCODE_SEL, SEL_UPL);
+                tf->tf_rsp = ((uint64_t)thread->u_stack_base) +
                     thread->u_stack_size;
         } else {
-                context->tf_cs = GDT_FIXED_SEL(GDT_KCODE_SEL, SEL_KPL);
-                context->tf_rsp = ((uint64_t)thread->k_stack_base) +
+                tf->tf_cs = GDT_FIXED_SEL(GDT_KCODE_SEL, SEL_KPL);
+                tf->tf_rsp = ((uint64_t)thread->k_stack_base) +
                     thread->k_stack_size;
         }
@@ -73 +85 @@
 }
 
-error_t hal_cpu_context_copy( thread_t * dst,
-                              thread_t * src )
+error_t hal_cpu_context_copy(thread_t *dst, thread_t *src)
 {
         x86_panic((char *)__func__);
@@ -80 +91 @@
 }
 
-void hal_cpu_context_destroy( thread_t * thread )
+void hal_cpu_context_destroy(thread_t *thread)
+{
+        x86_panic((char *)__func__);
+}
+
+void hal_cpu_context_switch(thread_t *old, thread_t *new)
+{
+        x86_panic((char *)__func__);
+}
+
+void hal_cpu_context_load( thread_t * thread )
 {
         x86_panic((char *)__func__);
@@ -103 +124 @@
 }
 
-void hal_cpu_context_switch( thread_t * old , thread_t * new )
-{
-        x86_panic((char *)__func__);
-}
-
-void hal_cpu_context_load( thread_t * thread )
-{
-        x86_panic((char *)__func__);
-}
-
 void hal_fpu_context_save( thread_t * thread )
 {