fix: corner case for x87 context restore on execve

[lunaix-os.git] / lunaix-os / kernel / asm / x86 / interrupts.c

diff --git a/lunaix-os/kernel/asm/x86/interrupts.c b/lunaix-os/kernel/asm/x86/interrupts.c
index 09bd1eb1409cfdcf18a78a64784739112f30107e..9f86cf25c480355f836c2568695a08478ca7af52 100644 (file)
--- a/lunaix-os/kernel/asm/x86/interrupts.c
+++ b/lunaix-os/kernel/asm/x86/interrupts.c
@@ -1,87 +1,47 @@
 #include <arch/x86/interrupts.h>
+#include <arch/x86/tss.h>
+
 #include <hal/apic.h>
 #include <hal/cpu.h>
-#include <lunaix/syslog.h>
-#include <lunaix/tty/tty.h>
-#include <lunaix/process.h>
-#include <lunaix/sched.h>
+
+#include <lunaix/isrm.h>
 #include <lunaix/mm/page.h>
 #include <lunaix/mm/vmm.h>
+#include <lunaix/process.h>
+#include <lunaix/sched.h>
+#include <lunaix/syslog.h>
+#include <lunaix/tty/tty.h>
 
-LOG_MODULE("intr")
-
-static int_subscriber subscribers[256];
-
-static int_subscriber fallback = (int_subscriber) 0;
-
-void
-intr_subscribe(const uint8_t vector, int_subscriber subscriber) {
-    subscribers[vector] = subscriber;
-}
-
-void
-intr_unsubscribe(const uint8_t vector, int_subscriber subscriber) {
-    if (subscribers[vector] == subscriber) {
-        subscribers[vector] = (int_subscriber) 0;
-    }
-}
-
-void
-intr_set_fallback_handler(int_subscriber subscribers) {
-    fallback = subscribers;
-}
+LOG_MODULE("INTR")
 
 extern x86_page_table* __kernel_ptd;
 
 void
 intr_handler(isr_param* param)
 {
-    // if (param->vector == LUNAIX_SYS_CALL) {
-    //     kprintf(KDEBUG "%p", param->registers.esp);
-    // }
-    __current->intr_ctx = *param;
-
-    cpu_lcr3(__kernel_ptd);
+    param->execp->saved_prev_ctx = __current->intr_ctx;
+    __current->intr_ctx = param;
 
-    // 将当前进程的页目录挂载到内核地址空间里（页目录挂载点#1），方便访问。
-    vmm_mount_pd(PD_MOUNT_1, __current->page_table);
+    volatile struct exec_param* execp = __current->intr_ctx->execp;
 
-    isr_param *lparam = &__current->intr_ctx;
-    
-    if (lparam->vector <= 255) {
-        int_subscriber subscriber = subscribers[lparam->vector];
-        if (subscriber) {
-            subscriber(lparam);
-            goto done;
-        }
-    }
-
-    if (fallback) {
-        fallback(lparam);
+    if (execp->vector <= 255) {
+        isr_cb subscriber = isrm_get(execp->vector);
+        subscriber(param);
         goto done;
     }
-    
+
     kprint_panic("INT %u: (%x) [%p: %p] Unknown",
-            lparam->vector,
-            lparam->err_code,
-            lparam->cs,
-            lparam->eip);
+                 execp->vector,
+                 execp->err_code,
+                 execp->cs,
+                 execp->eip);
 
 done:
-
-    // if (__current->state != PROC_RUNNING) {
-    //     schedule();
-    // }
-
     // for all external interrupts except the spurious interrupt
     //  this is required by Intel Manual Vol.3A, section 10.8.1 & 10.8.5
-    if (lparam->vector >= EX_INTERRUPT_BEGIN && lparam->vector != APIC_SPIV_IV) {
+    if (execp->vector >= IV_EX && execp->vector != APIC_SPIV_IV) {
         apic_done_servicing();
     }
 
-    cpu_lcr3(__current->page_table);
-
-    *param = __current->intr_ctx;
-
     return;
 }
\ No newline at end of file