refactor: simplify the vmm design, single responsibility. But using it should with...

[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 f6b55389ae68d8e43fd35db574f93301cdfb7528..a59e5f29a3a083266430976929a86bbab3db250e 100644 (file)
--- a/lunaix-os/kernel/asm/x86/interrupts.c
+++ b/lunaix-os/kernel/asm/x86/interrupts.c
@@ -1,45 +1,56 @@
 #include <arch/x86/interrupts.h>
 #include <hal/apic.h>
 #include <hal/cpu.h>
-#include <lunaix/syslog.h>
-#include <lunaix/tty/tty.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;
+static int_subscriber fallback = (int_subscriber)0;
 
 void
-intr_subscribe(const uint8_t vector, int_subscriber subscriber) {
+intr_subscribe(const uint8_t vector, int_subscriber subscriber)
+{
     subscribers[vector] = subscriber;
 }
 
 void
-intr_unsubscribe(const uint8_t vector, int_subscriber subscriber) {
+intr_unsubscribe(const uint8_t vector, int_subscriber subscriber)
+{
     if (subscribers[vector] == subscriber) {
-        subscribers[vector] = (int_subscriber) 0;
+        subscribers[vector] = (int_subscriber)0;
     }
 }
 
 void
-intr_set_fallback_handler(int_subscriber subscribers) {
+intr_set_fallback_handler(int_subscriber subscribers)
+{
     fallback = subscribers;
 }
 
+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;
-    
-    if (param->vector <= 255) {
-        int_subscriber subscriber = subscribers[param->vector];
+
+#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;
@@ -47,29 +58,26 @@ intr_handler(isr_param* param)
     }
 
     if (fallback) {
-        fallback(param);
+        fallback(lparam);
         goto done;
     }
-    
+
     kprint_panic("INT %u: (%x) [%p: %p] Unknown",
-            param->vector,
-            param->err_code,
-            param->cs,
-            param->eip);
+                 lparam->vector,
+                 lparam->err_code,
+                 lparam->cs,
+                 lparam->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 (param->vector >= EX_INTERRUPT_BEGIN && param->vector != APIC_SPIV_IV) {
+    if (lparam->vector >= EX_INTERRUPT_BEGIN &&
+        lparam->vector != APIC_SPIV_IV) {
         apic_done_servicing();
     }
 
-    *param = __current->intr_ctx;
-
+#ifdef USE_KERNEL_PT
+    cpu_lcr3(__current->page_table);
+#endif
     return;
 }
\ No newline at end of file