Merge pull request #4 from arceos-org/fix-present

arceos-org · Aug 9, 2024 · df614b5 · df614b5
2 parents 986e257 + d3134fe
commit df614b5
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/page_table_multiarch/src/bits64.rs b/page_table_multiarch/src/bits64.rs
@@ -104,7 +104,7 @@ impl<M: PagingMetaData, PTE: GenericPTE, H: PagingHandler> PageTable64<M, PTE, H
         flags: MappingFlags,
     ) -> PagingResult<(PageSize, TlbFlush<M>)> {
         let (entry, size) = self.get_entry_mut(vaddr)?;
-        if entry.is_unused() {
+        if !entry.is_present() {
             return Err(PagingError::NotMapped);
         }
         entry.set_flags(flags, size.is_huge());
@@ -117,7 +117,8 @@ impl<M: PagingMetaData, PTE: GenericPTE, H: PagingHandler> PageTable64<M, PTE, H
     /// mapping is not present.
     pub fn unmap(&mut self, vaddr: M::VirtAddr) -> PagingResult<(PhysAddr, PageSize, TlbFlush<M>)> {
         let (entry, size) = self.get_entry_mut(vaddr)?;
-        if entry.is_unused() {
+        if !entry.is_present() {
+            entry.clear();
             return Err(PagingError::NotMapped);
         }
         let paddr = entry.paddr();