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 864c7da58af3df7f09a292feee299df532474e43..9f86cf25c480355f836c2568695a08478ca7af52 100644 (file)
--- a/lunaix-os/kernel/asm/x86/interrupts.c
+++ b/lunaix-os/kernel/asm/x86/interrupts.c
@@ -1,7 +1,10 @@
 #include <arch/x86/interrupts.h>
 #include <arch/x86/tss.h>
+
 #include <hal/apic.h>
 #include <hal/cpu.h>
+
+#include <lunaix/isrm.h>
 #include <lunaix/mm/page.h>
 #include <lunaix/mm/vmm.h>
 #include <lunaix/process.h>
@@ -9,65 +12,34 @@
 #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)
 {
-    __current->intr_ctx = *param;
+    param->execp->saved_prev_ctx = __current->intr_ctx;
+    __current->intr_ctx = param;
 
-    isr_param* lparam = &__current->intr_ctx;
-
-    if (lparam->vector <= 255) {
-        int_subscriber subscriber = subscribers[lparam->vector];
-        if (subscriber) {
-            subscriber(param);
-            goto done;
-        }
-    }
+    volatile struct exec_param* execp = __current->intr_ctx->execp;
 
-    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:
     // 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();
     }