xt-sys/exectos
3
5
Fork 2
Code Issues 7 Pull Requests 1 Actions Projects 1 Releases Wiki Activity

Simplify stack page count calculation
Some checks failed
Builds / ExectOS (amd64, debug) (push) Failing after 20s
Details
Builds / ExectOS (i686, release) (push) Failing after 21s
Details
Builds / ExectOS (i686, debug) (push) Failing after 28s
Details
Builds / ExectOS (amd64, release) (push) Failing after 30s
Details

Browse Source
This commit is contained in:
Aiken Harris
2025-12-29 19:14:49 +01:00
parent 4c7c914a1c
commit f4c49e2f25
1 changed files with 5 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
xtoskrnl/mm/kpool.cc
View File

@@ -30,7 +30,7 @@ XTSTATUS
MM::KernelPool::AllocateKernelStack(OUT PVOID *Stack,
IN ULONG StackSize)
{
PFN_COUNT StackPtes, StackPages;
PFN_COUNT StackPages;
PMMPTE PointerPte, StackPte;
MMPTE TempPte, InvalidPte;
PFN_NUMBER PageFrameIndex;
@@ -40,11 +40,10 @@ MM::KernelPool::AllocateKernelStack(OUT PVOID *Stack,
*Stack = NULLPTR;
/* Convert the requested stack size into a page count */
StackPtes = SIZE_TO_PAGES(StackSize);
StackPages = StackPtes;
StackPages = SIZE_TO_PAGES(StackSize);
/* Reserve PTEs for the stack pages, plus a guard page */
StackPte = MM::Pte::ReserveSystemPtes(StackPtes + 1, SystemPteSpace, 0);
StackPte = MM::Pte::ReserveSystemPtes(StackPages + 1, SystemPteSpace, 0);
if(!StackPte)
{
/* Failed to reserve PTEs for the new kernel stack */
@@ -84,10 +83,10 @@ MM::KernelPool::AllocateKernelStack(OUT PVOID *Stack,
}
/* Zero the newly allocated stack memory, skipping the guard page */
RTL::Memory::ZeroMemory(MM::Paging::GetPteVirtualAddress(MM::Paging::GetNextPte(StackPte)), MM_PAGE_SIZE * StackPtes);
RTL::Memory::ZeroMemory(MM::Paging::GetPteVirtualAddress(MM::Paging::GetNextPte(StackPte)), MM_PAGE_SIZE * StackPages);
/* Return a pointer to the top of the new stack */
*Stack = MM::Paging::GetPteVirtualAddress(MM::Paging::AdvancePte(StackPte, StackPtes + 1));
*Stack = MM::Paging::GetPteVirtualAddress(MM::Paging::AdvancePte(StackPte, StackPages + 1));
return STATUS_SUCCESS;
}