From 687d18815da5742d201e876e71b8c0209a097a21 Mon Sep 17 00:00:00 2001
From: dzwdz
Date: Sat, 24 Jul 2021 21:18:33 +0200
Subject: sysenter: save the registers into _sysexit_regs, makes the code
cleaner (imo)
---
src/kernel/arch/i386/sysenter.c | 26 ++++++++------------------
1 file changed, 8 insertions(+), 18 deletions(-)
(limited to 'src/kernel/arch/i386/sysenter.c')
diff --git a/src/kernel/arch/i386/sysenter.c b/src/kernel/arch/i386/sysenter.c
index a7c7123..a2810b2 100644
--- a/src/kernel/arch/i386/sysenter.c
+++ b/src/kernel/arch/i386/sysenter.c
@@ -22,26 +22,16 @@ void sysexit(struct registers regs) {
_sysexit_real();
}
-void sysenter_stage2(int edi, int esi, void *ebp, void *esp,
- int ebx, int edx, int ecx, int eax)
-{
+_Noreturn void sysenter_stage2() {
uint64_t val;
- process_current->regs = (struct registers) {
- // EAX and EDX will get overriden with the return value later on
-
- .eax = eax,
- .ecx = ecx,
- .edx = edx,
- .ebx = ebx,
- .esi = esi,
- .edi = edi,
-
- .esp = (void*) ecx, // not a typo, part of my calling convention
- .eip = (void*) edx, // ^
- .ebp = ebp,
- };
+ struct registers *regs = &process_current->regs;
+
+ *regs = _sysexit_regs; // save the registers
+ regs->esp = regs->ecx; // fix them up
+ regs->eip = regs->edx;
- val = syscall_handler(eax, ebx, esi, edi);
+ val = syscall_handler(regs->eax, regs->ebx,
+ regs->esi, regs->edi);
regs_savereturn(&process_current->regs, val);
process_switch(process_current); // TODO process_resume
--
cgit v1.2.3