Tentative API

src/kernel/arch/x86_64/asm.rs

@@ -6,4 +6,4 @@ pub unsafe fn halt() {
     unsafe {
         asm!("hlt");
     }
-}
+}

src/kernel/syscall_runner.rs

@@ -3,20 +3,29 @@
 use crate::process::Process;
 
 // FUCKING ADD CONST UNWRAP!!!
-pub static KERNEL_VERSION_MAJOR: u8 = match u8::from_str_radix(env!("CARGO_PKG_VERSION_MAJOR"), 10) {
+pub static KERNEL_VERSION_MAJOR: u8 = match u8::from_str_radix(env!("CARGO_PKG_VERSION_MAJOR"), 10)
+{
     Ok(ver) => ver,
-    Err(_) => { panic!("Invalid version number ") },
+    Err(_) => {
+        panic!("Invalid version number ")
+    }
 };
-pub static KERNEL_VERSION_MINOR: u8 = match u8::from_str_radix(env!("CARGO_PKG_VERSION_MINOR"), 10) {
+pub static KERNEL_VERSION_MINOR: u8 = match u8::from_str_radix(env!("CARGO_PKG_VERSION_MINOR"), 10)
+{
     Ok(ver) => ver,
-    Err(_) => { panic!("Invalid version number ") },
+    Err(_) => {
+        panic!("Invalid version number ")
+    }
 };
-pub static KERNEL_VERSION_PATCH: u8 = match u8::from_str_radix(env!("CARGO_PKG_VERSION_PATCH"), 10) {
+pub static KERNEL_VERSION_PATCH: u8 = match u8::from_str_radix(env!("CARGO_PKG_VERSION_PATCH"), 10)
+{
     Ok(ver) => ver,
-    Err(_) => { panic!("Invalid version number ") },
+    Err(_) => {
+        panic!("Invalid version number ")
+    }
 };
 
 #[allow(unused_variables)]
 pub fn run_syscall(process: &mut Process) {
     // Get the syscall ID from the process's CPU registers.
-}
+}

src/lib/arch/aarch64/mod.rs

@@ -0,0 +1 @@
+

src/lib/arch/loongarch64/mod.rs

@@ -0,0 +1 @@
+

src/lib/arch/riscv64/mod.rs

@@ -0,0 +1 @@
+

src/lib/arch/x86_64/mod.rs

@@ -1 +1,2 @@
-mod syscall_impl;
+mod registers_impl;
+mod syscall_impl;

src/lib/arch/x86_64/registers_impl.rs

@@ -0,0 +1,4 @@
+#[allow(dead_code)]
+pub struct Registers {
+    // Private fields
+}

src/lib/arch/x86_64/syscall_impl.rs

@@ -3,7 +3,3 @@
 // The system call API for x86_64.
 
 use crate::syscall::*;
-
-struct Registers {
-
-}

src/lib/lib.rs

@@ -1,5 +1,6 @@
 #![no_std]
 #![allow(unused_imports)]
 mod arch;
+pub mod registers;
 pub mod syscall;
-pub use arch::current::*;
+pub use arch::current::*;

src/lib/registers.rs

@@ -0,0 +1,36 @@
+// Every architecture MUST implement this as part of the ABI.
+// Additional registers can be implemented with architecture-specific traits.
+pub unsafe trait RegStoreLoad
+where
+    Self: Sized,
+    Self: Default,
+{
+    // Create an instance of Registers by loading values from the CPU.
+    // This MUST NOT modify any CPU registers, and can only read them.
+    fn load_all() -> Self;
+    // Create a new instance to write to before we commit to the CPU.
+    // This MUST instantiate a zeroed set of registers.
+    // Do not represent unmodified registers as None variants of an Option.
+    fn new_zeroed() -> Self {
+        Self::default()
+    }
+    // Store the instance to the CPU and consume self.
+    // This MUST write ALL register values to the CPU, including zero values.
+    fn store_all(self);
+
+    // All of these functions below MUST NOT write directly to the CPU.
+    // The registers will be written when store_all() is called.
+
+    // Get the six argument or return register values.
+    // This MUST read registers in the same order as set_syscall_args sets them.
+    fn read_syscall_args(&self) -> [usize; 6];
+    // Get the syscall number or error number.
+    // This MUST NOT zero or otherwise overwrite the register.
+    fn read_syscall_num(&self) -> usize;
+    // Set the six argument or return register values.
+    // This MUST write registers in the same order as read_syscall_args reads them.
+    fn set_syscall_args(&mut self, new_values: [usize; 6]);
+    // Set the syscall number or error number.
+    // This MUST NOT change any other registers.
+    fn set_syscall_num(&mut self, new_value: usize);
+}

src/lib/syscall.rs

@@ -1,7 +1,7 @@
 #![allow(dead_code)]
 
 // TODO: Implement a nice API for system calls.
-// I don't want to have to define the argument/return value registers twice 
+// I don't want to have to define the argument/return value registers twice
 // per architecture. How do I make this work?
 
 pub enum SyscallArgs {
@@ -11,15 +11,16 @@ pub enum SyscallArgs {
     Args3(usize, usize, usize),
     Args4(usize, usize, usize, usize),
     Args5(usize, usize, usize, usize, usize),
-    Args6(usize, usize, usize, usize, usize, usize)
+    Args6(usize, usize, usize, usize, usize, usize),
 }
 
-#[repr(u16)]
+#[repr(usize)]
 pub enum SyscallError {
-    Ok = 0,
+    Ok,               // No error.
-    SyscallNotExist = 1,
+    Unspecified,      // Unspecified error occurred.
-    FileNotExist = 2,
+    SyscallNotExist,  // System call does not exist.
-    ProcessNotExist = 3,
+    FileNotExist,     // The file mentioned does not exist.
-    PermissionDenied = 4,
+    ProcessNotExist,  // The process mentioned does not exist.
-    Aborted = 5,
+    PermissionDenied, // The process lacks capabilities.
-}
+    Aborted,          // The kernel gave up on a blocking request.
+}