Merge pull request #28 from nekernel-org/dev

0.0.2

1 files changed, 83 insertions, 98 deletions

diff --git a/dev/kernel/src/PageMgr.cc b/dev/kernel/src/PageMgr.cc
index 8778a855..f60fa57a 100644
--- a/dev/kernel/src/PageMgr.cc
+++ b/dev/kernel/src/PageMgr.cc
@@ -1,6 +1,6 @@
 /* -------------------------------------------
 
-	Copyright (C) 2024-2025, Amlal El Mahrouss, all rights reserved.
+  Copyright (C) 2024-2025, Amlal El Mahrouss, all rights reserved.
 
 ------------------------------------------- */
 
@@ -11,100 +11,85 @@
 #include <HALKit/AMD64/Paging.h>
 #elif defined(__NE_ARM64__)
 #include <HALKit/ARM64/Paging.h>
-#endif // ifdef __NE_AMD64__ || defined(__NE_ARM64__)
-
-namespace Kernel
-{
-	PTEWrapper::PTEWrapper(Boolean Rw, Boolean User, Boolean ExecDisable, UIntPtr VirtAddr)
-		: fRw(Rw),
-		  fUser(User),
-		  fExecDisable(ExecDisable),
-		  fVirtAddr(VirtAddr),
-		  fCache(false),
-		  fShareable(false),
-		  fWt(false),
-		  fPresent(true),
-		  fAccessed(false)
-	{
-	}
-
-	PTEWrapper::~PTEWrapper() = default;
-
-	/// @brief Flush virtual address.
-	/// @param VirtAddr
-	Void PageMgr::FlushTLB()
-	{
-#ifndef __NE_MINIMAL_OS__
-		hal_flush_tlb();
-#endif // !__NE_MINIMAL_OS__
-	}
-
-	/// @brief Reclaim freed page.
-	/// @return
-	Bool PTEWrapper::Reclaim()
-	{
-		if (!this->fPresent)
-		{
-			this->fPresent = true;
-			return true;
-		}
-
-		return false;
-	}
-
-	/// @brief Request a PTE.
-	/// @param Rw r/w?
-	/// @param User user mode?
-	/// @param ExecDisable disable execution on page?
-	/// @return
-	PTEWrapper PageMgr::Request(Boolean Rw, Boolean User, Boolean ExecDisable, SizeT Sz, SizeT Pad)
-	{
-		// Store PTE wrapper right after PTE.
-		VoidPtr ptr = Kernel::HAL::mm_alloc_bitmap(Rw, User, Sz, NO, Pad);
-
-		return PTEWrapper{Rw, User, ExecDisable, reinterpret_cast<UIntPtr>(ptr)};
-	}
-
-	/// @brief Disable BitMap.
-	/// @param wrapper the wrapper.
-	/// @return If the page bitmap was cleared or not.
-	Bool PageMgr::Free(Ref<PTEWrapper>& wrapper)
-	{
-		if (!Kernel::HAL::mm_free_bitmap((VoidPtr)wrapper.Leak().VirtualAddress()))
-			return false;
-
-		return true;
-	}
-
-	/// @brief Virtual PTE address.
-	/// @return The virtual address of the page.
-	UIntPtr PTEWrapper::VirtualAddress()
-	{
-		return (fVirtAddr);
-	}
-
-	Bool PTEWrapper::Shareable()
-	{
-		return fShareable;
-	}
-
-	Bool PTEWrapper::Present()
-	{
-		return fPresent;
-	}
-
-	Bool PTEWrapper::Access()
-	{
-		return fAccessed;
-	}
-
-	Void PTEWrapper::NoExecute(const bool enable)
-	{
-		fExecDisable = enable;
-	}
-
-	Bool PTEWrapper::NoExecute()
-	{
-		return fExecDisable;
-	}
-} // namespace Kernel
+#endif  // ifdef __NE_AMD64__ || defined(__NE_ARM64__)
+
+namespace Kernel {
+PTEWrapper::PTEWrapper(Boolean Rw, Boolean User, Boolean ExecDisable, UIntPtr VirtAddr)
+    : fRw(Rw),
+      fUser(User),
+      fExecDisable(ExecDisable),
+      fVirtAddr(VirtAddr),
+      fCache(false),
+      fShareable(false),
+      fWt(false),
+      fPresent(true),
+      fAccessed(false) {}
+
+PTEWrapper::~PTEWrapper() = default;
+
+/// @brief Flush virtual address.
+/// @param VirtAddr
+Void PageMgr::FlushTLB() {
+#ifdef __NE_VIRTUAL_MEMORY_SUPPORT__
+  hal_flush_tlb();
+#endif  // !__NE_VIRTUAL_MEMORY_SUPPORT__
+}
+
+/// @brief Reclaim freed page.
+/// @return
+Bool PTEWrapper::Reclaim() {
+  if (!this->fPresent) {
+    this->fPresent = true;
+    return true;
+  }
+
+  return false;
+}
+
+/// @brief Request a PTE.
+/// @param Rw r/w?
+/// @param User user mode?
+/// @param ExecDisable disable execution on page?
+/// @return
+PTEWrapper PageMgr::Request(Boolean Rw, Boolean User, Boolean ExecDisable, SizeT Sz, SizeT Pad) {
+  // Store PTE wrapper right after PTE.
+  VoidPtr ptr = Kernel::HAL::mm_alloc_bitmap(Rw, User, Sz, NO, Pad);
+
+  return PTEWrapper{Rw, User, ExecDisable, reinterpret_cast<UIntPtr>(ptr)};
+}
+
+/// @brief Disable BitMap.
+/// @param wrapper the wrapper.
+/// @return If the page bitmap was cleared or not.
+Bool PageMgr::Free(Ref<PTEWrapper>& wrapper) {
+  if (!Kernel::HAL::mm_free_bitmap((VoidPtr) wrapper.Leak().VirtualAddress())) return false;
+
+  return true;
+}
+
+/// @brief Virtual PTE address.
+/// @return The virtual address of the page.
+UIntPtr PTEWrapper::VirtualAddress() {
+  return (fVirtAddr);
+}
+
+Bool PTEWrapper::Shareable() {
+  return fShareable;
+}
+
+Bool PTEWrapper::Present() {
+  return fPresent;
+}
+
+Bool PTEWrapper::Access() {
+  return fAccessed;
+}
+
+Void PTEWrapper::NoExecute(const bool enable) {
+  fExecDisable = enable;
+}
+
+Bool PTEWrapper::NoExecute() {
+  return fExecDisable;
+}
+}  // namespace Kernel