TODO figure out why schedrelease() messes with terminal proc's ring buffer

kernel/hal/x86_64/intr.c

@@ -17,8 +17,6 @@
 #include "proc/ps2kbproc/ps2kbproc.h"
 #include "rbuf/rbuf.h"
 
-IntrStackFrame *INTR_CURRENT_FRAME;
-
 typedef struct BackTraceFrame {
   struct BackTraceFrame *rbp;
   uint64_t rip;
@@ -182,8 +180,8 @@ void hal_syscalldispatch(IntrStackFrame *frame) {
     }
     uint64_t cr3;
     asm volatile("mov %%cr3, %0" : "=r"(cr3));
-    int32_t ret = fn(frame->regs.rdi, frame->regs.rsi, frame->regs.rdx,
-                     frame->regs.r10, frame->regs.r8,  frame->regs.r9);
+    int32_t ret = fn(frame, frame->regs.rdi, frame->regs.rsi, frame->regs.rdx,
+                            frame->regs.r10, frame->regs.r8,  frame->regs.r9);
 
     if (ret == E_DOSCHEDULING) {
       proc_sched((void *)frame);
@@ -192,14 +190,12 @@ void hal_syscalldispatch(IntrStackFrame *frame) {
   }
 }
 
-void intr_eoi(uint8_t irq) {
+void intr_eoi() {
   io_out8(PIC2_CMD, PIC_EOI);
   io_out8(PIC1_CMD, PIC_EOI);
 }
 
 void intr_handleintr(IntrStackFrame *frame) {
-  INTR_CURRENT_FRAME = frame;
-
   if (frame->trapnum <= 31) {
     kprintf("ERROR %s, 0x%lX\n", exceptions[frame->trapnum], frame->errnum);
     intr_dumpframe(frame);
@@ -214,7 +210,7 @@ void intr_handleintr(IntrStackFrame *frame) {
     switch (frame->trapnum) {
       case INTR_IRQBASE+0:
         PIT_TICKS++;
-        intr_eoi(frame->trapnum - INTR_IRQBASE);
+        intr_eoi();
         proc_sched((void *)frame);
         break;
       case INTR_IRQBASE+1:
@@ -225,7 +221,7 @@ void intr_handleintr(IntrStackFrame *frame) {
           rbuf_push(&PS2KB_BUF.rbuf, b);
           spinlock_release(&PS2KB_BUF.spinlock);
         }
-        intr_eoi(frame->trapnum - INTR_IRQBASE);
+        intr_eoi();
         break;
     }
   } else if (frame->trapnum == 0x80) {

kernel/hal/x86_64/intr.h

@@ -35,6 +35,4 @@ typedef struct {
 
 void intr_init(void);
 
-IntrStackFrame *INTR_CURRENT_FRAME;
-
 #endif // HAL_INTR_H_

kernel/proc/termproc/termproc.c

@@ -15,15 +15,11 @@ void termproc_init(Proc *proc) {
 }
 
 void termproc_fn(void) {
-  char buf[100];
   for (;;) {
-    hal_memset(buf, 0, sizeof(buf));
-    spinlock_acquire(&TERMPROC->pipes_spinlock);
-    IpcPipe *inpipe = TERMPROC->pipes[1];
-    spinlock_release(&TERMPROC->pipes_spinlock);
-    int32_t read = ipc_piperead(inpipe, (uint8_t *)buf, sizeof(buf));
-    if (read > 0) {
-      kprintf("%.*s", read, buf);
+    char c = 0;
+    int32_t read = ipc_piperead(TERMPROC->pipes[1], (uint8_t *)&c, 1);
+    if (read > 0 && c != 0) {
+      kprintf("%c", c);
     }
   }
 }

kernel/spinlock/spinlock.c

@@ -12,12 +12,14 @@ void spinlock_init(SpinLock *sl) {
 
 uint64_t spinlock_irqsave(void) {
   uint64_t flags;
-  asm volatile("pushfq; cli; popq %0" : "=r"(flags));
+  asm volatile("pushfq; cli; popq %0" : "=r"(flags) :: "memory");
   return flags;
 }
 
 void spinlock_irqrestore(uint64_t flags) {
-  asm volatile("pushq %0; popfq" :: "r"(flags));
+  if (flags & (1<<9)) {
+    asm volatile("sti" ::: "memory");
+  }
 }
 
 void spinlock_acquire(SpinLock *sl) {

kernel/syscall/sched.c

@@ -11,9 +11,7 @@ int32_t SYSCALL0(sys_schedrelease) {
   Proc *proc = PROCS.current;
   spinlock_release(&PROCS.spinlock);
 
-  if (INTR_CURRENT_FRAME != NULL) {
-    proc_sched((void *)INTR_CURRENT_FRAME);
-  }
+  /* proc_sched((void *)frame); */
 
   return E_OK;
 }

kernel/syscall/syscall.h

@@ -3,11 +3,13 @@
 
 #include <stdint.h>
 #include "compiler/attr.h"
+#include "hal/hal.h"
 
 #define SYSCALLS_MAX 0xff
 
 #define SYSCALL0(name) \
-  name(UNUSED uint64_t _1, \
+  name(IntrStackFrame *frame, \
+       UNUSED uint64_t _1, \
        UNUSED uint64_t _2, \
        UNUSED uint64_t _3, \
        UNUSED uint64_t _4, \
@@ -16,7 +18,8 @@
       )
 
 #define SYSCALL1(name, arg1) \
-  name(uint64_t arg1, \
+  name(IntrStackFrame *frame, \
+       uint64_t arg1, \
        UNUSED uint64_t _2, \
        UNUSED uint64_t _3, \
        UNUSED uint64_t _4, \
@@ -25,7 +28,8 @@
       )
 
 #define SYSCALL2(name, arg1, arg2) \
-  name(uint64_t arg1, \
+  name(IntrStackFrame *frame, \
+       uint64_t arg1, \
        uint64_t arg2, \
        UNUSED uint64_t _3, \
        UNUSED uint64_t _4, \
@@ -34,7 +38,8 @@
       )
 
 #define SYSCALL3(name, arg1, arg2, arg3) \
-  name(uint64_t arg1, \
+  name(IntrStackFrame *frame, \
+       uint64_t arg1, \
        uint64_t arg2, \
        uint64_t arg3, \
        UNUSED uint64_t _4, \
@@ -43,7 +48,8 @@
       )
 
 #define SYSCALL4(name, arg1, arg2, arg3, arg4) \
-  name(uint64_t arg1, \
+  name(IntrStackFrame *frame, \
+       uint64_t arg1, \
        uint64_t arg2, \
        uint64_t arg3, \
        uint64_t arg4, \
@@ -52,7 +58,8 @@
       )
 
 #define SYSCALL5(name, arg1, arg2, arg3, arg4, arg5) \
-  name(uint64_t arg1, \
+  name(IntrStackFrame *frame, \
+       uint64_t arg1, \
        uint64_t arg2, \
        uint64_t arg3, \
        uint64_t arg4, \
@@ -61,7 +68,8 @@
       )
 
 #define SYSCALL6(name, arg1, arg2, arg3, arg4, arg5, arg6) \
-  name(uint64_t arg1, \
+  name(IntrStackFrame *frame, \
+       uint64_t arg1, \
        uint64_t arg2, \
        uint64_t arg3, \
        uint64_t arg4, \
@@ -69,7 +77,7 @@
        uint64_t arg6 \
       )
 
-typedef int32_t (*SyscallFn)(uint64_t, uint64_t, uint64_t, uint64_t, uint64_t, uint64_t);
+typedef int32_t (*SyscallFn)(IntrStackFrame *, uint64_t, uint64_t, uint64_t, uint64_t, uint64_t, uint64_t);
 
 extern SyscallFn SYSCALL_TABLE[SYSCALLS_MAX];