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 a59e5f29a3a083266430976929a86bbab3db250e..9f86cf25c480355f836c2568695a08478ca7af52 100644 (file)
--- a/lunaix-os/kernel/asm/x86/interrupts.c
+++ b/lunaix-os/kernel/asm/x86/interrupts.c
@@ -1,6 +1,10 @@
 #include <arch/x86/interrupts.h>
 #include <arch/x86/interrupts.h>

+#include <arch/x86/tss.h>
+

 #include <hal/apic.h>
 #include <hal/cpu.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>
 #include <lunaix/mm/page.h>
 #include <lunaix/mm/vmm.h>
 #include <lunaix/process.h>


@@ -8,76 +12,36 @@
 #include <lunaix/syslog.h>
 #include <lunaix/tty/tty.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)
 {
 
 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;

 
 

-#ifdef USE_KERNEL_PT
-    cpu_lcr3(__kernel_ptd);
-
-    vmm_mount_pd(PD_MOUNT_1, __current->page_table);
-#endif
-
-    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",
         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
 
 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();
     }
 
         apic_done_servicing();
     }
 

-#ifdef USE_KERNEL_PT
-    cpu_lcr3(__current->page_table);
-#endif

     return;
 }
\ No newline at end of file
     return;
 }
\ No newline at end of file