From 29ff56335c4fae316356d1d352eab0fc8795a59e Mon Sep 17 00:00:00 2001
From: Ian Moffett <ian@osmora.org>
Date: Thu, 23 May 2024 23:49:43 -0400
Subject: kernel/amd64: Log messages during user faults

Signed-off-by: Ian Moffett <ian@osmora.org>
---
 sys/arch/amd64/amd64/trap.c | 41 +++++++++++++++++++++++++++++++++++++----
 1 file changed, 37 insertions(+), 4 deletions(-)

(limited to 'sys/arch/amd64')

diff --git a/sys/arch/amd64/amd64/trap.c b/sys/arch/amd64/amd64/trap.c
index 414b86d..bf8e3b5 100644
--- a/sys/arch/amd64/amd64/trap.c
+++ b/sys/arch/amd64/amd64/trap.c
@@ -40,6 +40,10 @@
 #include <sys/machdep.h>
 #include <vm/fault.h>
 
+__MODULE_NAME("trap");
+__KERNEL_META("$Hyra$: trap.c, Ian Marco Moffett, "
+              "Trap handling");
+
 static const char *trap_type[] = {
     [TRAP_BREAKPOINT]   = "breakpoint",
     [TRAP_ARITH_ERR]    = "arithmetic error",
@@ -141,6 +145,33 @@ handle_fatal(struct trapframe *tf)
     panic("Halted\n");
 }
 
+/*
+ * Handle a pagefault occuring in the userland
+ *
+ * @curtd: Current thread.
+ * @tf: Trapframe.
+ * @sched_tmr: Scheduler timer.
+ */
+static void
+handle_user_pf(struct proc *curtd, struct trapframe *tf)
+{
+    uintptr_t fault_addr;
+    vm_prot_t access_type;
+    int s;
+
+    fault_addr = pf_faultaddr();
+    access_type = pf_accesstype(tf);
+    s = vm_fault(fault_addr, access_type);
+
+    if (s != 0) {
+        KERR("Got page fault @ 0x%p\n", fault_addr);
+        KERR("Fault access mask: 0x%x\n", access_type);
+        KERR("Raising SIGSEGV to PID %d...\n", curtd->pid);
+        regdump(tf);
+        signal_raise(curtd, SIGSEGV);
+    }
+}
+
 /*
  * Handles traps.
  */
@@ -150,7 +181,6 @@ trap_handler(struct trapframe *tf)
     struct proc *curtd = this_td();
     struct timer sched_tmr;
     tmrr_status_t tmrr_s;
-    int s;
 
     /*
      * Mask interrupts so we don't get put in a funky
@@ -192,14 +222,17 @@ trap_handler(struct trapframe *tf)
 
     switch (tf->trapno) {
     case TRAP_ARITH_ERR:
+        KERR("Got arithmetic error - raising SIGFPE...\n");
+        KERR("SIGFPE -> PID %d\n", curtd->pid);
         signal_raise(curtd, SIGFPE);
         break;
     case TRAP_PAGEFLT:
-        s = vm_fault(pf_faultaddr(), pf_accesstype(tf));
-        if (s != 0)
-            signal_raise(curtd, SIGSEGV);
+        handle_user_pf(curtd, tf);
         break;
     default:
+        KERR("Got %s - raising SIGSEGV...\n", trap_type[tf->trapno]);
+        KERR("SIGSEGV -> PID %d\n", curtd->pid);
+        regdump(tf);
         signal_raise(curtd, SIGSEGV);
         break;
     }
-- 
cgit v1.2.3