fix: 10 separate fs functions, SF_FILE constant, pub(crate) FsBlock

examples/fs_demo.rs

@@ -1,12 +1,10 @@
 #![no_std]
 #![no_main]
 
-use kos::fs::fs_read;
+use kos::fs::read_file;
 
 #[no_mangle]
-pub extern "C" fn _start() -> ! {
-    let path = c"/rd/1/readme.txt";
-    let mut buf = [0u8; 512];
-    let (_status, _bytes) = fs_read(path, &mut buf);
-    loop {}
+pub fn kol_main() {
+    let mut buf = [0u8; 256];
+    let _ = read_file(c"/rd/1/readme.txt", &mut buf);
 }

src/fs.rs

@@ -1,14 +1,58 @@
 use core::ffi::CStr;
-use crate::sys::{syscall_fs, FileSystemRequest};
+use crate::sys::{fs_execute, FsBlock};
 
-pub fn fs_read(path: &CStr, buf: &mut [u8]) -> (u32, u32) {
-    let req = FileSystemRequest {
-        operation:   0,
-        offset_low:  0,
-        offset_high: 0,
-        byte_count:  buf.len() as u32,
-        buffer:      buf.as_mut_ptr() as u32,
-        path:        path.as_ptr() as u32,
+fn execute(op: u32, offset: u32, offset_hi: u32, count: u32, buf: *mut u8, path: &CStr) -> Result<(), u32> {
+    let info = FsBlock {
+        operation:  op,
+        offset,
+        offset_hi,
+        byte_count: count,
+        buffer:     buf,
+        _pad:       0,
+        path:       path.as_ptr() as *const u8,
     };
-    unsafe { syscall_fs(&req) }
+    match unsafe { fs_execute(&info) } {
+        0 => Ok(()),
+        e => Err(e),
+    }
+}
+
+pub fn read_file(path: &CStr, buf: &mut [u8]) -> Result<(), u32> {
+    execute(0, 0, 0, buf.len() as u32, buf.as_mut_ptr(), path)
+}
+
+pub fn read_folder(path: &CStr, buf: &mut [u8]) -> Result<(), u32> {
+    execute(1, 0, 0, buf.len() as u32, buf.as_mut_ptr(), path)
+}
+
+pub fn create_file(path: &CStr, buf: &[u8]) -> Result<(), u32> {
+    execute(2, 0, 0, buf.len() as u32, buf.as_ptr() as *mut u8, path)
+}
+
+pub fn write_file(path: &CStr, buf: &[u8]) -> Result<(), u32> {
+    execute(3, 0, 0, buf.len() as u32, buf.as_ptr() as *mut u8, path)
+}
+
+pub fn set_size(path: &CStr, size: u64) -> Result<(), u32> {
+    execute(4, size as u32, (size >> 32) as u32, 0, core::ptr::null_mut(), path)
+}
+
+pub fn get_attributes(path: &CStr, buf: &mut [u8]) -> Result<(), u32> {
+    execute(5, 0, 0, buf.len() as u32, buf.as_mut_ptr(), path)
+}
+
+pub fn set_attributes(path: &CStr, buf: &[u8]) -> Result<(), u32> {
+    execute(6, 0, 0, buf.len() as u32, buf.as_ptr() as *mut u8, path)
+}
+
+pub fn run(path: &CStr) -> Result<(), u32> {
+    execute(7, 0, 0, 0, core::ptr::null_mut(), path)
+}
+
+pub fn delete(path: &CStr) -> Result<(), u32> {
+    execute(8, 0, 0, 0, core::ptr::null_mut(), path)
+}
+
+pub fn create_folder(path: &CStr) -> Result<(), u32> {
+    execute(9, 0, 0, 0, core::ptr::null_mut(), path)
 }

src/func_constants.inc

@@ -20,3 +20,4 @@ SF_SYS_MISC             = 68
     SSF_MEM_ALLOC   = 12
     SSF_MEM_FREE    = 13
     SSF_LOAD_DLL    = 19
+SF_FILE                 = 70

src/sys.rs

@@ -67,27 +67,19 @@ extern "C" {
     // 68.19
     #[link_name = "_load_dll"]
     pub fn load_dll(name: *const u8) -> *const u32;
+
+    // 70
+    #[link_name = "_fs_execute"]
+    pub fn fs_execute(info: *const FsBlock) -> u32;
 }
 
 #[repr(C)]
-pub struct FileSystemRequest {
-    pub operation:    u32,
-    pub offset_low:   u32,
-    pub offset_high:  u32,
-    pub byte_count:   u32,
-    pub buffer:       u32,
-    pub path:         u32,
-}
-
-pub unsafe fn syscall_fs(req: *const FileSystemRequest) -> (u32, u32) {
-    let eax_result: u32;
-    let ebx_result: u32;
-    core::arch::asm!(
-        "int 0x40",
-        inout("eax") 70u32 => eax_result,
-        in("ebx") req,
-        out("ebx") ebx_result,
-        options(nostack)
-    );
-    (eax_result, ebx_result)
+pub(crate) struct FsBlock {
+    pub operation:  u32,
+    pub offset:     u32,
+    pub offset_hi:  u32,
+    pub byte_count: u32,
+    pub buffer:     *mut u8,
+    pub _pad:       u8,
+    pub path:       *const u8,
 }

src/syscalls.S

@@ -20,6 +20,7 @@ section '.text'
     public _get_lang
     public _load_dll
     public _set_event_mask
+    public _fs_execute
 
 _exit:
     mov eax, SF_TERMINATE_PROCESS
@@ -143,3 +144,10 @@ _set_event_mask:
     int 0x40
     ret
 
+_fs_execute:
+    push ebx
+    mov eax, SF_FILE
+    mov ebx, [esp + 4 * 2]
+    int 0x40
+    pop ebx
+    ret