make map_range_with_table_flags and map_range unsafe

Author: Freax13

src/structures/paging/mapper/mapped_page_table.rs

@@ -598,7 +598,7 @@ impl<'a, P: PageTableFrameMapping> Mapper<Size1GiB> for MappedPageTable<'a, P> {
     }
 
     #[inline]
-    fn map_range_with_table_flags<A>(
+    unsafe fn map_range_with_table_flags<A>(
         &mut self,
         pages: PageRange<Size1GiB>,
         flags: PageTableFlags,
@@ -801,7 +801,7 @@ impl<'a, P: PageTableFrameMapping> Mapper<Size2MiB> for MappedPageTable<'a, P> {
     }
 
     #[inline]
-    fn map_range_with_table_flags<A>(
+    unsafe fn map_range_with_table_flags<A>(
         &mut self,
         pages: PageRange<Size2MiB>,
         flags: PageTableFlags,
@@ -1024,7 +1024,7 @@ impl<'a, P: PageTableFrameMapping> Mapper<Size4KiB> for MappedPageTable<'a, P> {
     }
 
     #[inline]
-    fn map_range_with_table_flags<A>(
+    unsafe fn map_range_with_table_flags<A>(
         &mut self,
         pages: PageRange<Size4KiB>,
         flags: PageTableFlags,

src/structures/paging/mapper/mod.rs

@@ -380,10 +380,15 @@ pub trait Mapper<S: PageSize> {
 
     /// Maps frames from the allocator to the given range of virtual pages.
     ///
+    /// ## Safety
+    ///
+    /// This function invokes [`Mapper::map_to_with_table_flags`] internally, so
+    /// all safety requirements of it also apply for this function.
+    ///
     /// ## Errors
     ///
     /// If an error occurs half-way through a [`MapperFlushRange<S>`] is returned that contains the frames that were successfully mapped.
-    fn map_range_with_table_flags<A>(
+    unsafe fn map_range_with_table_flags<A>(
         &mut self,
         mut pages: PageRange<S>,
         flags: PageTableFlags,
@@ -427,11 +432,16 @@ pub trait Mapper<S: PageSize> {
 
     /// Maps frames from the allocator to the given range of virtual pages.
     ///
+    /// ## Safety
+    ///
+    /// This function invokes [`Mapper::map_to_with_table_flags`] internally, so
+    /// all safety requirements of it also apply for this function.
+    ///
     /// ## Errors
     ///
     /// If an error occurs half-way through a [`MapperFlushRange<S>`] is returned that contains the frames that were successfully mapped.
     #[inline]
-    fn map_range<A>(
+    unsafe fn map_range<A>(
         &mut self,
         pages: PageRange<S>,
         flags: PageTableFlags,
@@ -446,7 +456,9 @@ pub trait Mapper<S: PageSize> {
                 | PageTableFlags::WRITABLE
                 | PageTableFlags::USER_ACCESSIBLE);
 
-        self.map_range_with_table_flags(pages, flags, parent_table_flags, frame_allocator)
+        unsafe {
+            self.map_range_with_table_flags(pages, flags, parent_table_flags, frame_allocator)
+        }
     }
 
     /// Removes a mapping from the page table and returns the frame that used to be mapped.

src/structures/paging/mapper/offset_page_table.rs

@@ -102,7 +102,7 @@ impl<'a> Mapper<Size1GiB> for OffsetPageTable<'a> {
     }
 
     #[inline]
-    fn map_range_with_table_flags<A>(
+    unsafe fn map_range_with_table_flags<A>(
         &mut self,
         pages: PageRange<Size1GiB>,
         flags: PageTableFlags,
@@ -112,8 +112,10 @@ impl<'a> Mapper<Size1GiB> for OffsetPageTable<'a> {
     where
         A: FrameAllocator<Size4KiB> + FrameAllocator<Size1GiB> + ?Sized,
     {
-        self.inner
-            .map_range_with_table_flags(pages, flags, parent_table_flags, allocator)
+        unsafe {
+            self.inner
+                .map_range_with_table_flags(pages, flags, parent_table_flags, allocator)
+        }
     }
 
     #[inline]
@@ -223,7 +225,7 @@ impl<'a> Mapper<Size2MiB> for OffsetPageTable<'a> {
     }
 
     #[inline]
-    fn map_range_with_table_flags<A>(
+    unsafe fn map_range_with_table_flags<A>(
         &mut self,
         pages: PageRange<Size2MiB>,
         flags: PageTableFlags,
@@ -233,8 +235,10 @@ impl<'a> Mapper<Size2MiB> for OffsetPageTable<'a> {
     where
         A: FrameAllocator<Size4KiB> + FrameAllocator<Size2MiB> + ?Sized,
     {
-        self.inner
-            .map_range_with_table_flags(pages, flags, parent_table_flags, allocator)
+        unsafe {
+            self.inner
+                .map_range_with_table_flags(pages, flags, parent_table_flags, allocator)
+        }
     }
 
     #[inline]
@@ -344,7 +348,7 @@ impl<'a> Mapper<Size4KiB> for OffsetPageTable<'a> {
     }
 
     #[inline]
-    fn map_range_with_table_flags<A>(
+    unsafe fn map_range_with_table_flags<A>(
         &mut self,
         pages: PageRange<Size4KiB>,
         flags: PageTableFlags,
@@ -354,8 +358,10 @@ impl<'a> Mapper<Size4KiB> for OffsetPageTable<'a> {
     where
         A: FrameAllocator<Size4KiB> + ?Sized,
     {
-        self.inner
-            .map_range_with_table_flags(pages, flags, parent_table_flags, allocator)
+        unsafe {
+            self.inner
+                .map_range_with_table_flags(pages, flags, parent_table_flags, allocator)
+        }
     }
 
     #[inline]

src/structures/paging/mapper/recursive_page_table.rs

@@ -767,7 +767,7 @@ impl<'a> Mapper<Size1GiB> for RecursivePageTable<'a> {
     }
 
     #[inline]
-    fn map_range_with_table_flags<A>(
+    unsafe fn map_range_with_table_flags<A>(
         &mut self,
         pages: PageRange<Size1GiB>,
         flags: PageTableFlags,
@@ -984,7 +984,7 @@ impl<'a> Mapper<Size2MiB> for RecursivePageTable<'a> {
     }
 
     #[inline]
-    fn map_range_with_table_flags<A>(
+    unsafe fn map_range_with_table_flags<A>(
         &mut self,
         pages: PageRange<Size2MiB>,
         flags: PageTableFlags,
@@ -1236,7 +1236,7 @@ impl<'a> Mapper<Size4KiB> for RecursivePageTable<'a> {
     }
 
     #[inline]
-    fn map_range_with_table_flags<A>(
+    unsafe fn map_range_with_table_flags<A>(
         &mut self,
         pages: PageRange<Size4KiB>,
         flags: PageTableFlags,