chore: add conversion between PageError and SysError

kernel/src/objects/frame.rs

@@ -1,5 +1,9 @@
 use super::{cap::RawCap, Cap, KernelObject};
-use crate::{arch::layout::mmap_phys_to_virt, objects::cap::CapEntry, vspace::*};
+use crate::{
+    arch::layout::{mmap_phys_to_virt, PAGE_SIZE},
+    objects::cap::CapEntry,
+    vspace::*,
+};
 use uapi::{
     cap::ObjectType,
     error::{SysError, SysResult},
@@ -50,8 +54,7 @@ impl<'a> FrameCap<'a> {
     pub fn mint(ptr: PhysAddr, size: usize, attr: MapAttr, is_device: bool) -> RawCap {
         let size_bits = size.ilog2() as usize;
         debug_assert!(size_bits <= FrameCap::FRAME_SIZE_BITS);
-
-        // NOTE: we are not checking frame size
+        assert!(size >= PAGE_SIZE);
 
         let arg0 = 0
             | ((attr.bits() & Self::VM_RIGHT_MASK) << Self::VM_RIGHT_OFFSET)
@@ -126,14 +129,10 @@ impl<'a> FrameCap<'a> {
         self.as_object_mut().fill(fill.unwrap_or(0));
     }
 
-    pub fn map_page<T: TableLevel>(&self, root: &mut Table<T>, vaddr: VirtAddr, attr: MapAttr) -> SysResult {
+    pub fn map<T: TableLevel>(&self, root: &mut Table<T>, vaddr: VirtAddr, attr: MapAttr) -> SysResult {
         let masked_attr = attr & self.attr();
 
-        root.map(vaddr, self.cte.cap.get().ptr, masked_attr).map_err(|e| match e {
-            PageError::AlreadyMapped => SysError::AlreadyMapped,
-            PageError::MissingEntry => SysError::MissingEntry,
-            PageError::NotAligned => SysError::InvalidArgument,
-        })?;
+        root.map(vaddr, self.cte.cap.get().ptr, masked_attr)?;
 
         self.set_mapped_asid(0);
         self.set_mapped_vaddr(vaddr);
@@ -147,11 +146,13 @@ impl<'a> FrameCap<'a> {
             return Err(SysError::NotMapped);
         }
 
-        match root.lookup_mut(self.mapped_vaddr()) {
+        match root.lookup_mut(vaddr) {
             Some(entry) if entry.is_leaf() && entry.paddr() == self.cte.cap.get().ptr => {
                 entry.set_paddr(PhysAddr::default());
                 entry.set_attr(MapAttr::empty());
 
+                // todo: sfence.vma
+
                 self.set_mapped_asid(0);
                 self.set_mapped_vaddr(VirtAddr(0));
                 Ok(())

kernel/src/objects/table.rs

@@ -100,13 +100,8 @@ impl<'a> TableCap<'a> {
         array.fill(0);
     }
 
-    pub fn map_table<T: TableLevel>(&self, root: &mut Table<T>, vaddr: VirtAddr) -> SysResult {
-        root.map(vaddr, self.cte.cap.get().ptr, MapAttr::PAGE_TABLE)
-            .map_err(|e| match e {
-                PageError::AlreadyMapped => SysError::AlreadyMapped,
-                PageError::MissingEntry => SysError::MissingEntry,
-                PageError::NotAligned => SysError::InvalidArgument,
-            })?;
+    pub fn map<T: TableLevel>(&self, root: &mut Table<T>, vaddr: VirtAddr) -> SysResult {
+        root.map(vaddr, self.cte.cap.get().ptr, MapAttr::PAGE_TABLE)?;
 
         self.set_mapped_asid(0);
         self.set_mapped_vaddr(vaddr);
@@ -120,11 +115,13 @@ impl<'a> TableCap<'a> {
             return Err(SysError::NotMapped);
         }
 
-        match root.lookup_mut(self.mapped_vaddr()) {
+        match root.lookup_mut(vaddr) {
             Some(entry) if !entry.is_leaf() && entry.paddr() == self.cte.cap.get().ptr => {
                 entry.set_paddr(PhysAddr::default());
                 entry.set_attr(MapAttr::empty());
 
+                // todo: sfence.vma
+
                 self.set_mapped_asid(0);
                 self.set_mapped_vaddr(VirtAddr(0));
                 Ok(())

kernel/src/vspace/error.rs

@@ -0,0 +1,20 @@
+use uapi::error::SysError;
+
+#[derive(Debug)]
+pub enum PageError {
+    AlreadyMapped,
+    MissingEntry,
+    NotAligned,
+}
+
+pub type PageResult<T = ()> = Result<T, PageError>;
+
+impl From<PageError> for SysError {
+    fn from(e: PageError) -> Self {
+        match e {
+            PageError::AlreadyMapped => SysError::AlreadyMapped,
+            PageError::MissingEntry => SysError::MissingEntry,
+            PageError::NotAligned => SysError::InvalidArgument,
+        }
+    }
+}

kernel/src/vspace/mod.rs

@@ -1,9 +1,11 @@
 mod addr;
 mod entry;
+mod error;
 mod level;
 mod table;
 
 pub use addr::*;
 pub use entry::*;
+pub use error::*;
 pub use level::*;
 pub use table::*;

kernel/src/vspace/table.rs

@@ -1,16 +1,6 @@
-use super::{MapAttr, TableLevel};
-use core::fmt::Debug;
+use super::{MapAttr, PageResult, TableLevel};
 use utils::addr::{PhysAddr, VirtAddr};
 
-#[derive(Debug)]
-pub enum PageError {
-    AlreadyMapped,
-    MissingEntry,
-    NotAligned,
-}
-
-pub type PageResult<T = ()> = Result<T, PageError>;
-
 pub trait TableOps<'a, T: TableLevel> {
     /// # Safety
     /// `location` must be a page-aligned virtual address and will not be dropped.